Подтвердить что ты не робот

Дерево выражений - замена Math.Max

Когда я использую деревья выражений для замены метода, такого как Math.Max, похоже, что он успешно заменяет его в дереве выражений. Но когда я использую его в Entity Framework, он выдает исключение из-за того, что не поддерживает Math.Max ​​для Entity Framework. Но я явно заменяю его.

Кто-нибудь знает, почему? И способ исправить код?


using System;
using System.Collections.Generic;
using System.Linq;
using System.Linq.Expressions;

namespace ConsoleApplication1
{
    public static class CalculateDatabase
    {
        public static void Main(string[] args)
        {
            var calcs = GetCalculateToAmounts(GetTestItems(), 0.5m).ToList();
        }

        public static IQueryable<Item> GetTestItems()
        {
            var items = new List<Item>();
            items.Add(new Item()
            {
                DoNotItem = true,
                ReductionAmount = 2,
                PreviousDiscountAmount = 3,
                CurrentDiscountAmount = 10,
                CurrentAmount = 100,
                PreviousAmount = 50,
                CurrentBillAmount = 75
            });

            return items.AsQueryable();
        }

        public class Item
        {
            public bool DoNotItem { get; set; }
            public decimal ReductionAmount { get; set; }
            public decimal PreviousDiscountAmount { get; set; }
            public decimal CurrentDiscountAmount { get; set; }
            public decimal CurrentAmount { get; set; }
            public decimal PreviousAmount { get; set; }
            public decimal CurrentBillAmount { get; set; }
        }

        public static IQueryable<CalculateToAmount> GetCalculateToAmounts(this IQueryable<Item> entityItems, decimal percentage)
        {
            return entityItems.Select(CalculateAmountExpression(percentage));
        }

        public class CalcType
        { }

        public class CalculateToAmount
        {
            public CalcType CalcType { get; set; }
            public Item Item { get; set; }
            public decimal ItemAmount1 { get; set; }
            public decimal ItemAmount2 { get; set; }
            public decimal ItemAmount3 { get; set; }
            public decimal Bonus { get; set; }
            public decimal Discounts { get; set;  }
            public decimal Total { get; set; }
        }

        private static Expression<Func<Item, CalculateToAmount>> CalculateAmountExpression(this decimal percentage)
        {
            Expression<Func<Item, CalculateToAmount>> lambda = item => new CalculateToAmount()
            {
                Item = item,

                Bonus = item.DoNotItem
                    ? 0 
                    : item.CurrentBillAmount * (1 - percentage) + item.ReductionAmount,
                Discounts = item.PreviousDiscountAmount + item.CurrentDiscountAmount,
                Total = Math.Max(item.CurrentAmount + item.PreviousAmount, item.CurrentBillAmount)
            };

            var test = MathModifier.Modify(lambda);
            return test;
        }

        public class MathModifier : ExpressionVisitor
        {
            protected override Expression VisitMethodCall(MethodCallExpression node)
            {
                var isMinMethod = node.Method.Name.Equals("Min", StringComparison.InvariantCultureIgnoreCase);
                var isMaxMethod = node.Method.Name.Equals("Max", StringComparison.InvariantCultureIgnoreCase);

                if (!isMinMethod && !isMaxMethod)
                    return base.VisitMethodCall(node);

                var left = node.Arguments[0];
                var right = node.Arguments[1];

                var minMaxReplaceMethod =
                    isMinMethod
                        ? Expression.Condition(Expression.LessThan(left, right), left, right)
                        : Expression.Condition(Expression.GreaterThan(left, right), left, right);

                return minMaxReplaceMethod;
            }

            public static Expression<Func<TIn, TOut>> Modify<TIn, TOut>(Expression<Func<TIn, TOut>> expression)
            {
                var modifier = new MathModifier();
                return (Expression<Func<TIn, TOut>>)modifier.Visit(expression);
            }
        }
    }
}

Если вы вызываете

var calcs = GetCalculateToAmounts(GetTestItems(), 0.5).ToList()

он будет работать. Но если вы замените вышеуказанный GetTestItems() инфраструктурой сущности _dbContext.Items, это не сработает.

Чтобы протестировать этот код, вам нужно будет добавить структуру элемента в проект EF, выполнить миграцию и вставить ее в базу данных.

Хотел бы я сделать это менее техническим, чтобы на этот вопрос мог ответить более широкий круг людей. Надеюсь, щедрость адекватна для ответа. Если нет, пожалуйста, позвоните мне.

4b9b3361

Ответ 1

Я выполняю именно ваш предоставленный код с EF 6. Сущность Item с именем Table1. Код успешно завершен, исправление Math.Max ​​успешно завершено.

Отслеживание SQL с помощью SQL Profiler генерирует и отправляет следующий SQL-код в базу данных:

exec sp_executesql N'SELECT 
[Extent1].[Id] AS [Id], 
[Extent1].[DoNotItem] AS [DoNotItem], 
[Extent1].[ReductionAmount] AS [ReductionAmount], 
[Extent1].[PreviousDiscountAmount] AS [PreviousDiscountAmount], 
[Extent1].[CurrentDiscountAmount] AS [CurrentDiscountAmount], 
[Extent1].[CurrentAmount] AS [CurrentAmount], 
[Extent1].[PreviousAmount] AS [PreviousAmount], 
[Extent1].[CurrentBillAmount] AS [CurrentBillAmount], 
CASE WHEN ([Extent1].[DoNotItem] = 1) THEN cast(0 as decimal(18)) ELSE ([Extent1].[CurrentBillAmount] * (cast(1 as decimal(18)) - @p__linq__0)) + [Extent1].[ReductionAmount] END AS [C1], 
[Extent1].[PreviousDiscountAmount] + [Extent1].[CurrentDiscountAmount] AS [C2], 
CASE WHEN (([Extent1].[CurrentAmount] + [Extent1].[PreviousAmount]) > [Extent1].[CurrentBillAmount]) THEN [Extent1].[CurrentAmount] + [Extent1].[PreviousAmount] ELSE [Extent1].[CurrentBillAmount] END AS [C3]
FROM [dbo].[Table1] AS [Extent1]',N'@p__linq__0 decimal(1,1)',@p__linq__0=0.5

Заключение: код, который вы указали, работает, манипуляция с выражением дерева работает, ваша условная логика в дереве переводится в выражение SQL CASE.

Возможно, ваш EF-тест недостаточно изолирован, и исключение EF, связанное с Max.Math, исходит из другой части кода и/или опосредованно из уволенного события, перегрузки и т.д.

Ответ 2

Есть ли причина, по которой вы оставили это в земле IQueryable? IQueryable земля заставляет все работать в базе данных. Если вы переместите его на землю IEnumerable, я думаю, что он должен работать нормально:

    public static IEnumerable<CalculateToAmount> GetCalculateToAmounts(this IQueryable<Item> entityItems, decimal percentage)
    {
        var func = CalculateAmountExpression(percentage).Compile();
        return entityItems.AsEnumerable().Select(func);
    }