Сообщение Re[3]: Ширина кода - газетная vs книжная от 17.01.2025 6:29
Изменено 17.01.2025 7:27 swame
Re[3]: Ширина кода - газетная vs книжная
Здравствуйте, SkyDance, Вы писали:
S>>Вот пример кода другого разработчика который я недавно переформатировал. Какой из вариантов готов к дальнейшему расширению?
SD>Первоначальный вариант лучше: объем контекста одинаков, функции исполняются строго последовательно, читать одну функцию проще, чем скакать по нескольким.
вы может не поняли код. там 4 вывод совершенно разных таблиц в одной функции. общего у них только реализация некоторых начальных параметров.
по изначальному коду это просто не видно.
SD>К тому же ни одна из выделенных функций сама по себе не имеет применения. То есть выделение функции просто ради выделения. Зачем?
Для дальнейшего расширения функциональности, которой у разных функций будет разное.
SD>Выделять надо там, где или становится доступным повторное использование, или можно уменьшить размер контекста (путем передачи, скажем, 3 параметров вместо 12). Лучше и то, и другое сразу. Но если в конечном итоге все равно нужно разбираться, то проще разобраться в одной, чем в 5 функциях.
https://www.rsdn.org/article/patterns/rtp4.xml
Long Method
S>>Вот пример кода другого разработчика который я недавно переформатировал. Какой из вариантов готов к дальнейшему расширению?
SD>Первоначальный вариант лучше: объем контекста одинаков, функции исполняются строго последовательно, читать одну функцию проще, чем скакать по нескольким.
вы может не поняли код. там 4 вывод совершенно разных таблиц в одной функции. общего у них только реализация некоторых начальных параметров.
по изначальному коду это просто не видно.
SD>К тому же ни одна из выделенных функций сама по себе не имеет применения. То есть выделение функции просто ради выделения. Зачем?
Для дальнейшего расширения функциональности, которой у разных функций будет разное.
SD>Выделять надо там, где или становится доступным повторное использование, или можно уменьшить размер контекста (путем передачи, скажем, 3 параметров вместо 12). Лучше и то, и другое сразу. Но если в конечном итоге все равно нужно разбираться, то проще разобраться в одной, чем в 5 функциях.
https://www.rsdn.org/article/patterns/rtp4.xml
Long Method
Re[3]: Ширина кода - газетная vs книжная
Здравствуйте, SkyDance, Вы писали:
S>>Вот пример кода другого разработчика который я недавно переформатировал. Какой из вариантов готов к дальнейшему расширению?
SD>Первоначальный вариант лучше: объем контекста одинаков, функции исполняются строго последовательно, читать одну функцию проще, чем скакать по нескольким.
вы может не поняли код. там 4 вывод таблиц c совершенно разной итерацией в одной функции.
общего у них реализация некоторых начальных параметров и состав колонок.
по изначальному коду это просто не видно.
SD>К тому же ни одна из выделенных функций сама по себе не имеет применения. То есть выделение функции просто ради выделения. Зачем?
Для дальнейшего расширения функциональности, которой у разных функций будет разное.
SD>Выделять надо там, где или становится доступным повторное использование, или можно уменьшить размер контекста (путем передачи, скажем, 3 параметров вместо 12). Лучше и то, и другое сразу. Но если в конечном итоге все равно нужно разбираться, то проще разобраться в одной, чем в 5 функциях.
https://www.rsdn.org/article/patterns/rtp4.xml
Long Method
S>>Вот пример кода другого разработчика который я недавно переформатировал. Какой из вариантов готов к дальнейшему расширению?
SD>Первоначальный вариант лучше: объем контекста одинаков, функции исполняются строго последовательно, читать одну функцию проще, чем скакать по нескольким.
вы может не поняли код. там 4 вывод таблиц c совершенно разной итерацией в одной функции.
общего у них реализация некоторых начальных параметров и состав колонок.
по изначальному коду это просто не видно.
SD>К тому же ни одна из выделенных функций сама по себе не имеет применения. То есть выделение функции просто ради выделения. Зачем?
Для дальнейшего расширения функциональности, которой у разных функций будет разное.
SD>Выделять надо там, где или становится доступным повторное использование, или можно уменьшить размер контекста (путем передачи, скажем, 3 параметров вместо 12). Лучше и то, и другое сразу. Но если в конечном итоге все равно нужно разбираться, то проще разобраться в одной, чем в 5 функциях.
https://www.rsdn.org/article/patterns/rtp4.xml
Long Method