我该如何重写:

private Comparator<Domain> byRank;

...

byRank = new Comparator<Domain>() {
  @Override
  public int compare(Domain d1, Domain d2) {
    float tmp1 = d1.getDomainRank() == null ? 0 : d1.getDomainRank();
    float tmp2 = d2.getDomainRank() == null ? 0 : d2.getDomainRank();
    return Float.compare(tmp1, tmp2);
  }
};


进入lambda?

根据check null value before sorting using lambda expression,我尝试了以下方法:

byRank = Comparator.nullsFirst(Comparator.comparing(Domain::getDomainRank));


但是,它失败并显示:

java.lang.NullPointerException: null
    at java.util.Comparator.lambda$comparing$77a9974f$1(Comparator.java:469)
    at java.util.Comparators$NullComparator.compare(Comparators.java:83)
    at java.util.PriorityQueue.siftUpUsingComparator(PriorityQueue.java:669)
    at java.util.PriorityQueue.siftUp(PriorityQueue.java:645)
    at java.util.PriorityQueue.offer(PriorityQueue.java:344)
    at java.util.PriorityQueue.add(PriorityQueue.java:321)


编辑:lambda失败,即使我在比较之前检查比较对象是否为null:

Queue<Domain> topByRank = new PriorityQueue<>(TOP, byRank);

...

for (Domain domain : domains) {
  if (domain == null) { // check here
    continue;
  }
  topByRank.add(domain); // here it fails
}

最佳答案

您的代码会将空Domain放在首位。如果要检查空排名,则需要使用以下命令:

Comparator.comparing(Domain::getDomainRank, Comparator.nullsFirst(Comparator.naturalOrder()))


但是请记住,如果等级不能小于0,则这仅等效于原始比较器。否则,您必须测试类似的表达式:

Comparator.comparing(d -> d.getDomainRank() == null ? 0 : d.getDomainRank())


另外,您可能打算在原始代码中使用Float.MIN_VALUE而不是0。

10-06 10:11
查看更多