OrderUsers(ref users, orderField, orderBy);
private void OrderUsers(ref IQueryable<User> users, OrderField orderField, OrderBy orderBy)
{
switch (orderField)
{
case OrderField.Name:
users = orderBy == OrderBy.Asc
? users.OrderBy(x => x.Name)
: users.OrderByDescending(x => x.Name);
break;
case OrderField.CreatedDate:
users = orderBy == OrderBy.Asc
? users.OrderBy(x => x.CreatedDate)
: users.OrderByDescending(x => x.CreatedDate);
break;
default:
break;
}
}
Пересмотри реализацию
Причина в том что после любого метода IQueryable возвращается новый объект, а не изменяется старый, поэтому ты не можешь из метода OrderUsers заменить что-то в users без ref - тебе нужно именно ссылку на переменную users дать, чтобы её поменять. Иначе без реф ты перезаписываешь параметр внутри метода OrderUsers, но за пределы данного метода новое значение не попадёт. Если конкретнее, то ссылочный тип хоть и не копируется при передаче в метод (именно объект из кучи), но ссылка копируется, итого ты получаешь 2 ссылки, которые смотрят на один и тот же объект в куче. Ты без реф перезаписываешь чтобы вторая ссылка смотрела на новый объект, при этом старая останется смотреть на старый объект users, ref в данном случае передает ту же самую ссылку, поменяется объект и вне метода OrderUsers. Варианты следующие: 1) через реф как у тебя, но выглядит как уродство 2) создать метод расширения и написать users = users.OrderUsers(); 3) без метода расширения users = OrderUsers(users);
Это кстате справедливо к любому реф объекту а не только к IQueryable
Короче ему нужно почитать про fluent api и тогда подобный вопрос не возникнет
Согласен, соблюдать fluent лучше всего для linq
Обсуждают сегодня