На скриншоте за "самодокументирующимся" кодом (ну так гипердлинные идентификаторы обычно оправдывают) прячется вот такая красота:
НеПрервано = Прервано();Такое ощущение, что специально сделали поперек всей логики, чтобы читателю голову сломать, эдакая обфускация.
Если НеПрервано = Ложь Тогда
...
Этот код можно было бы еще "улучшить", добавив отрицание вместо явного сравнения с Ложью. Но
Прервано() тоже спроектирован не удачно и может вернуть не только значение булева типа, поэтому тут обошлось без Если Не НеПрервано Тогда... (а было бы веселее!)Ну и, очевидно, с такой кривой логикой "повествования" (
НеПрервано = Прервано()) какие-либо длинные "самодокументирующиеся" имена становятся не только бесполезными, но и вредят, т.к. в таком идентификаторе при беглом чтении кода "Не" можно и не заметить, или даже наоборот, увидев его в одном идентификаторе, на автомате прочитать во втором, похожем.(Если что, это из модуля 1С:Синхронизация с Битрикс24 для УТ 10.3)
#говнокод #codesmell