`
lc87624
  • 浏览: 142788 次
  • 性别: Icon_minigender_1
  • 来自: 北京
社区版块
存档分类
最新评论

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

    博客分类:
  • java
阅读更多
有一个需求是要对拼接出来的飞行线路列表进行排序,自然地用到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;
        ...
}

问题解决。
分享到:
评论

相关推荐

Global site tag (gtag.js) - Google Analytics