我该如何重写:
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。