这个LINQ查询太长了吗？

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();