AP
Юнит тесты пишешь?)
50/50
Size: a a a
2019 December 20
АК
Вот так мне и сказали) Поэтому решил уточнить, есть ли какие-то правила/рекомендации
ЕР
50/50
А на методы с 5 параметрами?)
ЕР
На самом деле необходимо разобраться почему нужно 5 параметров
ЕР
Если метод делает слишком много, заворачиванием из в dto/массив проблему не исправить особо
YV
Ну запихнёт он все в массив 1 параметра. Дальше что и как это поможет?
AP
А на методы с 5 параметрами?)
Есть и такое
ЕР
Есть и такое
Короче, скорее всего твой метод делает слишком много
ЕР
Но не факт
AD
Если метод делает слишком много, заворачиванием из в dto/массив проблему не исправить особо
ноуп, допустим у нас есть фильтрация, я могу сделать красивый объект Filter, а могу каждый параметр фильтра передавать как аргумент
ЕР
ноуп, допустим у нас есть фильтрация, я могу сделать красивый объект Filter, а могу каждый параметр фильтра передавать как аргумент
Приведи пример,
ЕР
Не понято что за Filter
BP
хорошо, если не 5 то какой по вашему должен быть максимум входных параметров? ))))
IT
смотря что должен делать класс 🙂 но после 5 уже нужно реально задумываться)
AD
Приведи пример,
фильтр по дате, пользователю, статусу, активности, дате регистрации и тд
можно запихнуть все эти параметры в объект, и передавать его (что круто и можно сделать красивый и реюзабельный сервис для подстановки условий в запрос), а можно топорно передать все параметры как аргументы
можно запихнуть все эти параметры в объект, и передавать его (что круто и можно сделать красивый и реюзабельный сервис для подстановки условий в запрос), а можно топорно передать все параметры как аргументы
AV
фильтр по дате, пользователю, статусу, активности, дате регистрации и тд
можно запихнуть все эти параметры в объект, и передавать его (что круто и можно сделать красивый и реюзабельный сервис для подстановки условий в запрос), а можно топорно передать все параметры как аргументы
можно запихнуть все эти параметры в объект, и передавать его (что круто и можно сделать красивый и реюзабельный сервис для подстановки условий в запрос), а можно топорно передать все параметры как аргументы
+
ЕР
фильтр по дате, пользователю, статусу, активности, дате регистрации и тд
можно запихнуть все эти параметры в объект, и передавать его (что круто и можно сделать красивый и реюзабельный сервис для подстановки условий в запрос), а можно топорно передать все параметры как аргументы
можно запихнуть все эти параметры в объект, и передавать его (что круто и можно сделать красивый и реюзабельный сервис для подстановки условий в запрос), а можно топорно передать все параметры как аргументы
Ну можно методами сеттить, query builder'ы те же
ЕР
Но тут я бы пихал в dto сколько нужно параметров и не парился бв
P
Если будет много необязательных параметров фильтрации, то вызывать такое может быть неудобно, когда пишешь на PHP, а не на Python.
ЕР
Если будет много необязательных параметров фильтрации, то вызывать такое может быть неудобно, когда пишешь на PHP, а не на Python.
Удобно тогда уж в c# )