Цитата:
Сообщение от
petr
Не понимаю наездов на исходный код, он на 100% правильно написан.
Если правильно написан означает компилируется и проходит тесты, то да. Этого здесь никто и не отрицал.
Критику не выдерживает сам подход использования статичестких select'ов против динамического построения Query.
Представьте себе, что помимо добавления условия по _inventTransID необходимо станет управлять ну скажем фильтром по InventDimId. Тогда один if разрастётся на 4 ветки:
учитываем inventTransID и не учитываем InventDimId
не учитываем inventTransID и учитываем InventDimId
учитываем inventTransID и учитываем InventDimId
не учитываем inventTransID и не учитываем InventDimId
А что делать если таких условий ещё больше? И вы действительно считаете что такой код сопровождать легче?