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

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

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

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

Функция должна выполнять одну задачу, и это должно быть очевидно. Вам не понадобится больше нескольких минут, чтобы понять, как и что делает эта функция.

Его должно быть столько, сколько нужно.

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

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

Как можно дольше и как можно короче.

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

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

Я не думаю, что имеет значение, сколько в нем строк ... если это эффективно.

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

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

В документе о стиле кодирования ядра Linux говорится:

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

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

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

это просто мнение с точки зрения oo:

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

очень тривиальный функциональный пример: вместо функции, которая вычисляет последовательность Фибоначчи, встроенную в цикл, я бы добавил функцию-преемник (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)

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