Question

我有以下方法，需要花费很长时间才能运行，并希望得到一些帮助，以使其运行更快，或更高效。

该方法的主要职责是获取从CSV文件创建的数据点列表，将文件数据点的Name属性映射到标记名列表中的HistorianTagname属性通过DataLoggerTagname属性并从映射创建结果列表。如果映射不存在，则忽略文件数据点。

我知道这是冗长的，但我希望这是有道理的。查看方法可能更容易：

    private IEnumerable<DataPoint> GetHistorianDatapoints(IEnumerable<DataPoint> fileDatapoints, IEnumerable<Tagname> historianTagnames)
    {
        /**
         ** REFACTOR THIS 
         **/

        foreach (var fileDatapoint in fileDatapoints)
        {
            var historianTagname = historianTagnames.FirstOrDefault(x => x.DataLoggerTagname.Equals(fileDatapoint.Name, StringComparison.OrdinalIgnoreCase));
            if (historianTagname != null)
            {
                var historianDatapoint = new DataPoint();

                historianDatapoint.Name = historianTagname.HistorianTagname;
                historianDatapoint.Date = fileDatapoint.Date;
                historianDatapoint.Value = fileDatapoint.Value;

                yield return historianDatapoint;
            }
        }
    }

注意：我完全控制了映射的类和方法，所以如果我做的事情从根本上是错误的。我会很高兴知道！

谢谢！

Answer 1

我首先要修理：

var historianTagname = historianTagnames.FirstOrDefault(x => x.DataLoggerTagname.Equals(fileDatapoint.Name, StringComparison.OrdinalIgnoreCase))

通过这个循环运行每次迭代都是非常昂贵的操作。

Answer 2

喜欢 @Sheldon Warkentin 说FirstOrDefault可能是你职能的瓶颈，最好创建一个{Name}是Dictionary，其中Name是关键，然后在你的函数中你可以通过钥匙获得价值。

像下面的东西：

// this is passed to method
IDictionary<string, Tagname> historianTagnames;
// .. method body
var historianTagname = historianTagnames[fileDatapoint.Name];

当然你需要添加正确的if。

Answer 3

以下是我的主张：

private IEnumerable<DataPoint> GetHistorianDatapoints(IEnumerable<DataPoint> fileDatapoints, IEnumerable<Tagname> historianTagnames)
{
    var tagNameDictionary = historianTagnames.ToDictionary(t => t.DataLoggerTagname, StringComparer.OrdinalIgnoreCase);

    foreach (var fileDatapoint in fileDatapoints)
    {                
        if (tagNameDictionary.ContainsKey(fileDatapoint.Name))
        {
            var historianTagname = tagNameDictionary[fileDatapoint.Name];
            var historianDatapoint = new DataPoint();

            historianDatapoint.Name = historianTagname.HistorianTagname;
            historianDatapoint.Date = fileDatapoint.Date;
            historianDatapoint.Value = fileDatapoint.Value;

            yield return historianDatapoint;
        }
    }
}

Answer 4

正如其他人所说，Dictionary<string, Tagname>可能表现得更好。

var historianDict = new Dictionary<string, Tagname>();
foreach (var tagName in historianTagnames) {
    historianDict[tagName.DataLoggerTagname.ToLowerInvariant()] = tagName;
}

foreach (var fileDatapoint in fileDatapoints) {
    if (historianDict.ContainsKey(fileDatapoint.Name.ToLowerInvariant()) {
        // ...
    }
}

是否有更有效的方法来创建基于现有列表和查找列表的列表？

4 个答案: