Resharper 删除了 foreach 的产量。为什么？

Question

我最近了解yield并创建了以下测试控制台程序：

    public static string Customers = "Paul,Fred,Doug,Mark,Josh";
    public static string Admins = "Paul,Doug,Mark";

    public static void Main()
    {
        var test = CreateEfficientObject();

        Console.WriteLine(test.Admins.FirstOrDefault());
        //Note that 'GetAllCustomers' never runs. 
    }

    public static IEnumerable<string> GetAllCustomers()
    {
        var databaseFetch = Customers.Split(',');
        foreach (var s in databaseFetch)
        {
            yield return s;
        }
    }

    public static IEnumerable<string> GetAllAdmins()
    {
        var databaseFetch = Admins.Split(',');
        foreach (var s in databaseFetch)
        {
            yield return s;
        }
    }

    static LoginEntitys CreateEfficientObject()
    {
        var returnObject = new LoginEntitys {};
        returnObject.Admins = GetAllAdmins();
        returnObject.Customers = GetAllCustomers();
        return returnObject;
    }
}
public class LoginEntitys
{
    public IEnumerable<String> Admins { get; set; }
    public IEnumerable<String> Customers { get; set; }
}

但我注意到 Resharper 想要将我的foreach循环转换为：

public static IEnumerable<string> GetAllCustomers()
{
    var databaseFetch = Customers.Split(',');
    return databaseFetch;
}

为什么 Resharper 想要从这个案例中去除产量？它完全改变了功能，因为它将不再在没有产量的情况下延迟加载。我只能猜测

• A）我使用yield不正确/实践不当
• B) 这是一个 Resharper 错误/建议，可以忽略。

任何见解都会很棒。

Answer 1

您是对的，这个提议的转换以微妙的方式改变了代码的功能，防止它推迟属性的评估并Split尽早执行评估。

也许那些实现它的人很清楚这是功能上的变化，并认为它仍然是一个有用的建议，如果现有语义很重要，或者如果他们实际上没有意识到语义正在被改变，那么可以忽略这个建议。我们没有什么好办法知道，我们只能猜测。如果这些语义对您的程序很重要，那么您不进行建议的转换是正确的。