Мой класс Запроса. Ваше мнение?

Похоже, вы написали очень красивую небольшую ORM (объектно-реляционное сопоставление) с реализацией шаблона репозитория (или близкого к нему). Вы также воспользовались шаблоном проектирования под названием Fluent Interfaces, который позволяет вам делать .Table .Fields каскадно друг от друга.

Все три этих шаблона очень хороши для включения в дизайн современных программных систем, как вы уже сделали. обратите внимание, они значительно уменьшают LOC, улучшают ремонтопригодность и значительно улучшают возможности тестирования. Мне также нравится, что вы создали свою функциональность, позволяющую правильно возвращать строго типизированные объекты с использованием универсальных шаблонов.

Изменить: Единственная возможная область, которую я мог видеть для улучшения, - это изменить тот факт, что каждый вызов является новым запросом, и, возможно, изменить свой код, чтобы использовать структуру внедрения зависимостей, которая будет вводить правильный поставщик, и когда объект вызывается, он будет инициировал транзакцию базы данных для запроса. Если ваш код уже в значительной степени делает это внутри класса Query, который можно назвать «Внедрение зависимостей бедняков» или, по крайней мере, он очень похож (в зависимости от вашей реализации), который, если он уже работает для вас, и вы не собираетесь действительно расти Если ваши типы баз данных не поддерживают DI для бедняков, все должно быть в порядке. Я просто больше не поклонник нового ключевого слова в любом использовании, поскольку оно обычно вызывает высокий уровень сцепления кода, который смягчает структура DI (или это ' s для метода сокрытия - плохое плохое дизайнерское решение).

В ответ на ответ Марка я обычно никогда не фанат генерации кода, это всегда кажется точкой отказа, однако у него есть очень Хороший момент для уменьшения количества задействованных магических нитей. Вместо этого я бы предпочел использовать лямбда-оператор и выражения, чтобы справиться с этим, так что вы можете добиться этого:

public static OrdersDataTable GetByOrderId( int id )
{
    return  
       new Query(connection)
       .Table(x => Inflector.Pluralize(x.GetType())
       .Fields(x=> { x.OrderID, x.CustomerID, x.Description, x.Amount })
       .Equals(x=>x.OrderID, id)
       .Execute<OrdersDataTable>();
}

Это устранит все проблемы рефакторинга волшебных строк, что позволит вам гораздо проще использовать встроенные инструменты рефакторинга Microsoft или Resharper (из Конечно, resharper может находить волшебные строки во время рефакторинга).

Inflector - это бесплатная библиотека (не могу вспомнить, OSS или нет), которая включает в себя функции для работы с текстом, метод Pluralize возьмет слово и сделает его ... множественное число, если вы не могли догадаться.