Есть метод, например: getUsers. Он юзается в 10-20 местах в проекте, и мне вдруг понадобилось получать юзеров отсортированных не по айдишнику (как сейчас), а по колонке order. В голове есть два варианта:
1) Сделать у существующего метода getUsers необязательный параметр (withSorting = false), и в тех самых 10-20 местах оно будет работать как сейчас, потому что там по дефолту проставляется фолс, а в нужном нам месте поставить true. Но мне говорили, что если добавляем всякие такие флаги то это служит нарушением SRP.
2) Создать метод getUsersSorted, например. Но как по мне, это тупо копирование метода getUsers с добавлением ордер бай.
Хотел бы спросить у знающих, как и почему лучше сделать?
3 комментария
Добавить комментарий