您如何使用流操作来计算列表中值的平均值，省略一些？

Question

我有一个方法可以返回多个模型对象的属性平均值：

List<Activity> activities = ...;
double effortSum = 0;
double effortCount = 0;
activities.stream().forEach(a -> {
    double effort = a.getEffort();
    if (effort != Activity.NULL) {
        effortCount++;            < Compilation error, local variable
        effortSum += effort;      < Compilation error, local variable
    }
});

但是，如上所述，上述尝试无法编译。我想到的唯一解决方案是使用AtomicReference 到Double 对象，但这看起来很笨拙，并且给应该简单的操作增加了大量的混乱。 （或者加番石榴，获得AtomicDouble，但得出同样的结论。）

是否有使用新的 Java 8 循环修改局部变量的“最佳实践”策略？

Activity的相关代码：

public class Activity {
    public static final double NULL = Double.MIN_VALUE;

    private double effort = NULL;

    public void setEffort(double effort) { this.effort = effort; }
    public double getEffort() { return this.effort; }

    ...
}

Answer 1

是否有使用新的 Java 8 循环修改局部变量的“最佳实践”策略？

是的：不要。你可以修改它们的属性——尽管这仍然是个坏主意——但你不能自己修改它们；如果变量是 final 或可能是 final，则只能从 lambda 内部引用变量。 （AtomicDouble 确实是一种解决方案，另一种是 double[1]，它只是作为一个持有者。）

这里实现“平均”操作的正确方法是

activities.stream()
    .mapToDouble(Activity::getEffort)
    .filter(effort -> effort != Activity.NULL)
    .average()
    .getAsDouble();

Answer 2

在您的情况下，有一个功能更强大的解决方案 - 只需从流中计算汇总统计信息，您可以从中获取过滤的元素数量及其总和：

DoubleSummaryStatistics stats = 
    activities.stream()
              .mapToDouble(Activity::getEffort)
              .filter(e -> e != Activity.NULL)
              .summaryStatistics();

long effortCount = stats.getCount();
double effortSum = stats.getSum();

是否有修改局部变量的“最佳实践”策略 使用新的 Java 8 循环？

不要尝试这样做。我认为主要问题是人们试图以一种命令式的方式使用新的 Java 8 特性来翻译他们的代码（就像在你的问题中一样——然后你就有麻烦了！）。

首先尝试看看您是否可以提供一个实用的解决方案（我相信这是 Stream API 的目标）。