Сравнение Java-подклассов

Question

Мое приложение может сравнивать экземпляры любых двух подклассов одного и того же абстрактного родительского класса. Я хочу, чтобы они сравнивались следующим образом:

1. Если они представляют собой разные подклассы, родительский класс должен выполнить сравнение.
2. Если они являются одним и тем же подклассом, подкласс должен выполнить сравнение.

Классы будут сравниваться с TreeMap, поэтому у меня есть выбор использования компаратора или реализация Comparable (или обоих?).

Я могу придумать несколько способов сделать это, но все они немного грязны и подвержены ошибкам. Есть ли элегантное решение?

Заранее спасибо ...

Answer 1

Вы могли бы попробовать

// Parent: 
@Override 
public final int compareTo(Parent other) 
{ 
    if (getClass() == other.getClasss()) { 
    // same type -> pass it to subclass implementation 
    return this.subCompare(other) 
    } 

    // different type -> do the comparison here based on Parent's logic 
    // ... 
} 

protected int subCompare(Parent other) 
{ 
    // this should not be called directly 
    return 0; // could throw an exception here too 
} 

// Derived1: 
@Override 
protected int subCompare(Parent other) 
{ 
    // this method is only called from Parent 
    Derived1 other1 = (Derived1) other; 
    // do the comparison based on Derived1's logic 
} 

Аналогично для других производных классов

Answer 2

Это не является прямым ответом на ваш вопрос, но:
Я считаю, что вы делаете, подвержены ошибкам и проблематичны.
Тест на равенство, налагаемый методом compareTo, должен, как правило, возвращать те же результаты, что и метод equals. То есть порядок, наложенный методом compareTo, составляет , если будет соответствовать equals.
Если этот контракт нарушен, вы можете столкнуться с проблемами с сортировкой коллекции (которую вы используете).
Почему я говорю это:
Тот факт, что вы хотите делегировать сравнение с родительским классом в определенных случаях, указывает мне, что вы добавили компонент значения в подкласс.
Если вы сделали это, то нет никакого способа, возможности сохранения equals контракта и вы могли бы работать в задачах с отсортированным контейнером (см.выше)

Answer 3

Это собирается быть ошибки так или иначе. Вы могли бы сделать что-то подобное в вашем подклассе:

класса Subclass1 ... {

public boolean equals(Object o) { 
    if(o instanceof Subclass1) { 
    return super.equals(o); 
    } 

    ... compare subclasses 
} 

}

Answer 4

Если иерархия классов будет удлинению или нет?
=> Если он не расширяется (или редко будет расширен), вы можете реализовать Comperator, чтобы весь код сравнения находился в одном месте.

Есть ли несколько способов заказать объекты? => Если да, то вам придется использовать компилятор для каждого заказа.

compareTo(other) (или compare(o1, o2)) вызывается из трех ситуаций:

1. this instanceof other.class (т.е. o1 instanceof o2.class):
   • это.класс должен сравнивать this и o2, так как он имеет больше информации, чем other.class;
2. other instanceof this.class (т.е. o2 instanceof o1.class):
   • other.class должны сравнить other и this и this.class.compareTo должны перевернуть результат и вернуть его.
3. существует первый общего предка P таким образом, что this instanceof P & & other instanceof P:
   • Р следует сравнить this и other: вызов super.compareTo (другой) рекурсивно и возвращает результат.

Answer 5

Это, как я делаю это сейчас. Я думаю, что это так элегантно, как он собирается получить:

public abstract class ParentClass implements Comparable<ParentClass> { 
    // compareTo(ParentClass) is implicitly abstract 
} 


public class SubClass1 extends ParentClass /* Comparable<> is implicitly implemented */ { 
    @Override 
    public int compareTo(ParentClass another) { 
     SubClass1 subAnother = (SubClass1) another; 
     return /* result of sub-class comparison */; 
    } 
} 


public class MyComparator implements Comparator<ParentClass> { 
    @Override 
    public int compare(ParentClass lhs, ParentClass rhs) { 

     // Are lhs and rhs instances of the same sub-class? 
     if(!lhs.getClass().equals(rhs.getClass())) 
      // They are different. Use parent class comparison. 
      return /* result of parent class comparison */; 

     // They are the same. Use sub-class comparison. 
     return lhs.compareTo(rhs); 
    } 
} 

Тогда я просто передать экземпляр MyComparator к TreeMap. Он обрабатывает сравнение между подклассами или передает сравнение с подклассом, если они одинаковы.

Что мне не нравится, так это то, что SubClass1.compareTo() будет бросать ClassCastException, если another является экземпляром другого подкласса. Однако мне не о чем беспокоиться, если я убеждаюсь, что подклассы сравниваются только по MyComparator.