Обходной путь 'Contains ()' с использованием Linq to Entities?

86

Я пытаюсь создать запрос, который использует список идентификаторов в предложении where, используя клиентский api Silverlight ADO.Net Data Services (и, следовательно, Linq To Entities). Кто-нибудь знает об обходном пути, когда Contains не поддерживается?

Я хочу сделать что-то вроде этого:

List<long?> txnIds = new List<long?>();
// Fill list 

var q = from t in svc.OpenTransaction
        where txnIds.Contains(t.OpenTransactionId)
        select t;

Пробовал это:

var q = from t in svc.OpenTransaction
where txnIds.Any<long>(tt => tt == t.OpenTransactionId)
select t;

Но получил "Метод Any" не поддерживается ".

Джеймс Блумер
источник
35
Примечание. Entity Framework 4 (в .NET 4) имеет метод «Содержит», на тот случай, если кто-то случайно об этом прочитает. Я знаю, что OP использовал EF1 (.NET 3.5).
DarrellNorton
7
@ Даррелл Я потратил полчаса впустую, потому что пропустил твой комментарий. Хотел бы я, чтобы ваш комментарий мигал и выделялся на экране.
Крис Дуайер

Ответы:

97

Обновление: EF ≥ 4 поддерживает Containsнапрямую (Checkout Any), поэтому вам не нужно обходное решение.

public static IQueryable<TEntity> WhereIn<TEntity, TValue>
  (
    this ObjectQuery<TEntity> query,
    Expression<Func<TEntity, TValue>> selector,
    IEnumerable<TValue> collection
  )
{
  if (selector == null) throw new ArgumentNullException("selector");
  if (collection == null) throw new ArgumentNullException("collection");
  if (!collection.Any()) 
    return query.Where(t => false);

  ParameterExpression p = selector.Parameters.Single();

  IEnumerable<Expression> equals = collection.Select(value =>
     (Expression)Expression.Equal(selector.Body,
          Expression.Constant(value, typeof(TValue))));

  Expression body = equals.Aggregate((accumulate, equal) =>
      Expression.Or(accumulate, equal));

  return query.Where(Expression.Lambda<Func<TEntity, bool>>(body, p));
}

//Optional - to allow static collection:
public static IQueryable<TEntity> WhereIn<TEntity, TValue>
  (
    this ObjectQuery<TEntity> query,
    Expression<Func<TEntity, TValue>> selector,
    params TValue[] collection
  )
{
  return WhereIn(query, selector, (IEnumerable<TValue>)collection);
}

ПРИМЕНЕНИЕ:

public static void Main()
{
  using (MyObjectContext context = new MyObjectContext())
  {
    //Using method 1 - collection provided as collection
    var contacts1 =
      context.Contacts.WhereIn(c => c.Name, GetContactNames());

    //Using method 2 - collection provided statically
    var contacts2 = context.Contacts.WhereIn(c => c.Name,
      "Contact1",
      "Contact2",
      "Contact3",
      "Contact4"
      );
  }
}
Шимми Вайцхандлер
источник
6
Предупреждение; когда arg является большой коллекцией (у меня было 8500 элементов int list), переполнение стека. Вы можете подумать, что проповедовать такой список - это безумие, но, тем не менее, я думаю, что это обнажает изъян в таком подходе.
dudeNumber4
2
Поправьте меня, если я ошибаюсь. но это означает, что когда переданная коллекция (фильтр) является пустым набором, это в основном приведет к получению всех данных, потому что он только что вернул параметр запроса. Я ожидал, что он отфильтрует все значения, есть ли способ сделать это?
Nap
1
Если вы имеете в виду, что, когда проверяемая коллекция пуста, она не должна возвращать результатов, в приведенном выше фрагменте замените действие if (!collection.Any()) //action;- replace простым возвратом пустого запроса запрошенного типа для лучшей производительности - или просто удалите эту строку.
Шимми Вайцхандлер,
1
return WhereIn (запрос, селектор, коллекция); следует заменить на return WhereIn (query, selector, (IEnumerable <TValue>) collection); чтобы избежать нежелательной рекурсии.
Антуан Обри,
1
Я считаю, что в коде есть ошибка. Если предоставленный список значений пуст, правильное поведение должно заключаться в том, чтобы не возвращать результатов - т. Е. Объекты запроса отсутствуют в коллекции. Однако код делает прямо противоположное - возвращаются все значения, а не ни одно из них. Я считаю, что вы хотите "if (! Collection.Any ()) return query.Where (e => false)"
ShadowChaser
18

Вы можете вернуться к написанию кода e-sql (обратите внимание на ключевое слово "it"):

return CurrentDataSource.Product.Where("it.ID IN {4,5,6}"); 

Вот код, который я использовал для создания некоторого e-sql из коллекции YMMV:

string[] ids = orders.Select(x=>x.ProductID.ToString()).ToArray();
return CurrentDataSource.Products.Where("it.ID IN {" + string.Join(",", ids) + "}");
Роб Фонсека-Энсор
источник
1
У вас есть дополнительная информация об этом? Префикс «it» появляется в примерах MSDN, но я нигде не могу найти объяснение, когда и зачем «это» необходимо.
Роберт Клейпул,
1
Используемый в динамическом запросе Entity Framework, взгляните на geekswithblogs.net/thanigai/archive/2009/04/29/… , Танигайнатан Сирандживи объясняет это там.
Shimmy Weitzhandler
13

Из MSDN :

static Expression<Func<TElement, bool>> BuildContainsExpression<TElement, TValue>(
    Expression<Func<TElement, TValue>> valueSelector, IEnumerable<TValue> values)
{
    if (null == valueSelector) { throw new ArgumentNullException("valueSelector"); }
    if (null == values) { throw new ArgumentNullException("values"); }
    ParameterExpression p = valueSelector.Parameters.Single();

    // p => valueSelector(p) == values[0] || valueSelector(p) == ...
    if (!values.Any())
    {
        return e => false;
    }

    var equals = values.Select(
             value => (Expression)Expression.Equal(valueSelector.Body, Expression.Constant(value, typeof(TValue))));

    var body = equals.Aggregate<Expression>((accumulate, equal) => Expression.Or(accumulate, equal));

    return Expression.Lambda<Func<TElement, bool>>(body, p);
} 

и запрос становится:

var query2 = context.Entities.Where(BuildContainsExpression<Entity, int>(e => e.ID, ids));
Джеймс Блумер
источник
3
Если вы хотите сделать «Не содержит», просто внесите следующие изменения в метод BuildContainsExpression: - Expression.Equal становится Expression.NotEqual - Expression.Or становится Expression.And
Мерритт
2

Я не уверен насчет Silverligth, но в linq to objects я всегда использую any () для этих запросов.

var q = from t in svc.OpenTranaction
        where txnIds.Any(t.OpenTransactionId)
        select t;
АндреасN
источник
5
Any не принимает объект типа последовательности - он либо не имеет параметров (в этом случае он просто «пуст ли он»), либо принимает предикат.
Джон Скит,
Я ужасно рад, что нашел этот ответ :) +1 Спасибо, AndreasN
SDReyes
1

Вот код, который я наконец использовал (проверка ошибок опущена для ясности) ...

// How the function is called
var q = (from t in svc.OpenTransaction.Expand("Currency,LineItem")
         select t)
         .Where(BuildContainsExpression<OpenTransaction, long>(tt => tt.OpenTransactionId, txnIds));



 // The function to build the contains expression
   static System.Linq.Expressions.Expression<Func<TElement, bool>> BuildContainsExpression<TElement, TValue>(
                System.Linq.Expressions.Expression<Func<TElement, TValue>> valueSelector, 
                IEnumerable<TValue> values)
        {
            if (null == valueSelector) { throw new ArgumentNullException("valueSelector"); }
            if (null == values) { throw new ArgumentNullException("values"); }
            System.Linq.Expressions.ParameterExpression p = valueSelector.Parameters.Single();

            // p => valueSelector(p) == values[0] || valueSelector(p) == ...
            if (!values.Any())
            {
                return e => false;
            }

            var equals = values.Select(value => (System.Linq.Expressions.Expression)System.Linq.Expressions.Expression.Equal(valueSelector.Body, System.Linq.Expressions.Expression.Constant(value, typeof(TValue))));
            var body = equals.Aggregate<System.Linq.Expressions.Expression>((accumulate, equal) => System.Linq.Expressions.Expression.Or(accumulate, equal));
            return System.Linq.Expressions.Expression.Lambda<Func<TElement, bool>>(body, p);
        }
Джеймс Блумер
источник
0

Огромное спасибо. Мне было достаточно метода расширения WhereIn. Я профилировал его и создал ту же команду SQL для базы данных, что и e-sql.

public Estado[] GetSomeOtherMore(int[] values)
{
    var result = _context.Estados.WhereIn(args => args.Id, values) ;
    return result.ToArray();
}

Создано это:

SELECT 
[Extent1].[intIdFRLEstado] AS [intIdFRLEstado], 
[Extent1].[varDescripcion] AS [varDescripcion]
FROM [dbo].[PVN_FRLEstados] AS [Extent1]
WHERE (2 = [Extent1].[intIdFRLEstado]) OR (4 = [Extent1].[intIdFRLEstado]) OR (8 = [Extent1].[intIdFRLEstado])
Jrojo
источник
0

Я думаю, что присоединение к LINQ может быть обходным решением.

Однако я не тестировал код. Надеюсь, это поможет. Ура. :-)

List<long?> txnIds = new List<long?>();
// Fill list 

var q = from t in svc.OpenTransaction
        join tID in txtIds on t equals tID
        select t;

Присоединяйтесь к LINQ:

http://weblogs.asp.net/salimfayad/archive/2008/07/09/linq-to-entities-join-queries.aspx

Габриэль Чанг
источник
0

Извините, новый пользователь, я бы прокомментировал фактический ответ, но, кажется, я еще не могу этого сделать?

В любом случае, что касается ответа с образцом кода для BuildContainsExpression (), имейте в виду, что если вы используете этот метод для сущностей базы данных (т.е. не в объектах в памяти), и вы используете IQueryable, он действительно должен перейти в базу данных так как он в основном выполняет много SQL »или« условий »для проверки предложения« where in »(запустите его с помощью SQL Profiler, чтобы увидеть).

Это может означать, что если вы уточняете IQueryable с помощью нескольких BuildContainsExpression (), он не превратит его в один оператор SQL, который запускается в конце, как вы ожидаете.

Обходной путь для нас заключался в использовании нескольких объединений LINQ для сохранения одного вызова SQL.

Шеннон
источник
0

В дополнение к выбранному ответу.

Заменить Expression.Orс Expression.OrElseиспользовать с NHibernate и исправить Unable to cast object of type 'NHibernate.Hql.Ast.HqlBitwiseOr' to type 'NHibernate.Hql.Ast.HqlBooleanExpression'исключение.

smg
источник