Сколько параметров функции слишком много?

Я не знаю, но я знаю это, когда вижу.

По словам Стива МакКоннелла в Code Complete, вам следует

Ограничьте количество параметров процедуры примерно семью

Если вам нужно спросить, это, вероятно, слишком много.

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

Не то чтобы я сам всегда этому следил;)

Роберт С. Мартин (дядя Боб) рекомендует максимум 3 в Чистый код: руководство по созданию гибкого программного обеспечения

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

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

Быстрый ответ: когда вам нужно остановиться и задать этот вопрос, у вас их слишком много.

Лично мне нравится, когда число меньше шести. Если требуется больше, решение зависит от проблемы. Один из подходов - использовать функции «установщика» для передачи значений объекту, который в конечном итоге выполнит желаемую функцию. Другой вариант - использовать структуру, как вы упомянули. В любом случае, вы не ошибетесь.

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

Учитывая широкое распространение Intellisense (или эквивалента в других IDE) и всплывающих подсказок с комментариями из XML-документации в Visual Studio, я не думаю, что есть твердый ответ на этот вопрос.

Слишком много параметра - это "Кодовый запах".

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

Ставить число для «Слишком много» - это очень субъективно и зависит от вашей организации и языка, который вы используете. Практическое правило состоит в том, что если вы не можете прочитать подпись своего метода и иметь представление о том, что это такое делать, чем у вас может быть слишком много информации. Лично я стараюсь не превышать 5 параметров.

Для меня это 5.

За этим сложно управлять (запомнить имя, порядок и т. Д.). Плюс, если я зашел так далеко, у меня есть версии со значениями по умолчанию, которые называют это.

Также зависит от функции, если ваша функция требует серьезного вмешательства пользователя или переменных, я бы не стал выходить за пределы диапазона 7-8. Что касается среднего количества параметров, то, на мой взгляд, наилучшим вариантом является 5-6. Если вы используете больше, чем это, вы можете рассматривать объекты класса как параметры или другие более мелкие функции.

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

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

В настоящее время вы можете передать ссылку на объект, который инкапсулирует какое-то сложное состояние (и поведение). Использование 7 из них определенно было бы слишком много.

Моя личная цель - избегать использования более четырех.

Это сильно зависит от типов аргументов. Если все они целые, то 2 может быть слишком много. (как мне запомнить, в каком порядке?) Если какой-либо аргумент принимает значение null, то число резко падает.

Настоящий ответ приходит, если спросить себя:

• насколько легко понять вызовы, когда я читаю код?
• насколько легко запомнить правильные аргументы и порядок аргументов при написании кода?

И это зависит от языка программирования .. В C действительно не редко можно встретить функции с 7 параметрами. Однако в C # я редко встречал больше 5 параметров, а лично я обычно использую меньше 3.

// In C 
 draw_dot(x, y, size, red, green, blue, alpha)

// In C# 
 Point point(x,y);
 Color color(red,green,blue,alpha);

 Tool.DrawDot(point, color);

Я бы сказал максимум 4. Я думаю, что все, что указано выше, должно быть помещено в класс.