Насколько велика должна быть длина функции (строки кода в функции)?

Question

Можно создать дубликат:
How many lines of code should a function/procedure/method have?

Я хотел бы знать, сколько строк кода должна быть функция есть? Сколько строк слишком много.

Я прочитал это некоторое время назад, это около 10 или 20 строк, но это было потому, что экран вмещал бы столько строк. Теперь, когда размер экрана станет больше, это не подтвердится.

Предположим, что никакая часть функции не используется нигде, кроме принципа DRY.

Хотелось бы услышать, что другие могут сказать об этом.

спасибо.

Примечание: Дубликат When is a function too long?, не смог найти его, когда я разместил.

Answer 1

Этот вопрос хорошо ответил на Code Complete. Стив МакКоннел написал целую страницу, чтобы ответить на этот вопрос. Его вывод:

Декада доказательства говорит, что процедуры такой длина (> 100 строк) не более ошибок, чем более короткие рутин. Пусть такие вопросы, как сплоченность рутинной, номер директивных точек, количество комментариев, необходимых для объяснить рутину, и другие сложностей, связанные соображения диктуют длину обычного , а не введение ограничения длины самих по себе , Сказали, что если вы хотите написать процедуры больше, чем около 200 строк, будьте осторожны.

Answer 2

У него должно быть столько, сколько ему нужно.

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

Answer 3

Это довольно произвольное эмпирическое правило. Некоторые вроде 20 строк, другие - правило без прокрутки. В конце концов, просто убедитесь, что он читается и легко понятен с первого взгляда. Прочтите свой SOLID principles и убедитесь, что у метода есть только 1 ответственность и т. Д.

Answer 4

Не думаю, что важно, сколько строк у него есть ... до тех пор, пока оно эффективно.

Любой код, который может быть повторно использован в любой точке вашей кодовой базы, должен быть перемещен в другую функцию/метод в том же классе или совместно используемом классе и вызван.

Answer 5

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

Answer 6

Линии не имеют значения, но сложность есть.

Функция должна выполнять одну задачу, и она должна быть очевидной. Это не должно занять несколько секунд, чтобы точно понять, как и что делает функция.

Answer 7

Если необходимо, как можно меньше.

Я беру 5-10 строк как правило, но если есть какая-то логика, которая не может быть легко переделана в несколько функций, я пишу дольше, когда это необходимо. С другой стороны, у меня часто есть функции, которые являются только строкой или двумя длинными.

Если вы не немедленно понять, что делает часть кода, написать для него новую функцию.

Answer 8

Ядро Linux Стиль кодирования документ говорит:

Функции должны быть короткими и сладкими, и делать только одну вещь. Они должны соответствовать одному или двум экранам текста (размер экрана ISO/ANSI равен 80x24, так как мы все знаем), и сделайте одно и сделайте , что хорошо.

Теперь я понимаю, что это в контексте кода ядра, но я думаю, что некоторые из его точек, которые он делает, имеют: как правило, действительная длина. Найти копию here. Раздел о функциях - глава 4.

В целом, длина функции не должна ограничиваться каким-либо искусственным правилом; если это имеет смысл, и потому что это упрощает чтение, но правило о 1-2 экранах не написано на камне.

Answer 9

это просто мнение от оо-перспективы:

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

пример простой тривиальной функции вместо функции, которая вычисляет последовательность фибоначчи inline в цикле, я бы добавил функцию-преемник (int a, int b), которая вызывается функцией fibonacci().

более сложным примером в моде будет http-клиент, который выполняет запрос GET. я бы сломал это примерно так:

Connection getConnection(String host, int port) 
Request createRequest(String[] params) 
void sendRequest(Request r) 
String getResponse(Connection c,Request r) 

Answer 10

Функции должны быть достаточно малыми, чтобы выполнять свою работу, но не меньше.