使用Stream.reduce计算Float集合元素Math.log之和得到NaN，求问题原因

你的代码逻辑完全偏离了需求，这才导致出现NaN结果，具体问题和修正方法如下：

错误原因 #

你对reduce的用法逻辑完全错误：

• reduce的第一个参数是累加初始值，第二个参数里的a是前一次的累加结果，b是当前遍历到的集合元素
• 你的代码每次都对累加结果a再调用Math.log，这相当于把「之前所有元素log的和」再取一次log，完全不是你要的「每个元素先log再求和」的逻辑
• 一旦累加结果a变成负数（比如多个负的log值相加），Math.log(a)就会返回NaN，后续所有运算都会继承这个NaN

正确实现方式 #

方式一：用map+sum（最简洁直观） #

先把Float转成double，对每个元素调用Math.log，最后直接求和：

var doubleValue = floatArrayList.stream()
        .mapToDouble(Float::doubleValue)
        .map(Math::log)
        .sum();

方式二：正确使用reduce #

如果一定要用reduce，初始值要设为0.0（求和的初始值），累加器只对当前元素b取log，加到之前的结果a上：

var doubleValue = floatArrayList.stream()
        .reduce(0.0, (a, b) -> a + Math.log(b), Double::sum);

额外注意 #

如果集合里有≤0的元素，Math.log会返回负无穷或NaN，最终结果也会异常，你可能需要提前过滤这类元素或者做特殊处理。

内容的提问来源于stack exchange，提问作者Mattias Törnqvist