有一个需求是要对拼接出来的飞行线路列表进行排序,自然地用到Collections.sort方法,然后实现自己的Comparator。
排序需要考虑多种因素,包括拼接出的线路的段数、线路经过的国家数等,其中有一个排序条件是:当线路的飞行时长大于待比较线路飞行时长的1.5倍时,忽略该线路在段数和国家数上的优势。
代码实现如下:
貌似看起来没有什么问题,但偶然发现,有些排序不符合预期,细想后发现,以上排序规则其实是违背对称性的。
这个条件在consumeTime1 > consumeTime2的情况下是符合预期的,但当把比较的两个对象掉转过来,排序规则却不是对称的,即可能出现o1 > o2,o2 !<(不小于) o1的情况。
修改代码如下:
问题解决。
排序需要考虑多种因素,包括拼接出的线路的段数、线路经过的国家数等,其中有一个排序条件是:当线路的飞行时长大于待比较线路飞行时长的1.5倍时,忽略该线路在段数和国家数上的优势。
代码实现如下:
Collections.sort(results, new Comparator<Map<String, Object>>() {
@Override
public int compare(Map<String, Object> o1, Map<String, Object> o2) {
...
//若线路的飞行时长大于待比较线路飞行时长的1.5倍,则不再比较其他排序因素
if(consumeTime1 > 1.5 * consumeTime2){
return 1;
}
//段数越少,排序越靠前
if(sectionNum1 > sectionNum2) return 1;
if(sectionNum1 < sectionNum2) return -1;
...
}
貌似看起来没有什么问题,但偶然发现,有些排序不符合预期,细想后发现,以上排序规则其实是违背对称性的。
if(consumeTime1 > 1.5 * consumeTime2)
这个条件在consumeTime1 > consumeTime2的情况下是符合预期的,但当把比较的两个对象掉转过来,排序规则却不是对称的,即可能出现o1 > o2,o2 !<(不小于) o1的情况。
修改代码如下:
Collections.sort(results, new Comparator<Map<String, Object>>() {
@Override
public int compare(Map<String, Object> o1, Map<String, Object> o2) {
...
//若线路的飞行时长小于待比较线路,并且乘以1.5倍后仍小于,则不再比较其他排序因素
if(consumeTime1 < consumeTime2 && 1.5 * consumeTime1 < consumeTime2){
return -1;
}
//若线路的飞行时长大于待比较线路,且超过其飞行时长的1.5倍,则不再比较其他排序因素
if(consumeTime1 > consumeTime2 && consumeTime1 > 1.5 * consumeTime2){
return 1;
}
//段数越少,排序越靠前
if(sectionNum1 > sectionNum2) return 1;
if(sectionNum1 < sectionNum2) return -1;
...
}
问题解决。