System.Linq.Enumerable.Select 的参数输入困难

Question

我已经盯着这个问题看了一段时间，不知道如何解决它。

我要做的就是填写与设备表中的地址匹配的 arptable 描述属性。我不想从 arptable 创建另一个集合。

错误：

无法从用法中推断方法“System.Linq.Enumerable.Select(System.Collections.Generic.IEnumerable, System.Func)”的类型参数。尝试明确指定类型参数。

这是有问题的代码：

 IEnumerable<ARPTABLE> GetARPTable()
    {
        IpTable arp = new IpTable();
        IEnumerable<ARPTABLE> arptable = arp.GetArpTable();


        arptable.ToList().ForEach(i =>
        {
            DeviceTable.Where(j => j.PhysicalAddress == i.MAC)
                       .Select(y =>
                           {
                               i.Description = y.Model ?? y.DeviceName;
                           });

        });

        return arptable;

    }

DeviceTable 在哪里

public ObservableCollection<Device> DeviceTable { get; set; }

感谢您的任何帮助（或更好的方法）。

Answer 1

编译器出现问题，因为您的 lambda 表达式编写不正确，因此在类型推断时失败。不幸的是，该方法的整个构造都被破坏了，我不确定我是否真的理解你在这里想要完成的事情。

但就让它编译而言，你的方法应该看起来更像这样：

IEnumerable<ARPTABLE> GetARPTable()
{
    IpTable arp = new IpTable();
    IEnumerable<ARPTABLE> arptable = arp.GetArpTable();

    foreach (var i in arptable)
    {
        Device j = DeviceTable.FirstOrDefault(j => j.PhysicalAddress == i.MAC);

        if (j != null)
        {
            i.Description = j.Model ?? j.DeviceName;
        }
    }

    return arptable;
}

一般注意：不要使用像Select() 这样的方法来简单地访问集合中的每个元素。除非您评估返回的 IEnumerable<T>，否则您提供给 Select() 方法的表达式实际上不会被评估，在这种情况下您没有这样做。即使您确实评估了IEnumerable<T>，这也是对该方法的低效误用。

此外，虽然 List<T>.ForEach() 可能被某些人认为很方便，但将 IEnumerable<T> 转换为 List<T> 只是为了调用该方法是一种浪费。常规的 foreach 语句可以正常工作并且效率更高。

Answer 2

LINQ 不能用于填充数据。它是一种查询语言，因此Select 方法返回一个新序列。只需使用 foreach 即可。我想它可能看起来像这样，虽然我不确定我的逻辑是否正确。

foreach(var table in arptable)
{
    var device = DeviceTable.SingleOrDefault(...);
    if (device != null)
    {
        table.Description = device.Model ?? device.DeviceName;
    }
}

至于你现在的表格

arptable.ToList().ForEach(i =>

这确实没有必要，如果不需要，为什么要将序列转换为列表？只是为了使用ForEach？我们可以做得更好。

DeviceTable.Where(j => j.PhysicalAddress == i.MAC)
           .Select(y => i.Description = y.Model ?? y.DeviceName);

这将返回一个新序列，您不会将其存储在任何局部变量中。 LINQ 查询不应该有副作用，它违背了 lambda 演算，即 LINQ 本身背后的思想。

Answer 3

我喜欢其他答案。如果您仍想使用linq，您可以这样做：

IEnumerable<ARPTABLE> GetARPTable()
    {
        IpTable arp = new IpTable();
        IEnumerable<ARPTABLE> arptable = arp.GetArpTable();

        arptable = arptable.Select(i =>
        {
            Device device = DeviceTable.SingleOrDefault(j => j.PhysicalAddress == i.MAC);

            if (device != null)
            {
                i.Description = device.Model ?? device.DeviceName;
            }

            return i;
        });

        return arptable;

    }