Всем привет. Продолжаем нашу борьбу с говнокодом.
В прошлый раз мы закончили на том, что отформатировали наш модуль, и говнокод стал более-менее симпатичным, но запашок все равно остался.
Сегодня мы поговорим о рефакторинге кода. Т.е. превращаем "говно"-код в "прекрасно/прэлестно/офигенно"-код.
В этом нам будет помогать новый скрипт "Анализ кода". Работа над ним началась только сегодня, поэтому возможностей в нем пока чуть менее, чем ничего.
Но работа ведется, идей море (лишь бы мозгов хватило реализовать), а значит все будет путем. Параллельно с этим скриптом буду дорабатывать и скрипт самого рефакторинга, для устранения тех проблем, которые будут выявляться Анализатором.
Прошу, пробуйте, пишите ваши отзывы и идеи.
На данный момент реализованы следующие проверки:
1. Размер метода (в строках, комментарии пока тоже считаются, потом сделаю настройку для их отключения)
2. Количество параметров в методе
3. Проверка строк кода длиннее определенного значения (по рекомендации 1С это 80 - 120 символов)
4. Проверка сложности условия (т.е. слишком много условных операторов И и ИЛИ)
5. Проверка "мертвых методов"
6. Проверка "мертвых меременных".
7. Проверка переменных на синтаксические ошибки (предложение от EvilBeaver)
8. Дублирование кода
Анализ дублирования запускается отдельной кнопкой т.к. процесс долгий и выполняется внешней обработкой, при этом конфигуратор блокируется.
На данный момент модуль из 5500 строк у меня проверяется 1 минуту. при минимальном блоке 4 строки и максимальном 20. Это можно регулировать в настройках.
Если кликнуть по колонке с количеством повторов в дереве, появится список с номерами строк, в которых встречается этот код.
Если выдается несколько одинаковых блоков со сдвигом в одну строку, увеличьте размер максимального блока в настройках.
Планируется:
1. Старый код (тот, который закомментировали, но так и не удалили)
2. ну и наверняка еще что-нибудь, совершенству кода нет предела.
P.S. Потестил на одной базе, оказывается столько там хлама было, жуть! Надеюсь и вам пригодится.
| Вложения: |
CodeAnalysis.zip [46.92 КБ]
Скачиваний: 449 |
|
Image 1.png [ 16.69 КБ | Просмотров: 14565 ] |
