无法翻译带有 LINQ 表达式的 ForEach 循环？

Question

如果过期日期是 2 天或 15 天，我需要检查我所有的化学品并向用户发送警告。 我已经这样设置了，所以警告被发送了两次。到期前 2 周一次，到期前 2 天一次。

所以我有这个循环：

foreach (var chemical in _context.Chemical.Where(c => (c.DateOfExpiration - DateTime.Now).TotalDays == 15 || (c.DateOfExpiration - DateTime.Now).TotalDays == 2))
{
    // send out notices
    var base = _context.Base.Find(chemical.BaseId);
    var catalyst = _context.Catalyst.Find(chemical.CatalystId);

    _warningService.SendSafetyWarning(chemical, base, catalyst.Name);
}

但每当我运行它时，我都会收到此错误：

System.InvalidOperationException: 'The LINQ expression 'DbSet<Chemical>
.Where(p => (p.DateOfExpiration - DateTime.Now).TotalDays == 15 || (p.DateOfExpiration - DateTime.Now).TotalDays == 2)'
could not be translated.

我不确定为什么我运行它时会出现这个错误。

有没有更好的方法来做我想做的事情？

谢谢！

Answer 1

DateTime.TotalDays 返回一个双精度值。

您可能会更幸运地将代码更改为：

foreach (var chemical in _context.Chemical.Where(c => ((int)(c.DateOfExpiration - DateTime.Now).TotalDays == 15) || ((int)(c.DateOfExpiration - DateTime.Now).TotalDays == 2)))
{
    // send out notices
    var base = _context.Base.Find(chemical.BaseId);
    var catalyst = _context.Catalyst.Find(chemical.CatalystId);

    _warningService.SendSafetyWarning(chemical, base, catalyst.Name);
}

Answer 2

这个问题有两种解决方案。

我将首先解释为什么它会引发错误。 EF 正在将您的代码转换为 SQL Expression。使用 Expression 时，您无法解析任何 C# 代码并期望它能够正常工作。 EF 需要能够将其转换为 SQL。这就是为什么您会收到异常说它无法翻译您的代码。

当直接在DbSet 或DbContext 上的Where 子句中工作时，您实际上是在与Where 方法对话以获取IQueryable。其中有一个Expression<Func<bool, T>> 参数，其中T 是您的DbSet<T> 类型。

您需要简化查询，以便将其转换为 SQL。或者您可以将您的IQueryable 转换为IEnumerable。

当您想在 SQL 服务器上运行查询时，您需要使用第一种方法。但是，当您可以在客户端上运行查询时，您可以使用第二种方法。

vivek nuna 提到的第一个方法是使用EntityFunctions。它可能不足以满足您的用例，因为它的功能有限。

DateTime.Now 每次迭代都不同的说法是不正确的。因为查询只转换一次，所以它不会在每次迭代时都运行查询。它根本不知道如何将您的查询转换为 SQL。

https://docs.microsoft.com/en-us/dotnet/api/system.data.objects.entityfunctions?view=netframework-4.8

第二种方法是在DbSet 之后添加AsEnumerable。这将从您的 SQL 服务器查询您的所有数据，并在 C# 中进行评估。如果您有大量数据集，通常不建议这样做。示例：

var chemicals = _context.Chemical.AsEnumerable(); // this will get the complete collection

chemicals.Where(i => i.Value == true); // everything will work now

---

并回答“更好”的方法。代码更简洁：

如果您有 Base 和 Catalyst 的导航属性，您也可以将其包含在查询中。

请注意，您仍然可以在使用_context.Chemical 上的Where 拉取所有客户端之前查询服务器端，然后包括并调用AsEnumerable。

var chemicals = _context.Chemical.Include(i => i.Base).Include(i => i.Catalyst).AsEnumerable();

foreach (var chemical in chemicals.Where(i => true)) // set correct where clause
{
    _warningService.SendSafetyWarning(chemical, chemical.Base, chemical.Catalyst);
}

Answer 3

您可以在之前声明变量，然后将其传递给您的表达式。喜欢DateOfExpiration == nextFifteenDays .....

var nextFifteenDays = DateTime.Now.Adddays(15);


var nextTwoDays = DateTime.Now.AddDays(2);

或者你可以使用EntityFunctions.Diffdays

@neil 很好地解释了错误的原因，即 DateTime.Now 将不断变化，因此无法使用。