我有以下域对象:
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
替换的现有问题,但所有问题都建议将一些逻辑直接添加到域对象中。但是,在这种情况下,我不想用仅针对我的模块的逻辑来污染它们。
答案 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
。