c# - 计算计算属性的最佳方法 - Linq 或 foreach 循环？

Question

事实上，我的代码正在运行，并以三种不同的方式进行了测试，但我很想知道实现它的最佳方式是什么。

有一个带有集合的 Vehicle.cs 类，并且想要计算所有条目的平均消耗量。

这有效：

public ICollection<FuelingEntry> FuelingEntries { get; set; }

public double TotalAverageConsumption
{
    get
    {
        double fuelingEntriesQuantitySum = 0;
        double fuelingEntriesTripOdometerSum = 0;

        foreach (var fuelingEntry in FuelingEntries)
        {
            fuelingEntriesQuantitySum += fuelingEntry.Quantity;
            fuelingEntriesTripOdometerSum += fuelingEntry.TripOdometer;
        }

        return (fuelingEntriesQuantitySum / fuelingEntriesTripOdometerSum) * 100;
    }
}

还有“基本”LINQ：

public ICollection<FuelingEntry> FuelingEntries { get; set; }

public double TotalAverageConsumption
{
    get
    {
        var fuelingEntriesQuantitySum = (from f in FuelingEntries select f.Quantity).Sum();
        var fuelingEntriesTripOdometerSum = (from f in FuelingEntries select f.TripOdometer).Sum();

        // var fuelingEntriesQuantitySum = FuelingEntries.Select(f => f.Quantity).Sum();
        // var fuelingEntriesTripOdometerSum = FuelingEntries.Select(f => f.TripOdometer).Sum();

        return (fuelingEntriesQuantitySum / fuelingEntriesTripOdometerSum) * 100;
    }
}

哪个是最好的方法还是有另一种？

Answer 1

正如所写的foreach循环的第一个选项更有效，因为您只需循环一次集合，您的 linq 版本循环两次。如果在访问属性时可以更新集合或集合项，这两种方法都将容易失败或出现奇怪的结果。