Использование интерфейса Comparable для примитивных типов данных

Question

Извините, если это глупый вопрос, я относительно новичок в программировании.

В основном у меня был вопрос о домашнем задании, чтобы создать сценарий почтового отделения, который поддерживает два типа предметов: предметы, отправленные по воздуху, и предметы, отправленные морем. для каждого типа я использовал метод calcFee для расчета общих затрат на доставку.

Вторая часть домашней работы требует расширения класса Item, чтобы он реализовал интерфейс Comparable, где элементы сортируются по их значению calcFee. Теперь я понимаю, что примитивные типы данных не могут быть отсортированы из интерфейса Comparable. Поскольку вопрос конкретно просит использовать интерфейс Comparable, я делаю что-то неправильно или просто неправильно создаю программу

Любая помощь будет принята с благодарностью.

package homework3; 
    import java.util.ArrayList; 
    import java.util.Collections; 
/** 
* 
* @author Josh 
*/ 
public class Homework3 { 

    /** 
    * @param args the command line arguments 
    */ 
    public static void main(String[] args) { 
      ArrayList<Item> items = new ArrayList(); 
      items.add(new airItems(11, "josh", "sam", 295)); 
      items.add(new airItems(11, "zosh", "sam", 295)); 
      items.add(new seaItems(11, "aosh", "sam", 11, 12, 15)); 

      Collections.sort(items); 

      for (Item i : items){ 
       i.calcFee(); 
       i.print(); 

      } 
    } 

} 

class Item implements Comparable<Item>{ 

    int id; 
    String from; 
    String to; 
    double fee; 

    public Item(int id, String from, String to){ 
     this.id = id; 
     this.from = from; 
     this.to = to; 


    } 

    public void print(){ 
     System.out.println("id: " + id + "\tFrom: " + from + "\tTo: " + to + "\tFee: " + fee); 
    } 

    public double calcFee(){ 
     return 0; 
    } 


    @Override 
    public int compareTo (Item item1){ 
     return this.fee.compareTo(item1.calcFee()); 
    } 
} 

class airItems extends Item{ 
    long weight; 

    public airItems(int id, String from, String to, int weight){ 
     super(id, from, to); 
     this.weight = weight; 
    } 

    public void print(){ 
     super.print(); 
     System.out.println("Weight: " + weight); 
     System.out.println(""); 

    } 

    @Override 
    public double calcFee(){ 
     if (weight < 100){ 
      fee = 2.5; 
     } 
     else if (weight < 200) { 
      fee = 5; 
     } 
     else if (weight < 300) { 
      fee = 7.5; 
     } 
    return fee;  
    } 

} 


class seaItems extends Item{ 
    int length; 
    int width; 
    int depth; 

    public seaItems(int id, String from, String to, int length, int width, int depth){ 
     super(id, from, to); 
     this.length = length; 
     this.width = width; 
     this.depth = depth; 
    } 

    @Override 
    public double calcFee(){ 
     if (length <= 10 && width <= 10 && depth <= 10){ 
      fee = 5; 
     } 
     else if (length <= 20 && width <= 20 && depth <= 20){ 
      fee = 10; 
     } 
     if (length <= 50 && width <= 50 && depth <= 50){ 
      fee = 5; 
     } 
     else if (length > 50 && width > 50 && depth > 50){ 
      fee = 0; 
     } 
     return fee; 
    } 

    @Override 
    public void print(){ 
     super.print(); 
     System.out.println("Length: " + length + "\tWidth:" + width + "\tdepth" + depth); 
     System.out.println(""); 
    } 



} 

Answer 1

Примитивные типы данных могут быть использованы совместно с Comparable интерфейсом, насколько я обеспокоен, но это не рекомендуется, поскольку это слишком сложно и ненужно, часто требуя некоторые писаки быть сделаны (хотя я не совсем уверен, что так не верьте мне на слово).

Что касается вашей задачи, самый простой способ сделать это было бы создать новую List<Item> и сортировать его, используя с Comparable интерфейсом, с помощью .stream:

 List<Item> collect = items.stream() 
      .sorted(Comparator.comparingDouble((Item it) -> it.calcFee()).reversed()) 
      .collect(toList()); 
    collect.forEach(it-> System.out.println(it)); 

Я использовал лямбда-выражение здесь, но контрольный элемент также является приемлемым: (Item::calcFee)

Теперь, это не будет работать с вашим @Override из print функции, вы должны переопределить toString как функция требует строки для возврата. Это может быть достигнуто, делая это:

@Override 
    public String toString() { 
     return "id: " + id + "\tFrom: " + from + "\tTo: " + to + "\tFee: " + fee; 
    } 

Результат таков:

id: 11 From: josh To: sam Fee: 7.5 
id: 11 From: josh To: sam Fee: 5.0 

Answer 2

Используйте compare() метод Double:

@Override 
public int compareTo (Item item){ 
    return Double.compare(calcFee(), item.calcFee()); 
} 

Лучше использовать calcFee(), а не fee в случае fee не был инициализирован. Если у вас нет элементов, это не сильно изменит производительность.