Признаки плохого программиста

«Бульдозерный код», который создает впечателение рефакторинга посредством разбития кусков кода на процедуры, которые, правда, затем невозможно использовать где-либо еще (высокая когезия).

===========

Иногда удобно разбить написанный кем-то ранее километровый алгоритм на функции, каждая из которых хоть и не вызывается больше нигде, но делает какой-то отдельный шаг и своим названием описывает, что она делает. Т.е. вместо многостраничной простыни, большая ф-ция становится чем-то вроде:

DoXXX(..);
DoYYY(..);
if (smth)
    DoZZZ(...);
else
    DoAAA(...);
...