这有点离题了，但由于这个问题被标记为最佳实践，我将继续发表我的意见。对于这种类型的东西，我非常喜欢把它变成一个属性:

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

            return products;
        }
    }
}

当然，它更像一个样板，但使用它的代码将看起来更干净:

prices = Whatever.AllProducts.Select (product => product.price);

vs

prices = Whatever.GetAllProducts().Select (product => product.price);

注意:我不会对任何可能需要一段时间才能完成工作的方法这样做。

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

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

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

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

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

当我计算列表中的下一项(甚至是下一组项)时，我倾向于使用yield-return。

使用版本2，在返回之前必须有完整的列表。 通过使用yield-return，您实际上只需要在返回前获得下一项。

除此之外，这有助于在更大的时间框架内分散复杂计算的计算成本。例如，如果列表连接到GUI，而用户从未访问到最后一页，则永远不会计算列表中的最终项。

yield-return更可取的另一种情况是IEnumerable表示无限集。考虑素数列表，或者无限随机数列表。您永远不能一次返回完整的IEnumerable，因此使用yield-return以增量方式返回列表。

在您的特定示例中，您拥有完整的产品列表，因此我将使用版本2。

那么这个呢?

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语句中使用)，我将直接返回它。

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

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

收益率有两大用途

它有助于在不创建临时集合的情况下提供自定义迭代。(加载所有数据并循环)

它有助于进行有状态迭代。(流)

下面是一个简单的视频，我已经创建了完整的演示，以支持上述两点

http://www.youtube.com/watch?v=4fju3xcm21M