Мне была поручена реорганизация подразделения Delphi. Вау. 10000 строк кода, без документации, тонны кода копирования и вставки.
Есть много методов, сделанных с копией и вставкой, которые могут быть реорганизованы, так или иначе, я теряюсь во всех этих строках, у меня есть раздел интерфейса, где я могу "найти свой путь", но в целом то, что предлагает yuo для решения этой проблемы вид задачи?
Спасибо.
Я также рекомендую закладок http://www.refactoring.com/catalog/ и http://www.industriallogic.com/xp/refactoring/catalog.html.
Также не помешало бы получить копию "чистого кода": "Руководство по гибкому программному мастерству" Роберта "Дядя Боб" Мартин и др. Легко распознать плохой код. Это гораздо сложнее узнать, когда вы пишете хороший код.
Предупреждение: сосредоточьтесь на реорганизации кода, над которым вам нужно работать. Его легко начать вниз по кроличьей дыре и завершить месячный рефакторинг кода, который не сразу был связан с этой задачей.
И избавь себя от некоторых проблем. Не пытайтесь "исправить" код и реорганизовать его одновременно. Рефакторинг сначала, затем исправить ошибки или добавить эту новую функцию. Помните, что рефакторинг изменяется без изменения внешнего поведения.
Сопротивляйтесь стремлению полностью переписать. Я усердно изучил, что дерьмовый код, отвечающий требованиям пользователя, предпочтительнее очищать код, который этого не делает. Crappy-код всегда может быть постепенно улучшен до тех пор, пока он не станет чем-то гордиться.
Я думаю, что самое лучшее, что вы можете сделать, это написать DUnit Tests для интерфейса. Это заставляет вас понимать существующий код, помогает во время отладки и гарантирует, что интерфейс будет действовать после рефакторинга.
Лучшие 12 причин для написания модульных тестов применимы в вашем случае:
Я столкнулся с подобными ситуациями. Мои соболезнования вам!
На мой взгляд, самое главное, что вы действительно понимаете весь код, как сегодня. Ум лучше, чем мой, может просто читать код и понимать его. Однако я не могу.
После прочтения кода для общего обзора, я обычно неоднократно выполнял его один раз в отладчике, пока не начну видеть некоторые шаблоны работы и не узнаю ранее прочитанный код. Возможно, это очевидно, но подумал, что я упоминаю об этом.
Вы также можете подумать о создании хорошего тестового набора, который работает с текущим кодом.
Разделяет ли раздел интерфейса кучу определений классов? Если это так, создайте новый блок для каждого класса и переместите каждый класс в его собственный блок.
Если вы используете Delphi 2007 или выше, вы можете использовать опцию" refactor/Move ", чтобы переместить эти классы в новое (пространство имен).
Следующий шаг - разделение больших классов на более мелкие классы. Это просто много ручной работы.
После того как ваш код делится на несколько единиц, вы можете исследовать каждый блок, обнаруживать идентичный код и генерировать базовые классы, которые будут использоваться как родительские для двух классов, которые имеют сходную функциональность.
Помимо понимания кода и т.д., эти инструменты могут помочь в реорганизации и реорганизации проекта:
Model Maker - мощный инструмент проектирования, реверсирования и рефакторинга: http://www.modelmakertools.com/modelmaker/index.html
Model Maker Code Explorer - мощный плагин для Delphi IDE, который поможет в рефакторинге, навигации по коду и т.д.: http://www.modelmakertools.com/code-explorer/index.html
Я бы использовал какой-то инструмент UML для создания диаграмм som-классов и других диаграмм, чтобы получить обзор системы, и начать разделение и комментирование, как сказал @Workshop Alex.
Используйте инструмент, такой как Doxygen, чтобы помочь вам сопоставить код.
Справка об этом здесь
Начните с малого и в конечном итоге выполните частичную или полную переписывание. Начните создавать базовые классы для выполнения частей головоломки, не меняя выход. Прополощите-повторите, пока у вас нет новой поддерживаемой кодовой базы. Когда вы нажмете эти процедуры copy-n-paste, у вас будут базовые классы для выполнения этой работы, и это действительно поможет ускорить выполнение задачи.