Здравей Тодоре,

Решението ти е умно направено.

За довършване и за пълни 100 точки, точно както казва Виделин - добре е да си провериш кода за излишни действия - най-нормалното нещо като пишеш код е да надпишеш доста излишни действия, и после може да минеш по кода и да видиш без кое може да се мине - особено ако тестове ти гърмят за време - значи правиш нещо излишно.

Първото, Виделин е прав - никъде в решеието не ползваш int, значи спокойно можеш да си оставиш числата от инпута под формата на стрингове. Това ни спестява две действия: int.Parse(input[i]); и превръщането обратно numbers[a].ToString();

Второто, в условието пише: "The input numbers will be distinct (no duplicates are allowed)."

Това значи, че имаме масив, в който всички числа са различни едно от друго, следователно ако позициите са различни - автоматично и числата на тези позиции ще са различни. Това свежда проверката до:

if (a != b && a != c && a != d && b != c && b != d && c != d)

Няма смисъл да губим време да извикваме числата от масива - сравняваме само позициите.

Трето, добре е винаги да си оглеждаш решението за многократно извършвани действия, като for (int a = 0; a < numbers.Length; a++), ако това numbers.Length ни трябва във всеки цикъл - добре е да го изчислим само веднъж, да го присвоим на променлива, и да ползваме променливата вместо 4 пъти да търсим колко е дължината на масивва. В дадената задача си имаме такава прооменлива, това е n; цикълът става for (int a = 0; a < n; a++).

В твоя защита, аз например съм толкова вманиачена в избягването на излишен код, че напоследък ми се случва да пропускам и необходимия, без да се уверя първо че изобщо не е излишен :)