ЕР
С именованными аргументами
Size: a a a
2019 December 20
P
Ага я их и имел в виду.
P
Массив параметров - это отложенный выстрел себе в ноги. Остаётся DTO.
AD
Остается глянуть что он хочет сделать) мош чел через ди в конструктор 5 сервисов передает (лог, сентри, газл какой нить и еще пару приблуд)
АК
да не. Просто формирование массива с идентичной структурой вызывается 4 раза в рамках одной функции. В этом массиве 5 параметров. Собственно с этого и началось, нужно ли делать отдельную функцию для формирования этого массива и вызвать ее в 4х местах.
ЕР
да не. Просто формирование массива с идентичной структурой вызывается 4 раза в рамках одной функции. В этом массиве 5 параметров. Собственно с этого и началось, нужно ли делать отдельную функцию для формирования этого массива и вызвать ее в 4х местах.
Без кода это по прежнему ни о чём)
Но звучит, конечно, не очень
Но звучит, конечно, не очень
ЕР
Но если оставить за кадром вопрос содержания функции, код инициализации массива с одинаковой структурой не стоит дублировать, если конечно в будущем не планируется изменять струткру конкретных массивов независимо
АК
Без кода это по прежнему ни о чём)
Но звучит, конечно, не очень
Но звучит, конечно, не очень
не очень звучит - добавление функции или не добавлении?)
АК
Да, и структура массива может измениться раз так в несколько лет
ЕР
не очень звучит - добавление функции или не добавлении?)
Не очень звучит содержание функции, почему там 5 параметров и почему там 4 раза формируется массив)
ЕР
Да, и структура массива может измениться раз так в несколько лет
Хороший архитектор максимизирует количество непринятых решений - оставь как есть, отрефакторишь через несколько лет)
СА
Просто интересно.. можно ли сказать человеку, что если у функции 5 входных параметров - это плохо
У нас вышла переводная статья примерно на эту тему: https://habr.com/ru/company/funcorp/blog/478760/ (заголовок не об этом, но по тексту затрагивается и разбираются способы улучшения)
АК
У нас вышла переводная статья примерно на эту тему: https://habr.com/ru/company/funcorp/blog/478760/ (заголовок не об этом, но по тексту затрагивается и разбираются способы улучшения)
Спасибо
ВС
Hello
L
How are you?
M
How are you?
Говорящий кот 😳
A
имя прикольное: лёгкий Лёха
A
*мягкий
M
Alex
*мягкий
Говорящая мышь 😳
M
Простите коллеги