Collections.sort не будет работать с пользовательским компаратором?

Question

import java.util.*;

public class ABC {
    public static void main(String[] args) {
        List<Integer> values = null;

        values = new ArrayList<Integer>();
        values.add(5);
        values.add(9);
        values.add(3);
        values.add(55);
        values.add(4);

        Collections.sort(values);
        System.out.println(values);



        values = new ArrayList<Integer>();
        values.add(5);
        values.add(9);
        values.add(3);
        values.add(55);
        values.add(4);

        Comparator<Integer> cmp = new Comparator<Integer>() {
                    @Override
                    public int compare(Integer o1, Integer o2) {
                        int o1i = o1;
                        int o2i = o2;
                        return o1i - o1i;
                    }
                };

        Collections.sort(values, cmp);
        System.out.println(values);
    }
}

Это печатает:

[3, 4, 5, 9, 55]
[5, 9, 3, 55, 4]

что явно не является ожидаемым результатом. Чего мне не хватает?

Answer 1

У вас есть ошибка:

Изменить

return o1i - o1i;

на

return o1i - o2i;

Answer 2

Ваш компаратор вычитает o1i - o1i, каждый раз давая 0.

(Вы ничего не получаете, присваивая o1 и o2 локальным int переменным; просто вычитайте o1 - o2.)

Answer 3

У вас ошибка ввода, у компаратора должно быть return o1i - o2i;

, а не return o1i - o1i;

Answer 4

используйте это

int o1i = o1;
                    int o2i = o2;
                    return o1i - o2i;

Это дает следующий результат

[3, 4, 5, 9, 55]

[3, 4, 5, 9, 55]

Answer 5

return o1i - o1i;

Полагаю, вы имели в виду;

return o1i - o2i;