170 похожих чатов

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

функция с кучей проверок на то что входящие данные валидны. Проходит какое-то время и нужно написать вторую функцию, которая делает на 90% то же самое что и первая но немного иначе. Проходит еще время - нужно расширить еще в другом месте. Как правильно сделать архитектуру для этого? Вот несколько вариантов которые мне кажутся не оч.
1. Добавить больше аргументов. Мне не нравится потому что много аргументов будут релевантны только в одном частном случае и бойлерплейтом в других.
2. Разбить на мелкие функции и вместо условного validate(args...) вызывать validateA(aArgs...) && vallidateB(bArgs...) && validateC(cArgs...). Минусом я вижу что если появится нужда для validateD(dArgs...) в будущем то нужно будет его добавлять во множестве разных мест.

может какой-то policy-based здесь подойдет?

8 ответов

8 просмотров

Вместо кучи проверок использовать больше осмысленных типов, в которых проверки будут зашиты на уровне конструктора

Может надо валидировать данные там где они получаются? А в функциях просто ассертить/падать.

Oleksandr-Senkovych Автор вопроса
Dmitry Sokolov
Может надо валидировать данные там где они получаю...

Это и есть функция валидации валидации, раньше неё ничего нет

Oleksandr Senkovych
Это и есть функция валидации валидации, раньше неё...

Непонятно всё. А может не надо писать вторую/третью/пятую функцию? Может сделать класс аргумента таким чтобы в него можно было вносить изменения? И его уже валидировать? Так то можно много чего нафантазировать... А может один параметр от другого зависит etc...

Oleksandr-Senkovych Автор вопроса
Dmitry Sokolov
Непонятно всё. А может не надо писать вторую/треть...

Ну мне кажется это ведь типичная ситуация с которой все сталкиваются: есть запросы которые надо провалидировать, логика сложна и может повторятся, копипастить не хочется, а хочется красивый и расширяемый подход. Класс аргумента можно сделать, не проблема. А что если результат одной проверки является входом другой, хранить его тоже в аргументе?

Oleksandr Senkovych
Ну мне кажется это ведь типичная ситуация с которо...

Ну запрос вообще может быть сложной структурой. И не факт что в одном месте можно весь его логически валидировать.

Oleksandr Senkovych
Ну мне кажется это ведь типичная ситуация с которо...

> результат одной проверки является входом другой Это как то странно выглядит. Обычно у validate возврат void или bool хотя бы

Похожие вопросы

Обсуждают сегодня

Всем привет! Имеется функция: function IsValidChar(ch: UTF8Char): Boolean; var i: Integer; ValidChars: AnsiString; begin ValidChars := 'abcdefghijklmnopqrstuvwxyzABCDE...
Евгений
44
И никого не интересует какие пакеты кто использует. ((% Заходишь на сайт симфони и видишь поддержку Украины - по законам РФ это ж экстремизм. Только никто не отказывается от с...
Am Ambrion
11
лучше скажите, причём тут паскаль?
Alexey Kulakov
36
Чтобы перехватить все нажимания буков на форме, надо хук ставить? Пробовал на форме ОнКейДаун, оно ловит клаву если фокус не на компоненте с вводом текста
Serjone
15
Народ! Впервые клиенту пришло письмо от РКН, у вас, дескать, есть яндекс метрика, а нигде не написано, что вы ее юзаете. Никто не сталкивался?
Sasha Beep
14
Всем привет! вывожу на общей стр дочерние ресурсыв каждом ресурсе галерея, и первая фотка должна выводиться на общей [!DocLister? &prepare=photo !]
Alekso
12
А можно вопрос? Мне сегодня сказали что у меня функция (которая просто заполняет массив значениями) не правильная void Full(double * arr, int n) { for (int i = 0; i < n; i...
† C E †
7
День добрый, подскажите пожалуйста, есть ли какой-то способ сказать ребару не компилировать определённое приложение? Всю доку их перечиатл ничего подобного не нашёл
Кирилл
14
Добрый вечер. Хочу чтобы у меня в классе поле было функцией, которая возвращает строку. Делаю так: interface ... TGetOutPath = function : String of object; ... protec...
Kirill Filippenok
12
Всем привет. Возможно ли выполнить код ahk без интерпритатора? Имея только скомпилированный ехе файл? Например как функция eval
proxy
6
Карта сайта