Чистка Delphi-кода?

Question

[S_talker]

Есть большой (несколько сот тысяч строк) проект на Delphi, который разрабатывается около 10 лет.

Полагаю, работал над ним не один десяток разработчиков. Следовательно, там можно увидеть всевозможные проявления человеческой глупости, халатности, криворукости и прочих неприятных вещей.

Хотелось бы немножко этот проект причесать, и максимально этот процесс автоматизировать (потому как вручную это может занять не одну неделю). А именно:

— удалить закомментированные блоки кода

— удалить код, который не линкуется в исполняемый файл

— удалить неиспользуемые uses

— удалить pas/dfm файлы, которые не относятся к проекту

— чтобы все предыдущие действия были полностью контролируемые


Ну, желательны еще разные плюшки, типа форматирования кода по шаблону, выявления потенциально опасных или просто подозрительных мест, и т.д. Но чистка кода (вышеназванные пункты) является приоритетной.

Answer 1

[Monca]

Автоматически не получится.
Uses можно не удалять, начиная с delphi 6 не нужные не подключаются.
Pas/dfm я удалял вручную, все что не были в файле проекта — потом возвращал, необходимые.
Форматировать к одному виду не советую — потом очень сложно сравнивать изменения.

Comments

[1001 101]

Не знаю, как насчёт D6, а вот в DXE2 если вручную удалить неиспользуемые Uses, то конечный размер скомпилированного приложения очень даже меняется.

Answer 2

[naum]

Лучше ручками, пишется быстро, регулировать можно оптимально под себя. Лучше использовать готовые движки / компоненты для парсинга delphi-кода (их множество, многие доступны без заморочек) и удобного контроля в последующем. Посмотрите на torry.net.

Хотя есть ряд нетривиальных задач (допустим, оценка покрытия — coverage), которые лучше решать готовыми инструментами (допустим, триалом AQTime), ради экономии времени. Хотя и это можно решить самому.

Зато спустя пару дней получится решение, которые, имхо, не стыдно выложить на паблик и помочь многим в дальнейшем :)

Comments

[Nashev]

Похоже, не получилось… А не подскажете какие-либо проверенные названия для этой задачи?

Answer 3

[Алексей Сидоров]

А потому, что нужно было юзать VCS. И вообще… Нынешнее состояние Delphi напоминает «судороги умирающего носорога»(с)

Comments

[retran]

10 лет назад его не было и альтернативами были убогий VB6 и тормозная на тот момент Java.

Answer 4

[Павел Некрасов]

Рефакторинг большого проекта может затянутся на долгие месяцы, оно вам надо? Может рефакторить только потенциально глючные, тормозные участки, с целью видимой оптимизации работы?

Comments

[S_talker]

Речь идет не о рефакторинге, а о чистке всего ненужного в исходном коде. Теоретически этот процесс легко автоматизируется. Только вот что-то нужной тулзы пока не нашел. Максимум — это чистка uses (и то, проект такого размера ей не удалось проглотить, сказала что не хватает дескрипшнов).

Answer 5

[Majnik]

Для форматирования посоветую GExperts (Experimental).
Для всего остального CnPack Wizards должен подойти.

Comments

[S_talker]

И GExperts и CnPack стоят. В CnPack нашел только Uses Cleaner (после недолгого шуршания дает ошибку о недостатке оконных хэндлов, после чего IDE становится неработоспособна до следующего перезапуска) и Comments Cropper (работает по принципу «Все или ничего», не позволяя контролировать процесс).
Короче говоря, ничего не подошло.

Answer 6

[SegaZero]

для форматирования лучше DelForEx пока нет
чистка uses насколько я помню возможна в CnPack
а вот удаление закомментированного кода легко сделать любым редактором, поддерживающим регулярки. Например заюзать поиск в GExperts по \(\*.*\*\) и когда появятся результаты сделать замену на пустую строку (там есть пункт replace all items)
неиспользуемый код лучше всего удалять специализированным софтом. AQTime тут выше уже давали, могу посоветовать еще Pascal Analyzer

а вообще, рефакторинг — это занятие долгое, нудное и сложноавтоматизируемое

Answer 7

[1001 101]

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