Почему опытные разработчики пишут тупой код и как распознать новичка за километр

все дело в привычке. Для простых условий, как по мне, намного проще прочитать однострочный тенарный оператор чем продиратся через четыре строчки if-else
для меня такой вариант намного понятнее

ClosesType closes = isColdOutside ? COAT : TSHIRT

чем

ClosesType closes;
if (isColdOutside)
    closes = COAT
else
    closes = TSHIRT