将 Foreach 转换为 linq 表达式

Question

我正在尝试从Device 表中获取针对当前客户的设备列表。我做到了，但使用了 foreach 循环，但我想使用 linq 来进行操作。

这是我的班级结构

public class Device
{
    public string type { get; set; }
    public int id { get; set; }
    public List<Role> roles { get; set; }
}


public class Role
{
    public int roleId { get; set; }
    public string roleName { get; set; }
    public int customerId { get; set; }
}

这是我的代码

var devList = list.Where(x => x.type == "device1").ToList();

foreach (var item in devList)
{
    if (item.roles != null)
    {
        var kss = item.roles.Where(z => z.customerId == kunId).FirstOrDefault();
        if (kss != null)
        {
            m.devicesList.Add(item);
        }
    }
}

我需要针对当前客户获取设备，因此我必须将我当前的用户 ID 与 customerId 进行比较。

如何将其转换为 linq-to-sql？

Answer 1

有两种方法可以解决这个问题。

首先，您可以将所有内容放入复杂的Where 操作中：

var devices = list.Where(d => d.type == "device1" && d.roles is object && d.roles.Any(r => r.customerId == kunId) );

其次，您可以将其分散到多个单独的操作中，但每个操作都更容易理解：

var devices = list.Where(d => d.type == "device1").
                   Where(d => d.roles is object).
                   Where(d => d.roles.Any(r => r.customerId == kunId) );

任一选项的性能将相似。额外的Where() 调用会产生开销，但并没有你想象的那么多...... JITter 可以用这种代码做一些惊人的事情，看到这两个结果我不会感到惊讶完全相同的 IL。

还要注意在这两种情况下都没有ToList() 调用。通过更长时间地使用IEnumerable，您通常可以大大提高性能和内存使用率。

此时，我不清楚您是否只是附加到现有的 m.devicesList 或这些值是否可以替换该集合中当前可能存在的任何内容。

假设m.devicesList 被声明为List<Device>（或类似），前一个选项将如下所示：

m.devicesList.AddRange(devices);

同样，没有必要曾经致电ToList()。

后者看起来像这样：

m.devicesList = devices.ToList();

我们还可以将这两个步骤合并到一个语句中。以下是四种可能的组合之一：

m.devicesList.AddRange(list.Where(d => d.type == "device1").
                            Where(d => d.roles is object).
                            Where(d => d.roles.Any(r => r.customerId == kunId) )
                      );

Answer 2

所以你有一个对象kunId，它与Role.CustomerId 的类型相同。您在对象devList 中还有一个Items 序列。最后，你还有一个属性m.DevicesList，它实现了ICollection<Item>（=你有一个方法Add(Item)）

您当前的方法检查您的项目序列中的每个Item。如果属性Roles 为空，则忽略该项目。另一方面，如果您有一些角色，您将获得第一个或默认角色，其属性CustomerId 的值等于 kunId。如果有这样的非默认角色，那么您将项目添加到m.DevicesList。

我将重新表述这个要求。

当且仅当 Item 具有属性 Roles 的非空值，并且这些角色中的至少一个角色具有等于 kunId 的属性 CustomerId 的值，然后您将项目添加到序列中m.DevicesList中的项目数

var itemsToAddToMDevicesList = devList.Where(item => 
    item.Roles != null &&
    item.Roles.Where(role => role.CustomerId == kunId).Any();

换句话说：您要添加到m.DeviceList 的Items 的序列是devList 中所有Items 的序列，其中属性Roles 具有非空值，并且至少具有属性 Roles 中的一个 Role，其属性 CustomerId 的值等于 kunId。

Answer 3

始终尝试首先用简单的自然语言“实现”您的要求。

您需要什么设备？那些：

1. 类型为“device1”
2. 具有分配给给定客户的角色

分别构建每个部分，添加必要的防护，例如roles != null 你会得到这样的结果：

var devicesFound = list.Where(d => 
   d.type == "device1" && 
   d.roles != null && 
   d.roles.Any(r => r.customerId == kunId));

m.devicesList.AddRange(devicesFound);

Answer 4

您应该能够删除分配给 devList 的 linq 表达式值末尾的 .ToList()。

此外，如果您的目标是优化您正在执行的操作，您可能会考虑维护当前代码，而不是将其转换为 linq 表达式。

不过，如果您需要转换 foreach 循环，请查看 official microsoft documentation，您应该能够在智能感知的帮助下自动完成。