如何将温度计算Java代码正确转换为Java 8实现并解决编译错误

你写的待修正版本存在4个明显问题，是编译失败的直接原因：

• Lambda表达式中使用的局部变量temp、count必须为final或等效final，不能在lambda内部直接修改这两个变量，这是Java 8 Lambda的基础语法限制
• forEach是Stream的终止操作，返回值为void，后面链式调用.collect(Collectors.toSet())属于语法错误
• 类型不匹配：temps.getValue()返回的是List<Temperatures>类型，不能直接和double类型的temp做加法运算，你没有展开嵌套的集合结构
• 逻辑偏差：你在流中加了按entry key过滤"India"的逻辑，但原有可运行代码中根本没有判断entry的key，只要传入的country参数是India，就会统计所有entry下的温度数据，和你的过滤逻辑不一致

不要用forEach修改外部变量，这是Stream的典型反写法，直接用Stream内置的数值统计能力即可，逻辑和原有代码100%对齐：

import java.util.DoubleSummaryStatistics;

public static double calculateTemps(String country, Map<String, List<Temperatures>> tempMap) {
    // 对齐原有逻辑：国家不匹配时直接返回和旧代码一致的结果
    if (!country.equalsIgnoreCase("India")) {
        System.out.println(0);
        return Double.NaN; // 旧代码此处count=0，计算0/0会得到NaN
    }

    // 一次遍历完成温度总和、数量统计，不需要写嵌套循环
    DoubleSummaryStatistics tempStats = tempMap.values().stream()
            .flatMap(List::stream) // 展开嵌套的List<Temperatures>为单个Temperatures对象流
            .mapToDouble(Temperatures::getTemp) // 提取温度值转为double流
            .summaryStatistics(); // 一次性获取总和、数量、平均值等统计值

    System.out.println(tempStats.getCount());
    return tempStats.getAverage();
}

• 如果你不需要额外打印count，代码可以更简洁，直接调用average()终端操作即可：

  public static double calculateTemps(String country, Map<String, List<Temperatures>> tempMap) {
      if (!country.equalsIgnoreCase("India")) {
          System.out.println(0);
          return Double.NaN;
      }
      return tempMap.values().stream()
              .flatMap(List::stream)
              .mapToDouble(Temperatures::getTemp)
              .average()
              .orElse(Double.NaN);
  }
  

• 不需要纠结map和forEach选哪个：处理嵌套集合结构拍平用flatMap，做字段值提取转换用map，forEach只适合做无状态的遍历输出，不要在里面做累加、计数这类有状态的外部变量修改操作，既不符合Stream的设计初衷，也容易踩语法和并发坑。

内容的提问来源于stack exchange，提问作者nikhil udgirkar