我有一个关于有效使用java comparator的问题。
class MyClass {
//Active State: OPEN, PENDING, RUNNING
private String state;
private Date startDate;
private Date endDate;
}
这里是 state
字段包括open、pending、running、closed、cancelled等,其中open、pending和running是活动状态。现在我想写一个比较器来排序 List<MyClass>
,以便活动的列在第一位,并按 startDate
其次是非活动的,根据 endDate
.
static final Set<String> ACTIVE;// this set contains OPEN, PENDING, RUNNING
List<MyClass> myList;//This is my list
...
Collections.sort(myList, new Comparator<MyClass>() {
@Override
public int compare(MyClass o1, MyClass o2) {
int c;
boolean isO2 = ACTIVE.contains(o2.getState());
boolean isO1 = ACTIVE.contains(o1.getState());
if (isO2 && isO1) {
c = DateTimeComparator.getInstance().compare(o2.getStartDate(), o1.getStartDate());
} else if (isO2) {
c = 1;
} else if (isO1) {
c = -1;
} else {
c = DateTimeComparator.getInstance().compare(o2.getEndDate(), o1.getEndDate());
}
return c;
}
});
我的问题是我上面实现的有单比较器好吗?还是有更好的方法?很可能我必须坚持使用Java7,但也欢迎使用Java8的解决方案。
1条答案
按热度按时间pgx2nnw81#
在Java8中,我认为使用comparator::comparing会更干净一些。例如:
在Java7中,假设类路径上有Guava,可以使用排序。例如: