Как рефакторить файлы в пару тысяч строк?

Question

[Дмитрий Ларин]

Это был простой сайт на 3 странички. Сейчас это сложный сайт на 3 странички. Бек за три года из пары php файлов превратился в сотню сложных файлов. Интеграция с срм, смс, телефонией, месседжерами и чертом лысым. Добавились агентские и партнерские кабинеты. Пачка вебхуков, виджеты...

В общем на беке есть много файлов со сложной логикой. некоторые из себя представляют оператор switch на 500 строк. C какой стороны подходить к рефрактору таких файлов?

Comments

[Антон]

К сносу, видимо, и модульной переделке под ООП.
Думаете, почему до сих пор существуют всякие битриксы в том виде, в котором они есть?
Их никто не рефакторит, а просто сверху навешивают с похмелья то, что взбрело в голову.

[Александр Новиков]

Ну если совсем нет идей - то надо начать с понимания основ процесса рефакторинга (Мартин Фаулер - Рефакторинг в помощь). А конкретных советов вы тут не получите - не видя исходного кода любой совет будет гаданием на кофейной гуще и абстракцией.

Answer 1

[Nikita Dergachov]

1. Открываешь файл.
2. Читаешь код.
3. Пытаешься понять.
4. Пишешь тесты
5. Рефакторишь.

Comments

[Дмитрий Ларин]

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

Answer 2

[Saboteur]

Вот и пришло время понять, для чего на самом деле нужен ООП.
К рефакторингу подходить именно через ООП - потихоньку переводить весь функционал в классы, стараясь продумывать архитектуру таким образом, чтобы один класс со всеми его методами комфортно помещался в голове программиста.

Answer 3

[Алексей Скобкин]

Декомпозиция.

Answer 4

[Lander]

Стоит начать с изучения методов рефакторинга.