在Java中避免使用instanceof

Question

在Java中避免使用instanceof

具有一系列“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映射到一个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”似乎更具可读性和可维护性。有什么简单但不臭吗？

注释已添加5/10/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

}

Answer 1

您可能对Steve Yegge的亚马逊博客中的这篇文章感兴趣：“当多态失败时”。基本上他正在解决这样的情况，当多态性导致比它解决的更多麻烦时。问题是，要使用多态，你必须使“处理”逻辑成为每个“切换”类的一部分 - 即在这种情况下为整数等。显然这不切实际。有时甚至在逻辑上它都不是放置代码的正确位置。他建议将'instanceof'方法视为几种邪恶中较小的一种。与您被迫编写臭代码的所有情况一样，请将其保持在一个方法（或最多一个类）中，以便气味不会泄漏。

Answer 2

正如评论中所强调的那样，访客模式将是一个不错的选择。但是如果没有对目标/接受者/被访者的直接控制，则无法实现该模式。这里是访问者模式可能仍然可以在这里使用的一种方式，即使你没有使用包装器直接控制子类（以Integer为例）：public class IntegerWrapper {     private Integer integer;     public IntegerWrapper(Integer anInteger){         integer = anInteger;     }     //Access the integer directly such as     public Integer getInteger() { return integer; }     //or method passthrough...     public int intValue() { return integer.intValue(); }     //then implement your visitor:     public void accept(NumericVisitor visitor) {         visitor.visit(this);     }}当然，包装最终类可能被认为是它自己的气味，但也许它很适合你的子类。就我个人而言，我认为instanceof这里的味道不是很糟糕，特别是如果只限于一种方法，我会很乐意使用它（可能超过我自己的建议）。正如你所说，它具有可读性，类型安全性和可维护性。一如既往，保持简单。