Разумно ли обращаться к БД из Automapper TypeConverter?

Question

Итак, я наткнулся на некоторый код в кодовой базе, над которой я работаю.

Мы используем Automapper для сопоставления объектов домена (Poco / dto) с моделями Entity Framework. Это все хорошо.

Однако в некоторых из картографов мы используем реализацию на ITypeConverter, которая создается с экземпляром нашего EF dbcontext. Который используется для поиска таких вещей, как идентификаторы для данной строки.

Это похоже на запах кода для меня (но, очевидно, мои коллеги думают, что это круто).

Это хорошая или плохая идея - обращаться к БД внутри Automapper TypeConverter?

Answer 1

Я обнаружил одну причину, почему это плохая идея.

Если одно отображение выполняет вызов в БД, то вам придется выполнить этот вызов в любом случае.

Однако, если мапперу необходимо отобразить коллекцию объектов, маппер выполнит N db вызовов. Который потенциально может быть выполнен как один вызов.