如何在不使用instanceof的情况下查找类型？

Question

我的类 Query 中有一个接口类型 Criteria 的 List。

List<Criteria> criteria = new ArrayList<Criteria>();

我有几个 Criteria 的具体实现。我想为 Query 提供一个迭代我的 criteria 列表的方法，并根据具体类型执行一些逻辑。

我目前正在使用 instanceof 这样做：

for(Criteria c : criteria) {
    if(c instanceof ContextualCriteria){
        // logic
    }
    ...
}

这是唯一/最好的方法吗？

Answer 1

逻辑是否明显属于 Criteria 本身？如果是这样，请将其放入 Criteria 接口中，并为实现 Criteria 接口的每个具体类适当地实现它。这显然是很好的多态方法。

不幸的是，在现实生活中，面向对象并不总是那么简单 - 有时将每个类型的行为放入类型本身是没有意义的，因此您可能需要使用 instanceof 来代替。您可能可能有一个从“标准类”到代表要采取的操作的某个接口的映射，但这很容易变得更加混乱。

通过访问者模式进行双重调度有时可以稍微改进 -因此逻辑仍然可以位于“调用”类中的单独方法中，但每个 Criteria 都可以通过单个接口方法回调用正确的方法。就我个人而言，我倾向于发现这会增加耦合并很快变得丑陋，但其他人对此深信不疑。

Answer 2

如果逻辑不属于 Jon Skeet 建议的标准，那么您可以使用访问者模式 。

在 ConcreteCriteria 中：

public void accept(CriteriaVisitor v) {
    v.visit(this);
}

在客户端代码中：

public void method() {
    for (Criteria c : criteria) {
        c.accept(this);
    }
}

public void visit(ConcreteCriteria c) {
    // do logic here
}

public void visit(Criteria c) {
    // othervise...
}

这摆脱了 instanceof，但要小心，我发现如果您不熟悉代码，这种模式很难理解。

Answer 3

好吧，你可以……

if (ContextualCriteria.class.equals(c.getClass()) {

尽管这只是一种看起来更奇特的 instanceof 编写方式。 （好吧，几乎：这测试它是否正是该类，而不是子类的类——因为您需要 isAssignableFrom()）。

消除气味的正确方法是在 Criteria 中实现多态方法，例如，该方法在子类中被重写。

Answer 4

界面是策略模式的一半！要根据类型改变逻辑，请尽可能将其推到接口后面，以便 Criteria 有一个 doLogic()。您可以向该方法传递您可能需要在调用代码中更改的任何参数，或者返回新信息 - 这是非常特定于实现的，并且很难从相关代码中提供建议。

如果一切顺利，您的调用代码就会结束

for (Criteria c : criteria) {
    c.doLogic();
}

Answer 5

您还可以在实现 Criteria 的类中实现逻辑

public interface Criteria {
  public void logic();
}

在 ContextualCriteria 等类中具有多种实现
你的循环看起来很干净：

for(Criteria c : criteria) {  
   c.logic();
}  

Answer 6

您还可以使用：

for(Criteria c : criteria) {
   if(c.getClass() == ContextualCriteria.class){
       // logic
   }
   if ...
}

请注意 Object#getClass() 返回 c 的运行时类型，因此如果 ContextualCriteria 可以被子类化，则无法可靠地使用它。为此，您需要使用 Class#isAssignableFrom()：

for(Criteria c : criteria) {
   if(ContextualCriteria.class.isAssignableFrom(c)){
       // logic
   }
   if ...
}

Answer 7

这并没有什么问题。

从未使用过 instanceof 的人只编写了玩具应用程序。