В своих приложениях я использую подход разделение функционала на слои, слои я стараюсь группировать на основании функциональности
- Data Access Object
- Data Access Layer
- Business Logic Layer
- GUI
Для работы с бд я использую entity-framework. Довольно часто я стал ловить себя на мысли что я не понимаю/не уверен на каком из слоев должна быть реализована та или иная функциональность.
Например: необходимо получить список объектов из бд, пусть класс будет называться Request
, тогда я делаю так:
В DAL у меня есть generic
репозиторий() c реализацией CRUD
и торчащим наружу свойством protected virtual IQueryable<T> Entities{...}
В BLL я обращаюсь к свойству Entities
вытягиваю из бд необходимые мне данные, материализую(ToList()
,Single()
), преобразую в DTO объекты которые отдаю в GUI.
В целом меня такой подход устраивал всем, базовые методы(CRUD
) реализованы в одном классе реализующим generic
репозиторий, методы для работы с конкретными сущностями реализованы в слое BLL
.
Но тут наступил переломный момент: мне понадобилось использовать хранимые процедуры/функции.
Для поддержания чистоты приложения доступ к хранимкам хочу(мне так кажется правильней) сделать в слое DAL я это представляю как то так :
Сделать IRequestRepository
в который в качестве зависимости передать generic
репозиторий плюс реализовать необходимые методы/функции, например такие как обращение и получения результата хранимой функции, так же мне будет необходимо выставить наружу в generic
репозитории DbContext
что бы можно было обращаться к хранимкам(_context.Database.SqlQuery<T>(...)
), дальше же все остается так как и было.
как альтернатива отказаться от generic
репозитория и для каждого необходимого класса из DAO реализовать класс репозитория, но в этом случае в каждом классе будет дублирование CRUD методов, что мне кажется не правильным.
Пример реализации:
public interface IRepository<T> where T: class
{
T GetById(object id);
void Insert(T entity);
void Insert(IEnumerable<T> entities);
void Update(T entity);
void Update(IEnumerable<T> entities);
void Delete(T entity);
void Delete(IEnumerable<T> entities);
IQueryable<T> Table { get; }
DbContext Context { get; }
}
public interface IRequestRepository
{
Request GetById(object id);
void Insert(Request request);
void Insert(IEnumerable<Request> requests);
void Update(Request request);
void Update(IEnumerable<Request> requests);
void Delete(Request request);
void Delete(IEnumerable<Request> requests);
}
public class RequestRepository: IRequestRepository
{
private readonly IRepository<Request> _requestRepository;
public RequestRepository(IRepository<Request> requestRepository)
{
this._requestRepository = requestRepository;
}
#region Методы обертки над методами из IRepository
Request GetById(object id) {...}
void Insert(Request request) {...}
void Insert(IEnumerable<Request> requests) {...}
void Update(Request request) {...}
void Update(IEnumerable<Request> requests) {...}
void Delete(Request request) {...}
void Delete(IEnumerable<Request> requests) {...}
#endregion
#region Методы обертки над хранимыми функциями/процедурами
IEnumerable<Request> GetListOfActiveRequests()
{
var _requestList = _requestRepository.Context.Database
.SqlQuery<Request>("Select * from dbo.GetListOfActiveRequests()");
//...
}
#endregion
//Прочие методы
}
Как все таки лучше/правильней поступить в описанном мной случае?
UPD:
исходя из ответа @PavelMayorov, если я правильно понял структура классов у меня вырисовывается следующим образом
public interface IRequestRepository
{
#region CRUD методы
Request GetById(object id);
void Insert(Request request);
void Insert(IEnumerable<Request> requests);
void Update(Request request);
void Update(IEnumerable<Request> requests);
void Delete(Request request);
void Delete(IEnumerable<Request> requests);
#endregion
}
public interface IRequestHistoryRepository
{
#region CRUD методы
#endregion
}
public class RequestRepository: IRequestRepository
{
private readonly DbContext _context;
public RequestRepository(DbContext context)
{
this._context=context;
}
// Реализация необходимых методов
// для работы с "сущностью"
}
public class RequestHistoryRepository: IRequestHistoryRepository
{
private readonly DbContext _context;
public RequestHistoryRepository(DbContext context)
{
this._context=context;
}
// Реализация необходимых методов
// для работы с "сущностью"
}
вот только в этом случае будет дублирование в каждом таком репозитории кода CRUD
try
{
if (request == null)
throw new ArgumentNullException("entity");
//Создание/Чтение/Редактирование/Удаление
}
catch (DbEntityValidationException dbEx)
{
var msg = string.Empty;
foreach (var validationErrors in dbEx.EntityValidationErrors)
foreach (var validationError in validationErrors.ValidationErrors)
msg += string.Format("Property: {0} Error: {1}", validationError.PropertyName, validationError.ErrorMessage) + Environment.NewLine;
var fail = new Exception(msg, dbEx);
//Debug.WriteLine(fail.Message, fail);
throw fail;
}