Java пользовательский компаратор не работает

Question

Эй, я хочу, чтобы сравнить свои собственные объекты, так что я могу сортировать их в моей ArrayList, мой объект имеет 3 поля, один из них являются INT моего компаратора код выглядит

public class ImageComparator implements Comparator<Obj> { 
    @Override 
    public int compare(Obj 1, Obj 2) { 

     return 1.Id > 2.Id ? 1 : 0; 
    } 
} 

Collections.sort(myList, new ImageComparator()); 

Что я делаю неправильно?

Answer 1

Вы пропустили -1 значение,

попробовать вместо условия стандартного метода, например:

public class ImageComparator implements Comparator<Obj> { 
    @Override 
    public int compare(Obj o1, Obj o2) { 
     return Integer.compare(o1.Id, o2.Id); 
    } 
} 

Answer 2

Попробуйте это. Здесь необходимо проверить 3 случая (<, >, =).

public class ImageComparator implements Comparator<Obj> { 
    @Override 
    public int compare(Obj o1, Obj o2) { 

     if (o1.Id > o2.Id) return 1; 
     else if (o1.Id < o2.Id) return -1; 
     else return 0; 

    } 
} 

Collections.sort(myList, new ImageComparator()); 

Answer 3

Вы не можете использовать цифру в качестве первой буквы в variable/method/class names. для более переменных правил присвоения имен Click Here

public class ImageComparator implements Comparator<Obj> { 
    @Override 
    public int compare(obj obj1, obj obj2) { 
     return ((obj)obj1).Id - ((obj)obj2).Id; 
    } 
} 

Answer 4

compare метод компаратор возвращает 1, -1, или 0, если объект 1 больше, меньше или равно объекта 2. Таким образом, вы должны проверить эти 3 и вернуть правильное значение для идентификатора вашего объекта.

public int compare(Obj ob1, Obj obj2){ 
    if(obj1.Id > obj2.Id) 
     return 1; 
    if(obj1.Id < obj2.Id) 
     return -1; 
    return 0; 
} 

Answer 5

Сначала вы не можете использовать числа в качестве имен переменных. Затем взгляните на javadocs: return «отрицательное целое число, ноль или положительное целое число, поскольку первый аргумент меньше, равен или больше второго».

Answer 6

Вы можете попробовать:

return v1.Id < v2.Id ? -1 : v1.Id > v2.Id ? 1 : 0;