Здравствуйте, SkyDance, Вы писали:

S>>Вот пример кода другого разработчика который я недавно переформатировал. Какой из вариантов готов к дальнейшему расширению?

SD>Первоначальный вариант лучше: объем контекста одинаков, функции исполняются строго последовательно, читать одну функцию проще, чем скакать по нескольким.

вы может не поняли код. там 4 вывод таблиц c совершенно разной итерацией в одной функции.
общего у них реализация некоторых начальных параметров и состав колонок.
по изначальному коду это просто не видно.

SD>К тому же ни одна из выделенных функций сама по себе не имеет применения. То есть выделение функции просто ради выделения. Зачем?
Для дальнейшего расширения функциональности, которой у разных функций будет разное.

SD>Выделять надо там, где или становится доступным повторное использование, или можно уменьшить размер контекста (путем передачи, скажем, 3 параметров вместо 12). Лучше и то, и другое сразу. Но если в конечном итоге все равно нужно разбираться, то проще разобраться в одной, чем в 5 функциях.


https://www.rsdn.org/article/patterns/rtp4.xml

Long Method