Наличие цепочки операций instanceof считается «запахом кода». Стандартный ответ - «использовать полиморфизм». Как бы я это сделал в таком случае?
Есть несколько подклассов базового класса; ни один из них не находится под моим контролем. Аналогичная ситуация была бы с классами Java Integer, Double, BigDecimal и т. Д.
if (obj instanceof Integer) {NumberStuff.handle((Integer)obj);}
else if (obj instanceof BigDecimal) {BigDecimalStuff.handle((BigDecimal)obj);}
else if (obj instanceof Double) {DoubleStuff.handle((Double)obj);}
Я контролирую NumberStuff и так далее.
Я не хочу использовать много строк кода вместо нескольких строк. (Иногда я делаю HashMap, отображающий Integer.class в экземпляр IntegerStuff, BigDecimal.class в экземпляр BigDecimalStuff и т. Д. Но сегодня я хочу что-то попроще.)
Я бы хотел что-нибудь простое:
public static handle(Integer num) { ... }
public static handle(BigDecimal num) { ... }
Но Java так не работает.
При форматировании я хочу использовать статические методы. Вещи, которые я форматирую, являются составными, где Thing1 может содержать массив Thing2s, а Thing2 может содержать массив Thing1s. У меня возникла проблема, когда я реализовал свои средства форматирования следующим образом:
class Thing1Formatter {
private static Thing2Formatter thing2Formatter = new Thing2Formatter();
public format(Thing thing) {
thing2Formatter.format(thing.innerThing2);
}
}
class Thing2Formatter {
private static Thing1Formatter thing1Formatter = new Thing1Formatter();
public format(Thing2 thing) {
thing1Formatter.format(thing.innerThing1);
}
}
Да, я знаю HashMap, и еще немного кода может это исправить. Но "instanceof" кажется таким удобочитаемым и удобным в сравнении. Есть что-нибудь простое, но не вонючее?
Примечание, добавленное 10.05.2010:
Оказывается, новые подклассы, вероятно, будут добавлены в будущем, и мой существующий код должен будет корректно их обрабатывать. В этом случае HashMap для класса не будет работать, потому что класс не будет найден. Цепочка операторов if, начиная с самых конкретных и заканчивая наиболее общими, в конце концов, вероятно, лучшая:
if (obj instanceof SubClass1) {
// Handle all the methods and properties of SubClass1
} else if (obj instanceof SubClass2) {
// Handle all the methods and properties of SubClass2
} else if (obj instanceof Interface3) {
// Unknown class but it implements Interface3
// so handle those methods and properties
} else if (obj instanceof Interface4) {
// likewise. May want to also handle case of
// object that implements both interfaces.
} else {
// New (unknown) subclass; do what I can with the base class
}