且构网

分享程序员开发的那些事...
且构网 - 分享程序员编程开发的那些事

为什么Collections.max()不返回String的集合的实际最大值?

更新时间:2023-02-04 09:00:50

由于您的元素是字符串, Collections.max()返回值最大的值按字典顺序



如果您希望要以数字比较字符串,您需要使用双参数版本的 Collections.max() 并提供适当的比较器:

  ArrayList< String> dirNo = new ArrayList< String>(); 

dirNo.add(1);
dirNo.add(2);
dirNo.add(3);
dirNo.add(4);
dirNo.add(5);
dirNo.add(6);
dirNo.add(7);
dirNo.add(8);
dirNo.add(9);
dirNo.add(10);
dirNo.add(11);

比较器< String> cmp = new Comparator< String>(){
@Override
public int compare(String o1,String o2){
return Integer.valueOf(o1).compareTo(Integer.valueOf ));
}
};
System.out.println(max:+ Collections.max(dirNo,cmp));


ArrayList<String> dirNo = new ArrayList<String>();

dirNo.add("1");
dirNo.add("2");
dirNo.add("3");
dirNo.add("4");
dirNo.add("5");
dirNo.add("6");
dirNo.add("7");
dirNo.add("8");
dirNo.add("9");
dirNo.add("10");
dirNo.add("11");

System.out.println("max : " + Integer.parseInt(Collections.max(dirNo)));

After executing above code, print 9 as output.

But actually max value should be 11.

Why am I getting 9 as max ?

Since your elements are strings, Collections.max() is returning the value that's the largest lexicographically.

If you wish to compare the strings numerically, you need to use the two-argument version of Collections.max() and supply an appropriate comparator:

    ArrayList<String> dirNo = new ArrayList<String>();

    dirNo.add("1");
    dirNo.add("2");
    dirNo.add("3");
    dirNo.add("4");
    dirNo.add("5");
    dirNo.add("6");
    dirNo.add("7");
    dirNo.add("8");
    dirNo.add("9");
    dirNo.add("10");
    dirNo.add("11");

    Comparator<String> cmp = new Comparator<String>() {
        @Override
        public int compare(String o1, String o2) {
            return Integer.valueOf(o1).compareTo(Integer.valueOf(o2));
        }
    };
    System.out.println("max : " + Collections.max(dirNo, cmp));