Question

我是LINQ的新手，并且不想过度使用并且难以维护此代码。

您认为这个LINQ查询太长了吗？

IList<ListViewItem> data = runAnalysis.PassAnalyses.Cast<PassAnalysis>().
  Select(passAnalysis => passAnalysis.GetValue(PeakStatistics.PeakStatisticsProperty)).
  SelectMany(peakStatistics => peakStatistics.Statistics.
    Where(statisticsBase => statisticsBase.Name == statisticType).
    Select(statisticsBase => new ListViewItem {Content = statisticsBase})).ToList();

Answer 1

我会说它由于嵌套而变得复杂 - 但实际上并不需要。这是一个查询的例子，它会给出相同的结果（我认为），但更简单......

IList<ListViewItem> data = runAnalysis.PassAnalyses
      .Cast<PassAnalysis>()
      .Select(pass => pass.GetValue(PeakStatistics.PeakStatisticsProperty))
      .SelectMany(peakStats => peakStats.Statistics)
      .Where(statisticsBase => statisticsBase.Name == statisticType)
      .Select(statisticsBase => new ListViewItem {Content = statisticsBase})
      .ToList();

如果没有嵌套，很容易看出转型是如何进行的：

从PassAnalyses开始
将每个项目投射到PassAnalysis
选择PeakStatistics
从每个元素中选择其中的所有统计信息，并展平此序列
过滤任何名称错误的统计信息
将每个结果转换为ListViewItem
将整个序列转换为列表

此时，很容易将其转换为查询表达式：

IList<ListViewItem> data =
   (from PassAnalysis pass in runAnalysis.PassAnalyses
    from statsBase in pass.GetValue(PeakStatistics.PeakStatisticsProperty)
                          .Statistics
    where statsBase.Name == statisticType
    select new ListViewItem { Content = statsBase })
   .ToList();

请注意，我已经省略了第一个Select和SelectMany;如果需要，可以使用let子句。我还为pass范围变量使用了显式类型，以使编译器生成Cast<PassAnalysis >()调用。

这与原始版本略有不同，因为它会使用不同形式的SelectMany传播原始pass值，但结果将是相同。

最后调用ToList()有些难看，因为它没有查询表达式语法......但你可以使用一个中间变量：

var query = ...;
IList<ListViewItem> data = query.ToList();

Answer 2

IMO查询很好，虽然它看起来有些密集。您可以将其重新格式化为更宽松的空间：

var query = from analysis in runAnalysis.PassAnalyses.Cast<PassAnalysis>()
            let value = analysis.GetValue(PeakStatistics.PeakStatisticsProperty)
            from statistic in value.Statistics
            where statistic.Name == statisticType
            select new ListViewItem { Content = statistic };

IList<ListViewItem> data = query.ToList();

这个LINQ查询太长了吗？

2 个答案: