语法问题 IEnumerable;使用yield返回的方法

Question

这是我的方法：

static IEnumerable<DateTime> GetMonths(DateTime from, DateTime to)
{
    // if logs is not uptodate
    TimeSpan logsMissingTimespan = to - from;

    if (logsMissingTimespan != new TimeSpan(0))
    {
        return GetMonthsBetweenTwoDates(from, to);
    }

    return null; // Why this line ?
}

private static IEnumerable<DateTime> GetMonthsBetweenTwoDates(DateTime from, DateTime to)
{

    DateTime date = from;
    DateTime lastDate = DateTime.MaxValue;

    while (date < to)
    {
        if (lastDate.Month != date.Month)
        {
            lastDate = date;
            yield return lastDate;
        }
        date = date.AddDays(1);
    }
}

它工作正常，但我想我可以写一些更干净的东西，如下所示：

static IEnumerable<DateTime> GetMonths(DateTime from, DateTime to)
{
    TimeSpan logsMissingTimespan = to - from;

    if (logsMissingTimespan == new TimeSpan(0))
    {
        yield break;
    }

    return GetMonthsBetweenTwoDates(from, to);
}

但我有一条错误消息：

无法从迭代器返回值。使用yield return语句返回一个值，或者使用yield break结束迭代。

为什么我应该return null以及正确的语法是什么？

编辑：

所以，正确的方法是使用 Enumerable.Empty ：

static IEnumerable<DateTime> GetMonths(DateTime from, DateTime to)
{
    // if logs is not uptodate
    TimeSpan logsMissingTimespan = to - from;

    if (logsMissingTimespan != new TimeSpan(0))
    {
        return GetMonthsBetweenTwoDates(from, to);
    }

    return Enumerable.Empty<DateTime>();
}

Answer 1

您不需要在第一次检查时使用 Yield Break 来结束迭代。

if (logsMissingTimespan == new TimeSpan(0))
{
    return null;
}    
return GetMonthsBetweenTwoDates(from, to);

Answer 2

因为您使用了 yield 这个词，所以它现在期望该方法一次yield一个元素。每次迭代只能使用 yield return 或 yield break 返回一个元素。

您应该使用 Enumerable.Empty(); 而不是 yield break。

Answer 3

前两个示例的形式会产生不同类型的输出。

如果满足条件，您的第一个示例将直接返回 IEnumerable ，如果不满足，则返回 null 引用。您的第二个示例始终返回一个IEnumerable，但条件确定它是否包含任何元素。

第二个示例是通过使用迭代器块来完成的。 C# 编译器使用 yield 语法将您编写的函数转换为实现 IEnumerable 的自定义（隐藏）类型和实现 的类型>IEnumerator。这些类型实现必要的状态机，以实现（希望）您放入函数中的逻辑。正因为如此，你不能混合范例；您必须从函数返回 IEnumerable 实例（并且根本不使用 yield ），或者所有内容都必须通过 yield 返回代码>.

如果您只关心返回 null 引用这一事实，则可以通过返回 Enumerable.Empty 而不是 null 来使这些方法在语义上相同代码>.

Answer 4

方法要么是用迭代器块实现的，要么不是——所以要么一切都以yield return和yield break表示，要么都没有这是。

但是，您不需要做任何特别的事情。您原来的 GetMonthsBetweenTwoDates 已经可以在 to == from 处工作，因为它永远不会进入 while 循环。

另一方面，您对 lastDate 的使用对我来说看起来很可疑 - 特别是，如果 from 碰巧在同一个月，它看起来会做不同的事情DateTime.MaxValue。