No meu projeto uso o EF6
, uso repositórios genéricos e na minha camada BLL
eu tenho a classe ClienteBLL
, que é uma classe que tem os métodos de implementação logica da classe.
Em um dos métodos eu retorno uma lista que recebe todos os clientes do banco e depois faça uma ordenação
Método:
public IList<ClienteDTO> TodosClientes()
{
List<ClienteDTO> lista = clienteRepo.BuscarTodos().ToList();
lista.Sort(delegate (ClienteDTO a, ClienteDTO b) { return a.nome.CompareTo(b.nome); });
return lista;
}
Minha duvida: É errado ordenar a lista dessa forma? Se sim, o que recomenda?
Achei uma maneira bem melhor: How to Sort a List by a property in the object
Então com base na resposta do link eu fiz assim:
public IList<ClienteDTO> TodosClientes()
{
return clienteRepo.BuscarTodos().OrderBy(o => o.nome).ToList();
}
Agora não tenho duvidas que está certo!
Meu repositório genérico:
public abstract class GenericRepository<T> : IDisposable, IGenericRepository<T> where T : class
{
GdPContext contexto = new GdPContext();
public IQueryable<T> BuscarTodos()
{
IQueryable<T> query = contexto.Set<T>();
return query;
}
public IQueryable<T> Buscar(Expression<Func<T, bool>> predicado)
{
IQueryable<T> query = contexto.Set<T>().Where(predicado);
return query;
}
public void Adicionar(T entity)
{
contexto.Set<T>().Add(entity);
}
public void Excluir(T entity)
{
contexto.Set<T>().Remove(entity);
}
public void Editar(T entity)
{
contexto.Set<T>().AddOrUpdate(entity);
}
public void Salvar()
{
contexto.SaveChanges();
}
public void Dispose()
{
contexto.Dispose();
}
}
OrderBy
em um método genérico na sua classe de repositório e utilizar para ordernar na sua SQL, o que está acontecendo é que você materializa a informação e depois ordena na memória isso causa baixo desempenho.ToList()
,First()
, etc) poderia adicionar o OrderBy para que na sua SQL esteja presente a ordenação e com o eu relatei não em memória.