Например, не обработка исключений.
Не красивый код, с тупыми переменными аля "а1+суп+ивгы"
Костыли вместо фич платформы.
Запросы и циклы вместо СКД.

как вам количество параметров в отчете? Вместо этого можно было сделать "Контрагент.Подразделение" но переделывать пока лень...
https://s8.hostingkartinok.com/uploads/images/2017/07/8249a3f64013bfea3038c190aa9c90a8.jpg

Во-первых: "работает - не трогай!"
Во-вторых: если речь не идёт о большой совместной разработке (через хранилище и т.д.), твои героические усилия по разгребанию чужого г. не оценит ровным счетом никто. Поэтому и время на это тратить не нужно.
Поэтому в подавляющем большинстве случаев научился ловить дзен и п.4.
Если уж быдлокод откровенно мешает, приходится переписывать, т.е. п. 1.
Пункты 2 и 3 неконструктивны и бессмысленны.

Если работает оставляю как есть,
Заказчику нужно не феншуй наводить, а удоволетворять потребности бизнеса.
чесно сказать сам иногда грешу этим, так заказчику нужно подешевле и поскорее, за подумать и феншуй в большинстве случаев платить не готов,а за бесплатно лишний вечер тратить нет желания.

Сегодня перл прочитал:
ЕстьУОснованияОснованиеОснования = ОбщегоНазначения.ЕстьРеквизитДокумента("ДокументОснование", Основание.ДокументОснование.Метаданные());

Люблю его и холю. Иногда рефакторю. Особенно в запросах.