考虑到确切的两个代码片段，我认为版本1更好，因为它可以更有效。假设有很多产品，调用者希望将其转换为dto。

var dtos = GetAllProducts().Select(ConvertToDto).ToList();

在版本2中，首先创建一个Product对象列表，然后创建另一个ProductDto对象列表。版本1没有Product对象的列表，只构建了所需的ProductDto对象的列表。

Even without converting, Version 2 has a problem in my opinion: The list is returned as IEnumerable. The caller of GetAllProducts() does not know how expensive the enumeration of the result is. And if the caller needs to iterate more than once, she will probably materialize once by using ToList() (tools like ReSharper also suggest this). Which results in an unnecessary copy of the list already created in GetAllProducts(). So if Version 2 should be used, the return type should be List and not IEnumerable.

那么这个呢?

public static IEnumerable<Product> GetAllProducts()
{
    using (AdventureWorksEntities db = new AdventureWorksEntities())
    {
        var products = from product in db.Product
                       select product;

        return products.ToList();
    }
}

我想这里干净多了。不过，我手头没有VS2008可以检查。 在任何情况下，如果Products实现了IEnumerable(似乎-它在foreach语句中使用)，我将直接返回它。

Yield return对于需要遍历数百万个对象的算法来说非常强大。考虑以下示例，您需要计算可能的拼车行程。首先我们生成可能的行程:

    static IEnumerable<Trip> CreatePossibleTrips()
    {
        for (int i = 0; i < 1000000; i++)
        {
            yield return new Trip
            {
                Id = i.ToString(),
                Driver = new Driver { Id = i.ToString() }
            };
        }
    }

然后迭代每一次旅行:

    static void Main(string[] args)
    {
        foreach (var trip in CreatePossibleTrips())
        {
            // possible trip is actually calculated only at this point, because of yield
            if (IsTripGood(trip))
            {
                // match good trip
            }
        }
    }

如果您使用List而不是yield，您将需要为内存分配100万个对象(~190mb)，而这个简单的示例将花费~1400ms运行。但是，如果使用yield，就不需要将所有这些临时对象都放到内存中，而且算法速度会大大加快:本例只需要大约400ms就可以运行，完全不消耗内存。

在这种情况下，我将使用代码的版本2。由于您拥有可用产品的完整列表，而这正是此方法调用的“消费者”所期望的，因此需要将完整的信息发送回调用方。

如果该方法的调用者一次只需要“一个”信息，而下一个信息的消耗是按需的，那么使用yield return将是有益的，它将确保当一个单位的信息可用时，执行命令将返回给调用者。

可以使用yield return的例子如下:

复杂的，一步一步的计算，调用者每次等待一个步骤的数据 在GUI中分页-用户可能永远不会到达最后一页，只有子集的信息需要在当前页面上公开

为了回答你的问题，我会使用版本2。

假设您的产品LINQ类使用类似的yield来枚举/迭代，第一个版本更有效，因为它每次迭代只产生一个值。

第二个例子是使用ToList()方法将枚举器/迭代器转换为列表。这意味着它手动遍历枚举器中的所有项，然后返回一个平面列表。

直接返回列表。好处:

这样就更清楚了 该列表是可重用的。(迭代器不是)不是真的，谢谢Jon

You should use the iterator (yield) from when you think you probably won't have to iterate all the way to the end of the list, or when it has no end. For example, the client calling is going to be searching for the first product that satisfies some predicate, you might consider using the iterator, although that's a contrived example, and there are probably better ways to accomplish it. Basically, if you know in advance that the whole list will need to be calculated, just do it up front. If you think that it won't, then consider using the iterator version.