列表仅含Null值时使用Comparator触发空指针异常的解决咨询

你的代码抛出NullPointerException的核心原因是：当列表全为null时，虽然nullsLast会处理单null的比较场景，但自定义比较器c本身没有处理null参数的逻辑——如果在比较流程中不小心把null传入了c，调用d1.isAfter(d2)这类方法时就会触发空指针。另外，当max返回包含null的Optional后，直接调用get()获取值，后续若对这个null进行操作也会引发NPE。

以下是几种可行的解决方式：

• 用内置比较器替代自定义实现
  LocalDate本身实现了Comparable接口，直接用Comparator.naturalOrder()就能完成正确的日期比较，配合nullsLast可以完美处理null场景，无需自己写比较逻辑：

  List<LocalDate> list1 = new ArrayList<>();
  list1.add(null);
  list1.add(null);
  Optional<LocalDate> lastUsedDate = list1.stream()
          .max(Comparator.nullsLast(Comparator.naturalOrder()));
  // 避免直接调用get()，先判断是否有值
  lastUsedDate.ifPresent(System.out::println);
  // 或者指定默认值
  LocalDate result = lastUsedDate.orElse(null);
  

• 提前过滤null元素
  如果业务逻辑不需要考虑null元素，直接过滤掉再求最大值，这样当列表全为null时，max会返回空的Optional，避免后续处理null值：

  Optional<LocalDate> lastUsedDate = list1.stream()
          .filter(Objects::nonNull)
          .max(Comparator.naturalOrder());
  // 处理空Optional的情况
  if (lastUsedDate.isEmpty()) {
      System.out.println("没有有效日期");
  } else {
      System.out.println(lastUsedDate.get());
  }
  

• 修复自定义比较器的null处理逻辑
  如果一定要用自定义比较器，记得在比较逻辑里先判断null（双重保险更稳妥）：

  Comparator<LocalDate> c = (d1, d2) -> {
      if (d1 == null && d2 == null) return 0;
      if (d1 == null) return -1;
      if (d2 == null) return 1;
      return d1.isAfter(d2) ? 1 : (d1.isBefore(d2) ? -1 : 0);
  };
  Optional<LocalDate> lastUsedDate = list1.stream()
          .max(Comparator.nullsLast(c));
  

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