如何在 Linq 语句中进行转换并分配正确的值

Question

我一直在尝试以基于表达式的方式编写以下代码，但不确定如何在比较后对对象进行赋值。任何帮助将不胜感激。

var pcs = from a in collection
      group a by a.TotType
      into g
      select new
      {
          TType = g.Key,
          SColl = g.Select(r => r)
     };

for (int i = 0; i < processResult.PAndLReport.BreakdownTotTypeCollection.Count; i++)
{
  foreach (var ttypes in pcs)
  {
    if (processResult.PAndLReport.BreakdownTotTypeCollection[i].TotType == ttypes.TType)
    {
      BreakdownCollection coll = new BreakdownCollection();
      foreach (var item1 in ttypes.SColl)
       {
         coll.Add(item1);
       }

      processResult.PAndLReport.BreakdownTotTypeCollection[i].BreakdownCollection = coll;
     }
  }
}

Answer 1

所以 - 代码可以工作，但是非常冗长。

这是减少代码的尝试。我做了一些假设（此代码并不执行与其他代码完全相同的分配），但它可能接近您想要的。

var pcs = collection.ToLookup(a => a.TotType);

foreach(var bttcoll in processResult
    .PAndLReport
    .BreakdownTotTypeCollection)
{
  var items = pcs[bttcoll.ToTType];
    //do you have a constructor that takes an IEnumerable<Item> ?
  bttcoll.BreakdownCollection = new BreakdownCollection(items)
}

---

一般来说，您应该避免更改 linq 查询内的对象状态。 Linq 来自函数式编程思维，其中状态永远不会改变（相反，会创建具有所需状态的新对象）。

Answer 2

如果 David B 给出的答案有效，那就太好了，但不要在代码中使用它。第一个（大概）有效，并且更容易让下一个人理解。

需要考虑的一些事情：

1. 如果您的代码将使用很长时间，那么维护它的时间将比最初编写的时间多得多。当你忘记自己到底在做什么后不久，你就必须改变行为。让自己变得更轻松。
2. 如果你的代码不会使用很长时间，那么你花了更多的时间来调整风格和流程，然后就会恢复。
3. 与声明性样式相比，使用表达式样式 Linq 没有速度优势。编译器将为您转换代码，并且在大多数情况下将与您自己编写的代码一样高效。