Łańcuch operacji „instanceof” jest uważany za „zapach kodu”. Standardowa odpowiedź brzmi „użyj polimorfizmu”. Jak bym to zrobił w tym przypadku?
Istnieje wiele podklas klasy bazowej; żaden z nich nie jest pod moją kontrolą. Analogiczna sytuacja miałaby miejsce w przypadku klas Java Integer, Double, BigDecimal itp.
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);}
Mam kontrolę nad NumberStuff i tak dalej.
Nie chcę używać wielu linii kodu, w których wystarczyłoby kilka linii. (Czasami tworzę HashMap mapujący Integer.class na instancję IntegerStuff, BigDecimal.class na instancję BigDecimalStuff itp. Ale dzisiaj chcę czegoś prostszego.)
Chciałbym coś tak prostego jak to:
public static handle(Integer num) { ... }
public static handle(BigDecimal num) { ... }
Ale Java po prostu nie działa w ten sposób.
Chciałbym używać statycznych metod podczas formatowania. Rzeczy, które formatuję, są złożone, gdzie Thing1 może zawierać tablicę Thing2s, a Thing2 może zawierać tablicę Thing1s. Miałem problem, kiedy zaimplementowałem moje elementy formatujące w ten sposób:
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);
}
}
Tak, znam HashMap i trochę więcej kodu też może to naprawić. Ale „instanceof” wydaje się tak czytelny i łatwy do utrzymania w porównaniu. Czy jest coś prostego, ale nie śmierdzącego?
Uwaga dodana 10.05.2010:
Okazuje się, że prawdopodobnie w przyszłości zostaną dodane nowe podklasy, a mój istniejący kod będzie musiał z wdziękiem sobie z nimi radzić. HashMap on Class nie zadziała w takim przypadku, ponieważ Class nie zostanie znaleziona. Łańcuch instrukcji if, zaczynający się od najbardziej szczegółowego i kończący się na najbardziej ogólnym, jest prawdopodobnie najlepszy:
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
}