Раздел 12. Методика разбора проблем для кода 1С
Есть основной сценарий использования:
- есть объектная модель исходного кода - авторы плагина разрабатывают логику анализа проблемных мест: это полностью их право, они читают стандарты и облекают стандарты в правила.
- есть правило - архитектор может его отключить, может включить: это полностью его право.
- есть замечание - в месте кода, где правило сработало, ведущий разработчик может его учесть, как неактуальное, а может исправить: это полностью его право.
- есть технический долг на проекте - менеджер его видит, может согласовывать релиз, а может и не согласовать, это полностью его право.
- Длительные споры исключаются - каждый нажимает те кнопки, которые считает верными. Исключаем “священные войны” как писать правильно, с должным уровнем прозрачности.
- Это контроль качества кода, его уровня. В принципе, можно вообще ничего не улучшать - работает же.
- Речь идет именно о планомерном улучшении сопровождаемости кода, простоте развития. Поэтому разбор проблем выглядит, как принятие решение о целесообразности изменений, предлагаемых сонаром, и выделении под это рабочего времени.
- Минимальная схема работы проста - видим плохой/подозрительный код - исправляем его согласно расширенному описанию правила.