Question

我有两个元素集合。我试图在第一个集合中获取第二个集合中具有匹配ID的所有元素，并针对匹配元素运行CopyToDomain方法。

以下代码工作正常，但我对它的详细程度感到有些惊讶。 ReSharper在这里没有推荐任何东西，但我想知道是否在这两个集合中进行交集然后将方法映射到元素上会更清楚吗？你会做出这样的改变，还是我应该停止烦恼并保持原样？

Task task = new Task();
IList<TaskAttributeDto> taskAttributeDtos = new List<TaskAttributeDto>();
taskAttributeDtos.Add(new TaskAttributeDto{ ID = 1});
taskAttributeDtos.Add(new TaskAttributeDto{ ID = 2});

foreach (TaskAttributeDto taskAttributeDto in taskAttributeDtos)
{
    TaskAttribute matching = task.TaskAttributes.FirstOrDefault(t => t.ID == taskAttributeDto.ID);
    if (matching != null)
    {
        taskAttributeDto.CopyToDomain(matching);
    }
}

Answer 1

您所做的基本上是一个连接，因此您可以使用Linq连接语法：

var matches =
    from dto in taskAttributesDtos
    join attribute in task.TaskAttributes on dto.ID equals attribute.ID
    select new { dto, attribute};

foreach (var m in matches)
    m.dto.CopyToDomain(m.attribute);

Answer 2

foreach(TaskAttributeDto taskAttributeDto in taskAttributeDtos.Where(t1 => TaskAttributes .Any(t2 => t2.Id == t1.Id)))
{
    taskAttributeDto.CopyToDomain(taskAttributeDto);
}

Answer 3

如果您担心查询语法的可读性，可以尝试使用方法语法：

var matches = taskAttributesDtos.Join(task.TaskAttributes,
                                      a => a.ID,
                                      b => b.ID,
                                      (a, b) => new { dto = a, attribute = b } );

foreach (var m in matches)
    m.dto.CopyToDomain(m.attribute);

Answer 4

task.TaskAttributes.Where(t => taskAttributeDtos.Select(d => d.ID).Contains(t.ID))
    .ToList()
    .ForEach(t => taskAttributeDtos.First(d => d.ID == t.ID).CopyToDomain(t));

Answer 5

原始代码找到第一个匹配项。联接查找所有匹配项，组联接允许您从每个匹配组中选择一名获胜者。

var matches =
    from dto in taskAttributesDtos
    join attribute in task.TaskAttributes
      on dto.ID equals attribute.ID
      into attributeGroup  //into clause causes GroupJoin to be called.
    select new { dto, attribute = attributeGroup.First() };

foreach (var m in matches)
    m.dto.CopyToDomain(m.attribute);

是否有更清晰简洁的方式用更好的LINQ来表达这种说法？

5 个答案: