Ява не проверено литой

Question

У меня есть класс компаратора на Java для сравнения записей на карте:

public class ScoreComp implements Comparator<Object> {

    public int compare(Object o1, Object o2) {

        Entry<Integer, Double> m1 = null;
        Entry<Integer, Double> m2 = null;

        try {
            m1 = (Map.Entry<Integer, Double>)o1;
            m2 = (Map.Entry<Integer, Double>)o2;
        } catch (ClassCastException ex){
            ex.printStackTrace();
        }

        Double x = m1.getValue();
        Double y = m2.getValue();
        if (x < y)
            return -1;
        else if (x == y)
            return 0;
        else
            return 1;        
     }

}

когда я компилирую эту программу, я получаю следующее:

warning: [unchecked] unchecked cast
found   : java.lang.Object
required: java.util.Map.Entry<java.lang.Integer,java.lang.Double>
            m1 = (Map.Entry<Integer, Double>)o1;

Мне нужно отсортировать записи карты на основе двойных значений.

Если я создаю следующий компаратор, я получаю сообщение об ошибке при вызове функции сортировки массивов (я получаю набор записей с карты и затем использую набор в качестве массива).

public class ScoreComp implements Comparator<Map.Entry<Integer, Double>>

как реализовать этот сценарий.

Answer 1

Если вы используете этот компаратор для сортировки TreeMap, то это не сработает. TreeMap компараторы предназначены для сравнения только ключей карты, а не записей key-> value. Если вашему компаратору нужен доступ к значениям, он должен будет найти их на самой карте, например,

final Map<Integer, Double> map = ....

public class ScoreComp implements Comparator<Integer>  {
   public int compare(Integer key1, Integer key2) {
    Double x = map.getValue();
    Double y = map.getValue();
    if (x < y)
        return -1;
    else if (x == y)
        return 0;
    else
        return 1; 
   }
}

---

edit: Из ваших комментариев, я думаю, что ваш лучший вариант - создать класс, который инкапсулирует идентификатор и значение, поместить эти значения в список и отсортировать их.

public class Item implements Comparable<Item> {
   int id;
   double value;

   public int compareTo(Item other) {
      return this.value - other.value;
   }
}

, а затем

List<Item> list = new ArrayList<Item>();
// ... add items here
Collections.sort(list);

Поскольку Item сам по себе Comparable, вам не нужен внешний Comparator (если только он вам не нужен).

Answer 2

Укладчик

описал, как исправить код, который вы показали.Вот как исправить код в вашем комментарии: во-первых, не используйте массивы, потому что массивы не работают с универсальными типами (вы не можете иметь массив универсального типа)Вместо этого вы можете использовать List и Collections.sort() метод:

    List<Map.Entry<Integer, Double>> mList = 
        new ArrayList<Map.Entry<Integer, Double>>(Score.entrySet()); 
    Collections.sort(mList, new ScoreComp());

Answer 3

Как насчет переписать как

public class ScoreComp implements Comparator<Map.Entry<Integer, Double>> {

    public int compare(Map.Entry<Integer, Double> o1, Map.Entry<Integer, Double> o2) {
        if ( o1.getValue()  < o2.getValue()  ) return -1;
        else if ( o1.getValue() == o2.getValue()  ) return 0;
        return 1;
    }
}