使用Java Collections sort方法遇到的一个小问题

有一个需求是要对拼接出来的飞行线路列表进行排序,自然地用到Collections.sort方法,然后实现自己的Comparator。
排序需要考虑多种因素,包括拼接出的线路的段数、线路经过的国家数等,其中有一个排序条件是:当线路的飞行时长大于待比较线路飞行时长的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;
...
}

问题解决。
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值