「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 などを制御できます。
数行で済むところに多くのコード行を使用したくありません。(時々、Integer.class を IntegerStuff のインスタンスに、BigDecimal.class を BigDecimalStuff のインスタンスにマッピングする HashMap を作成します。しかし、今日はもっと単純なものが必要です。)
私はこのような単純なものが欲しいです:
public static handle(Integer num) { ... }
public static handle(BigDecimal num) { ... }
しかし、Java はそのようには機能しません。
書式設定時に静的メソッドを使用したいと思います。私が書式設定しているものは複合であり、Thing1 には配列 Thing2 を含めることができ、Thing2 には Thing1 の配列を含めることができます。次のようにフォーマッターを実装したときに問題が発生しました。
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」は比較すると非常に読みやすく、保守しやすいようです。シンプルだけど臭くないものってありますか？
2010 年 5 月 10 日に追加された注:
新しいサブクラスが将来追加される可能性が高く、既存のコードでそれらを適切に処理する必要があることがわかりました。その場合、クラスが見つからないため、クラスの 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
}