Question

我正在尝试使用LINQ重新编写以下内容

foreach (WMCommon.Services.StakeOut.assembly ass in assemblies) 
{
  foreach (var agg in aggregate) 
  {
    if ( agg.catagory.unitActn   == ass.unitActn   &&
         agg.catagory.unitCode   == ass.unitCode   &&
         agg.catagory.unitLength == ass.unitLength
    ) 
    {
      ass.quantity = agg.qty;
    }
  }
}

据我所知：

assemblies.Where( a => a.quantity = ( aggregate.Where( p => p.catagory.unitActn == a.unitActn && p.catagory.unitCode == a.unitCode && p.catagory.unitLength == a.unitLength).Select(s=>s.qty)));

提前感谢您的帮助。我希望LINQ比嵌套的FOREACH快得多吗？

Answer 1

我希望LINQ比嵌套的FOREACH快得多？

通常，LINQ不会改善您的性能，除非您改变它的工作方式。 LINQ实际上只是为您执行迭代。

在这种情况下，您似乎可以使用连接来改善整体效果，因为这会给您带来同样的效果：

var query = from WMCommon.Services.StakeOut.assembly ass in assemblies
            join agg in aggregate
            on new { ass.unitActn, ass.unitCode, ass.unitLength } equals new { (agg.catagory.unitActn, agg.catagory.unitCode, agg.catagory.unitLength }
            select new { ass, agg };

foreach(var pair in query)
    pair.ass.quantity = pair.agg.qty;

Answer 2

为什么选择Linq？它不可能比你得到的更快。与你所获得的相比，它不太可能更简单，更容易阅读或更容易调试。

如果您希望提高性能，您的嵌套循环将在O（mn）时间运行，其中 m 是程序集集合的大小， n 是聚集集合的大小，所以基本上是O（n ²）时间。

集合是否按键值排序？如果是这样的话，通过在锁步中迭代两个集合来合并两个集合将是一个巨大的胜利。这应该会让你感受到O（n）表现。

如果没有订购它们，您可以通过获取聚合集合并将其转换为像Dictionary这样的查找表来获得胜利，然后再遍历程序集集合。然后是对程序集的直接迭代，可能会快速查找字典。

Answer 3

试试这个：

assemblies.ForEach(a => a.quantity = (aggregate.Where(p => p.catagory.unitActn == a.unitActn && p.catagory.unitCode == a.unitCode && p.catagory.unitLength == a.unitLength).Select(s=>s.qty)));

但请注意它不会更快

将Nested Foreach转换为Linq

3 个答案: