Em um dos projetos que peguei vi o seguinte código, que implementa um método de listagem de dados de uma determinada entidade, porém esta listagem é utilizada apenas para leitura:
/// <summary>
/// Listar todas entidades
/// </summary>
/// <returns></returns>
public IList<T> ListarTodos()
{
return ctxContexto.Set<T>().ToList();
}
O método funciona perfeitamente, a minha dúvida é com relação a desempenho e boas práticas de programação com o EntityFramework. Há um tempo atrás vi uma pergunta aqui no SOpt que falava da diferença entre IEnumerable, IQueryable e IList.
Baseado na questão acima citada, pensei "Se meu método retornar um List<T>
estarei dando à pessoa que chamar o método a possibilidade de inserir ou remover itens da lista", o que não é correto, visto que a minha lista é somente leitura.
List<Cliente> lista = new Servico<Cliente>().ListarTodos();
lista.RemoveAt(0)
lista.Add(new Cliente());
Mudei o retorno do método para IEnumerable conforme abaixo:
/// <summary>
/// Listar todas entidades
/// </summary>
/// <returns></returns>
public IEnumerable<T> ListarTodos()
{
return ctxContexto.Set<T>().AsEnumerable();
}
O código acima também funcionou, mais toda vez que chamo o método ListarTodos, se eu realizar mais de uma verificação com o item da lista no mesmo contexto de código:
List<Cliente> lista = new Servico<Cliente>().ListarTodos();
int count = lista .Count();
Console.WriteLine(count);
Console.WriteLine(lista .Any());
É apresentada a mensagem pelo Resharper.
Possible multiple enumeration of IEnumerable
Isto também já foi tratado em uma pergunta aqui do SOpt e eu entendi o porque da mesma, as minhas dúvidas são:
Qual é a melhor abordagem neste caso?
Alguma delas me trará algum benefício de desempenho?