Linq：在进行投影时设置属性

Question

我正在做一个简单的 GroupBy 获取第一个元素，但我想修改每个结果的一个属性。

class M
{
 public string Name {get; set;}
 public int NOfPeopleWithTheSameName {get; set;}
 public string P1 {get; set;}
 public string P2 {get; set;}
 public string P3 {get; set;}
 public string P4 {get; set;}
 public string P5 {get; set;}
}  


List<M> myList = GetMyList();


var l2 = myList
   .GroupBy(m => m.Name)
   .Select(group => new M { Name = group.Key, NOfPeopleWithTheSameName = group.Count() });

这很简单，但如果类有很多属性（因为每个属性值都应该复制到新的属性值），这种方法不是最好的吗？我应该一个一个地复制它们。

我想简单地获取元素并更改属性 NOfPeopleWithTheSameName

Answer 1

你不需要创建一个新的M，你可以返回现有的，例如：

var l2 = myList
   .GroupBy(m => m.Name)
   .Select(group => 
   {
        var m = group.First();
        m.NOfPeopleWithTheSameName = group.Count();
        return m;
   });

但是，如果您打算为此添加一个属性到您的模型中，我建议您使用一个不同的类来包装初始模型和计数 - 不要污染您的模型。例如，你可以有一个像这样的通用包装类：

public class ModelCount<T>
{
    public T Model { get; set; }
    public int Count { get; set; }
}

现在像这样分组：

var l2 = myList
   .GroupBy(m => m.Name)
   .Select(group => new ModelCount<M> 
   {
        Model = group.First(),
        Count = group.Count()
   });

Answer 2

是的，你可以...

class Test 
{
    public string Name { get; set; }
    public int Number { get; set; }
}

var tests = new List<Test> { /* ... your data here ... */ };

var modifiedTests = tests.Select(test => { test.Name = "MODIFIED"; return test; });

// this actually executes above query and modifies your original items in the list:
var modifiedMaterialized = modifiedTests.ToList();

但你真的，真的（！）不应该！

LinQ 是语言集成的查询。有副作用的查询是邪恶的。胡子扭邪。只是不要，你会为自己省去很多痛苦。

---

我认为您想要的不是 LinQ，而是常规循环：

class M
{
 public string Name {get; set;}
 public int NOfPeopleWithTheSameName {get; set;}
 public string P1 {get; set;}
 public string P2 {get; set;}
 public string P3 {get; set;}
 public string P4 {get; set;}
 public string P5 {get; set;}
}  

List<M> myList = GetMyList();

var groups = myList.GroupBy(m => m.Name).ToList();

foreach(var group in groups)
{
    foreach(var member in group)
    {
        member.NOfPeopleWithTheSameName = group.Count();
    }
}