2015-04-05 71 views
0

我有一个填充DMatch对象的列表,此对象包含属性“distance”。比较器类显示错误排序

我想要做的是根据“距离”递增或desceningly那种名单。

在下面的代码

我用比较器类和sorte D出现列表desceingly,但指数(8,9,10,11)的输出,他不排序良好。

请让我知道我在做什么错误,以及如何解决这个错误

代码

Collections.sort(list_RawMatchesMatDMatch, ascOrder); 
       for (int i = 0; i < list_RawMatchesMatDMatch.size(); i++) { 
        Log.D(TAG, "main", "rawMatDMatchLiat(" + i + ").distance: " + list_RawMatchesMatDMatch.get(i).distance);       
       } 

    static Comparator<DMatch> ascOrder = new Comparator<DMatch>() { 

    public int compare(DMatch arg0, DMatch arg1) { 
     // TODO Auto-generated method stub 

     return (int) (arg1.distance - arg0.distance); 

    } 
}; 

输出

Debug: MainClass -> main: rawMatDMatchLiat(0).distance: 454.26645 
Debug: MainClass -> main: rawMatDMatchLiat(1).distance: 447.3835 
Debug: MainClass -> main: rawMatDMatchLiat(2).distance: 442.294 
Debug: MainClass -> main: rawMatDMatchLiat(3).distance: 438.98862 
Debug: MainClass -> main: rawMatDMatchLiat(4).distance: 433.11084 
Debug: MainClass -> main: rawMatDMatchLiat(5).distance: 427.35114 
Debug: MainClass -> main: rawMatDMatchLiat(6).distance: 426.2335 
Debug: MainClass -> main: rawMatDMatchLiat(7).distance: 425.16232 
Debug: MainClass -> main: rawMatDMatchLiat(8).distance: 423.816 
Debug: MainClass -> main: rawMatDMatchLiat(9).distance: 423.6815 
Debug: MainClass -> main: rawMatDMatchLiat(10).distance: 424.51855 
Debug: MainClass -> main: rawMatDMatchLiat(11).distance: 424.66693 
Debug: MainClass -> main: rawMatDMatchLiat(12).distance: 423.53986 
Debug: MainClass -> main: rawMatDMatchLiat(13).distance: 423.79712 
Debug: MainClass -> main: rawMatDMatchLiat(14).distance: 423.62012 
Debug: MainClass -> main: rawMatDMatchLiat(15).distance: 421.72028 
Debug: MainClass -> main: rawMatDMatchLiat(16).distance: 419.46274 
Debug: MainClass -> main: rawMatDMatchLiat(17).distance: 418.12198 

回答

6

这是没有好:

return (int) (arg1.distance - arg0.distance); 

因为int cast会截断结果。相反,这样做:

if (arg1.distance > arg0.distance) { 
    return 1; 
} else if (arg1.distance < arg0.distance) { 
    return -1; 
} else { 
    return 0; 
} 

或者更好:

return Double.compare(arg1.distance, arg0.distance); 
+0

@Pshemo:是的,记得和纠正它。 – 2015-04-05 11:37:22

+0

但为什么“公共int比较(DMatch arg0,DMatch arg1)”的方法总是“int”我试图让浮动,但这是不可能的 – rmaik 2015-04-05 11:43:26

+0

@rmaik:我不明白你的意见 - 请解释。 – 2015-04-05 11:45:32