Является ли хорошим тоном постоянно использовать много методов/функций?

Question

[Urukhayy]

Представим задачу: нужно сделать функцию, которая должна высчитывать некоторые геометрические характеристики исходя из каких-то данных. Причем этих характеристик куча.
Так вот, для нахождения каждой характеристики лучше ведь своя функция?

И есть ли какие-либо рекомендации по объёму строк в функциях?

Answer 1

[Денис Инешин]

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

Answer 2

[AlikDex]

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

Answer 3

[asdz]

Я думаю можно.

Answer 4

[Виталий Инчин ☢]

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

Answer 5

[mamkaololosha]

Вынесите всё в отдельный namespace или класс.

Answer 6

[Oleg]

> И есть ли какие-либо рекомендации по объёму строк в функциях?

Зависит от языка.

Простые правила, которые вычитал в одной из книг, и стараюсь использовать:
1. 1 дело = 1 функция.
Т.е. в вашем случае - не должно быть функции которая сразу же считает и площадь поверхности и объём объекта

2. если код используется больше чем в 1 месте - выносим его в отдельную функцию

3. нормальная длинна функции - не больше 1 экрана.
Если больше - стоит задуматься о том, чтобы её разделить.

4. В идеале - функция не должна заниматься несколькими делами:
считать, лезть в базу, заниматья выводом.
для каждой из задач должна быть 1 функция.

Comments

[Mrrl]

Т.е. в вашем случае - не должно быть функции которая сразу же считает и площадь поверхности и объём объекта
Бывает так, что для площади и объёма нужны одни и те же довольно долгие операции (прочитать огромную модель с диска / вычислить координаты точки по сложным формулам / посчитать возможные пересечения частей модели...) В этом случае вычисление площади и объёма отдельными функциями потребует вдвое больше времени, чем одновременное. И чем тогда лучше жертвовать - принципами или временем пользователя?

[Oleg]

Mrrl: не понял, почему в вашем примере нужно чем то жертвовать :)

можно сделать так:
1. функция которая читает модель
2. функция которая считаетывает координаты точек по сложным формулам
3. считаем пересечения

3. функция в которую передаёте п1 и п2 и оно считает только площадь, на основе данных
3. функция в которую передаёте п3 и п4 и оно считает только объём на основе данных

+ если нужно, можно написать функцию "управленец", в которую передаёшь все нужные параметры (в том числе - откуда читать из файла, что считать, по какому алгоритм и тп)

При этом, пункты 1-2-3 можно (даже нужно) инкапсулировать в отдельный объект, и чтобы данные считались по нужде, сохранялись в локальном кеше.

Это если говорить про то, как "правильно писать" (хотя опять же - дело вкуса) .
Иногда правда нужно писать не правильно, а быстро - тогда совсем другой вопрос :)

[Mrrl]

Функция, которая читает модель? Ну да, есть такая функция. Даже целый класс - для разных способов навигации по модели. Вот только модель целиком в память не лезет, поэтому для каждого прохода всё равно приходится читать и интерпретировать файлы с данными.
Сохранять вычисленные координаты точек? Многовато их будет. Если писать на диск - то читать потом будет не дешевле, чем снова вычислять. А вычислять два раза, один для объёма, другой для площади - трата времени.
"Писать быстро" - это чтобы быстро появилась программа, или чтобы она быстро работала?

[Oleg]

> "Писать быстро" - это чтобы быстро появилась программа, или чтобы она быстро работала?
А Это как раз зависит от того что нужно :)

Т.е. можно:
1. писать быстро
2, чтобы работало быстро
3. чтобы архитектура была "правильная".

Всё вместе - не всегда возможно.

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

Answer 7

[Mrrl]

Иногда на каждую характеристику лучше своя функция.
Иногда, если набор характеристик в каком-то смысле однородный (проекции модели на три координатные плоскости, число красных/зелёных/синих рёбер...) можно какой-то признак характеристики передать параметром - тогда одна функция будет вычислять любую из нескольких характеристик.
Иногда, если несколько характеристик удобно вычислять вместе, и они логически связаны между собой (и с большой вероятностью потребуются вместе) - объединить их в структуру и заставить функцию заполнять её всю.
Иногда - когда для вычисления характеристик уже не надо лезть в потроха объекта, а можно воспользоваться другими характеристиками, и, возможно, какими-нибудь итераторами, а кроме того, вычисление само по себе трудоёмко и требует сложных алгоритмов - вынести это вычисление в отдельный класс.