使用“ instanceof”运算符是否被认为是不良设计？

Question

在我的一个项目中，我有两个“数据传输对象” RecordType1和RecordType2，它们从RecordType的抽象类继承。

我希望两个RecordType对象在“
process”方法中由相同的RecordProcessor类处理。我的第一个想法是创建一个通用的处理方法，该方法将两个特定的处理方法委托给以下人员：

public RecordType process(RecordType record){

    if (record instanceof RecordType1)
        return process((RecordType1) record);
    else if (record instanceof RecordType2)
        return process((RecordType2) record);

    throw new IllegalArgumentException(record);
}

public RecordType1 process(RecordType1 record){
    // Specific processing for Record Type 1
}

public RecordType2 process(RecordType2 record){
    // Specific processing for Record Type 2
}

我读过Scott Meyers在 Effective C ++中 编写了以下内容：

“每当您发现自己编写的代码形式为’如果对象的类型为T1，则执行某些操作，但是如果对象的类型为T2，则进行其他操作’，请打自己一下。”

如果他是正确的话，显然我应该打自己的耳光。我真的看不出这是多么糟糕的设计（当然，除非有人将RecordType子类化并添加一个RecordType3，而不在处理它的通用“
Process”方法中添加另一行，从而创建一个NPE），以及我能想到的替代方案其中涉及将特定处理逻辑首当其冲地放在RecordType类本身中，这对我来说真的没有多大意义，因为从理论上讲，我想对这些记录执行许多不同类型的处理。

有人可以解释为什么这可能被认为是不好的设计，并提供某种替代方法，但仍将处理这些记录的责任归于“处理中”类吗？

更新：

• 改变return null以throw new IllegalArgumentException(record);
• 为了澄清起见，有一个简单的RecordType.process（）方法不能满足以下三个原因：首先，处理实际上离RecordType太远了，不能在RecordType子类中保留其自己的方法。同样，理论上可以由不同的处理器执行一系列不同类型的处理。最后，RecordType被设计为具有最少状态更改方法的简单DTO类。

Answer 1

该访问者模式通常在这样的情况下使用。尽管代码有些复杂，但是添加新的RecordType子类后，您
必须 在所有地方实现逻辑，否则它将无法编译。与instanceof所有的地方也很容易错过一个或两个地方。

例：

public abstract class RecordType {
    public abstract <T> T accept(RecordTypeVisitor<T> visitor);
}

public interface RecordTypeVisitor<T> {
    T visitOne(RecordType1 recordType);
    T visitTwo(RecordType2 recordType);
}

public class RecordType1 extends RecordType {
    public <T> T accept(RecordTypeVisitor<T> visitor) {
        return visitor.visitOne(this);
    }
}

public class RecordType2 extends RecordType {
    public <T> T accept(RecordTypeVisitor<T> visitor) {
        return visitor.visitTwo(this);
    }
}

用法（请注意通用返回类型）：

String result = record.accept(new RecordTypeVisitor<String>() {

    String visitOne(RecordType1 recordType) {
        //processing of RecordType1
        return "Jeden";
    }

    String visitTwo(RecordType2 recordType) {
        //processing of RecordType2
        return "Dwa";
    }

});

我也建议抛出一个异常：

throw new IllegalArgumentException(record);

而不是在null找不到这两种类型时返回。