Entity Framework "SELECT IN" не использует параметры - PullRequest
Новая
       4

Entity Framework "SELECT IN" не использует параметры

0 голосов
/ 28 августа 2018

Почему Entity Framework помещает литеральные значения в сгенерированный SQL, а не использует параметры при использовании «SELECT IN»: 

using (var context = new TestContext())
{
    var values = new int[] { 1, 2, 3 };
    var query = context.Things.Where(x => values.Contains(x.Id));

    Console.WriteLine(query.ToString());
}

Это приводит к следующему SQL: 

SELECT
    [Extent1].[Id] AS [Id]
    FROM [dbo].[PaymentConfigurations] AS [Extent1]
    WHERE [Extent1].[Id] IN (1, 2, 3)

Я вижу много планов кэшированных запросов в SQL Server. Есть ли способ заставить EF ставить параметры, а не жестко закодированные значения, или активировать сниффинг параметров - единственный вариант?

Это происходит также в EF Core.

Ответы [ 2 ]

0 голосов
/ 28 августа 2018

Не могу сказать , почему Разработчики EF (Core) решили использовать константы вместо переменных при переводе Enumerable.Contains. Как отметил @Gert Arnold в комментариях, это может быть связано с ограничением числа параметров запроса SQL.

Интересно, что EF (6.2) и EF Core (2.1.2) генерируют IN с параметрами, когда вы используете эквивалентное выражение ||, например: 

var values = new int[] { 1, 2, 3 };
var value0 = values[0];
var value1 = values[1];
var value2 = values[2]; 
var query = context.Things.Where(x =>
    x.Id == value0 ||
    x.Id == value1 ||
    x.Id == value2);

EF6.2 сгенерированный запрос 

SELECT
    [Extent1].[Id] AS [Id]
    FROM [dbo].[Things] AS [Extent1]
    WHERE [Extent1].[Id] IN (@p__linq__0,@p__linq__1,@p__linq__2)

EF Core 2.1 делает нечто подобное.

Таким образом, решение заключается в преобразовании выражения Contains в выражение на основе ||. Это должно быть динамически с использованием Expression методов класса. И чтобы его было проще использовать, он может быть инкапсулирован в пользовательский метод расширения, который внутренне использует ExpressionVisitor для выполнения преобразования.

Примерно так: 

public static partial class EfQueryableExtensions
{
    public static IQueryable<T> Parameterize<T>(this IQueryable<T> source)
    {
        var expression = new ContainsConverter().Visit(source.Expression);
        if (expression == source) return source;
        return source.Provider.CreateQuery<T>(expression);
    }

    class ContainsConverter : ExpressionVisitor
    {
        protected override Expression VisitMethodCall(MethodCallExpression node)
        {
            if (node.Method.DeclaringType == typeof(Enumerable) &&
                node.Method.Name == nameof(Enumerable.Contains) &&
                node.Arguments.Count == 2 &&
                CanEvaluate(node.Arguments[0]))
            {
                var values = Expression.Lambda<Func<IEnumerable>>(node.Arguments[0]).Compile().Invoke();
                var left = Visit(node.Arguments[1]);
                Expression result = null;
                foreach (var value in values)
                {
                    // var variable = new Tuple<TValue>(value);
                    var variable = Activator.CreateInstance(typeof(Tuple<>).MakeGenericType(left.Type), value);
                    // var right = variable.Item1;
                    var right = Expression.Property(Expression.Constant(variable), nameof(Tuple<int>.Item1));
                    var match = Expression.Equal(left, right);
                    result = result != null ? Expression.OrElse(result, match) : match;
                }
                return result ?? Expression.Constant(false);
            }
            return base.VisitMethodCall(node);
        }

        static bool CanEvaluate(Expression e)
        {
            if (e == null) return true;
            if (e.NodeType == ExpressionType.Convert)
                return CanEvaluate(((UnaryExpression)e).Operand);
            if (e.NodeType == ExpressionType.MemberAccess)
                return CanEvaluate(((MemberExpression)e).Expression);
            return e.NodeType == ExpressionType.Constant;
        }
    }
}

Применение к примеру запроса 

var values = new int[] { 1, 2, 3 };
var query = context.Things
    .Where(x => values.Contains(x.Id))
    .Parameterize();

производит желаемый перевод.

0 голосов
/ 28 августа 2018

Параметризованные запросы с IN могут быть выполнены, хотя это немного округло. Вам нужно будет использовать прямой запрос SQL и сгенерировать параметризованный SQL вручную, что-то вроде следующего: 

var values = new object[] { 1, 2, 3 };
var idx = 0;
var query = context.Things.SqlQuery($@"
    SELECT
        [Extent1].[Id] AS [Id]
    FROM [dbo].[PaymentConfigurations] AS [Extent1]
    WHERE [Extent1].[Id] IN ({string.Join(",", values.Select(i => $"@p{idx++}"))})",
    values);

Сгенерированный список имен параметров напрямую встроен в SQL, используемый в запросе, и предоставляет значения в параметре values. Обратите внимание, что вам может потребоваться, чтобы ваш массив values был object[], а не int[], чтобы гарантировать его распаковку в параметры SqlQuery. Этот подход не так легко поддерживать, как запрос LINQ, однако иногда нам приходится идти на эти компромиссы ради эффективности.

...