Дерево выражений - замена 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, выполнить миграцию и вставить ее в базу данных.
Хотел бы я сделать это менее техническим, чтобы на этот вопрос мог ответить более широкий круг людей. Надеюсь, щедрость адекватна для ответа. Если нет, пожалуйста, позвоните мне.
Ответы
Ответ 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);
}