Отложенная казни в хранилище шаблона

Question

В моих UserRepository У меня есть GetActive метод:

public IEnumerable<User> GetActive() 
{ 
    var users = context.Set<UserTbl>().Where(x => x.IsActive); 

    foreach(var user in users) 
    { 
     yield return entityMapper.CreateFrom(user); 
    } 
} 

entityMapper используются для отображения от EF-порожденной UserTbl к доменным User сущности.

Существует тысячи пользователей, поэтому я хочу, чтобы метод GetActive отложил выполнение, возвращая IEnumerable<User>, так что весь список не извлекается без необходимости. Я сделал это выше с foreach и yield.

При тестировании, кажется, что все данные извлекаются независимо. Следующие два вызова принимают одно и то же время:

// Get only 5 users in memory 
var someUsers = UserRepository.GetActive().Take(5).ToList(); 

// Get all 100,000 users into memory 
var allUsers = UserRepository.GetActive().ToList(); 

Что я делаю неправильно?

Answer 1

В тот момент, когда вы используете foreach, данные перечислены. Вы должны использовать IQueryable только до ToList. Ваша идея об отсрочке данных с использованием IEnumerable выглядит неплохо, но это неправильно. IEnumerable всегда возвращает все данные, это просто не заставляет провайдера данных хранить все это в памяти. Вы должны использовать IQueryable, если вы хотите, чтобы поставщик возвращал части данных. Но тогда вы не можете использовать foreach и yield, потому что он всегда перечисляет все данные в своем параметре. Единственный способ сделать то, что вы хотите, - передать требуемый запрос в метод GetActive.

public IEnumerable<User> GetActive(Func<IQueryable<User>, IQueryable<User>> modifier) 
{ 
    var users = modifier(context.Set<UserTbl>().Where(x => x.IsActive)); 

    foreach(var user in users) 
    { 
     yield return entityMapper.CreateFrom(user); 
    } 
} 

// Get only 5 users in memory 
var someUsers = UserRepository.GetActive(q=>q.Take(5)).ToList(); 

// Get all 100,000 users into memory 
var allUsers = UserRepository.GetActive(q=>q).ToList(); 

Но я бы порекомендовал не иметь репозиториев в вашей архитектуре вообще. Они вводят излишнюю сложность по сравнению с уже сложным ORM. См. Больше в Repository pattern with Entity framework