在Java中没有instanceof或getClass()的动态调度

时间:2016-04-06 16:01:01

标签: java design-patterns instanceof dynamic-dispatch

我有以下域对象:

public interface Event {}    
public class FirstEvent {}    
public class SecondEvent {}

然后我有另一个模块,它应该与我的域对象完全分离,这意味着它知道域对象,但域对象不应该知道这个附加模块的存在。

在这个模块中,我通过公共接口Event接收对象,我需要根据特定的事件类型采取不同的行动。

目前我的代码如下:

if (event instanceof FirstEvent.class) {
    doFirst();
}
else if (event instanceof SecondEvent.class) {
    doSecond();
}

它工作正常,但静态分析工具和代码审查员抱怨我不应该使用instanceof,我应该用更多面向对象的方法替换它。反思或getClass()也不是一种选择。

如何用Java做到这一点?

我已经回顾了许多有关instanceof替换的现有问题,但所有问题都建议将一些逻辑直接添加到域对象中。但是,在这种情况下,我不想用仅针对我的模块的逻辑来污染它们。

2 个答案:

答案 0 :(得分:3)

访客模式,又名Double Dispatch,在这里通常很有用。

使用每个已知事件类型的方法定义接口,并且每个事件都实现一个接口方法,该方法允许外部对象使用该接口的实现来调用它。然后,该事件确保使用自己的'这个'来调用接口的类型特定方法。参考,所以你没有得到任何明确的降低。

public interface EventVisitor  {
    visit(FirstEvent firstEvent);
    visit(SecondEvent secondEvent);
}

public class FirstEvent {
    ...
    public void allowVisit(EventVisitor ev) {
        ev.visit(this); // calls the 'FirstEvent' overriden method
    }
    ...
}


public class SecondEvent {
    ...
    public void allowVisit(EventVisitor ev) {
        ev.visit(this); // calls the 'SecondEvent' overriden method
    }
    ... 
}

public class MyOtherObject implements EventVisitor, EventListener {
   ...
   public void signalEvent(Event e) {
       e.allowVisit(this);
   }

   public void visit(FirstEvent e) {
       // handle FirstEvent type
   }

   public void visit(SecondEvent e) {
       // handle SecondEvent type
   }

}

此类事情的缺点是,添加新的事件类型变得很困难,因为EventListener接口必须枚举它们。你可以'善良'用一种全能的方法解决这个问题,但它很麻烦而且仍然难以升级。

答案 1 :(得分:0)

排除Reflection是可以理解的,但使用getClass()不应该是问题。

我解决这个问题的方法:

import java.util.concurrent.ConcurrentHashMap;
import java.util.Map;

public class DynamicDispatch{
    Map<String,Event> events = new ConcurrentHashMap<String,Event>();
    public DynamicDispatch(){
        Event event = new FirstEvent();
        events.put(event.getName(),event);
        event = new SecondEvent();
        events.put(event.getName(),event);
    }
    public Event getEvent(String eventName){
        return events.get(eventName);
    }
    public static void main(String args[]){
        DynamicDispatch dispatchObj = new DynamicDispatch();
        Event event = dispatchObj.getEvent(args[0]);
        System.out.println("dispatchObj:"+event+":"+event.getName());
    }
}

interface Event {
    public String getName();
}
class FirstEvent implements Event{
    public String getName(){
        //return this.getClass().getSimpleName();
        return "FirstEvent";
    }
}
class SecondEvent implements Event{
    public String getName(){
        //return this.getClass().getSimpleName();
        return "SecondEvent";
    }
}

输出:

java DynamicDispatch FirstEvent
dispatchObj:FirstEvent@72d86c58:FirstEvent

java DynamicDispatch SecondEvent
dispatchObj:SecondEvent@72d86c58:SecondEvent

我仍然倾向于使用return this.getClass().getSimpleName();而不是对值进行硬编码。

我希望上面的代码可以使用static analysis tools