Question

我有一个界面：

public interface Handler<E extends Event> {
    void handle(E event);
}

以及它的两个实现：

public class SignupEventHandler implements Handler<SignupEvent> {
    @Override
    public void handle(SignupEvent event) {}
}

和

public class SignoutEventHandler implements Handler<SignoutEvent> {
    @Override
    public void handle(SignoutEvent event) {}
}

请注意，Event 本身是一个接口，由 SignupEvent 和 SignoutEvent 实现。

我想知道为 Handler 实现工厂的正确方法是什么。

public class HandlerFactory {
    public static Handler<? extends Event> getHandler(Event event) {
        if(event.type().equals("SignupEvent")) {
            return new SignupEventHandler();
        } else if(event.type().equals("SignoutEvent")) {
            return new SignoutEventHandler();
        } else {
            throw new IllegalArguementException("Unrecognised event type");
        }
    }
}

我面临的问题是我不能做 HandlerFactory.getHandler(event).handle(event)，因为它想要一个 capture of ? of Event。

Event event = getDeserialisedEvent(message, Event.class);
Handler<? extends Event> handler = HandlerFactory.getHandler(event);
handler.handle(event);    //can't do this

有没有一种干净的方法来实现这一点？我发现的一种方法是：

public class HandlerFactory {
    public static <E extends Event> Handler<E> getHandler(E event) {
        if(event.type().equals("SignupEvent")) {
            return (Handler<E>) new SignupEventHandler();    //unchecked cast, but it's guaranteed to be type-safe
        } else if(event.type().equals("SignoutEvent")) {
            return (Handler<E>) new SignoutEventHandler();   //unchecked cast, but it's guaranteed to be type-safe
        } else {
            throw new IllegalArguementException("Unrecognised event type");
        }
    }
}

这种方法有效，但我想知道是否有更简洁的方法来做到这一点。

Answer 1

方式略有不同，但都基本相同。他们都以一种或另一种方式使用强制转换。

例如，您可以使用原始类型：

public static Handler getHandler(Event  event)

Answer 2

如果你有一个委托给正确处理程序的类而不是工厂，你可以很好地解决这个问题。

public class EventHandlerFacade {

    public void handle(Event event) {
        if (event instanceof SignupEvent) {
            new SignupEventHandler().handle((SignupEvent) event);
        }
        else if (event instanceof SignoutEvent) {
            new SignoutEventHandler().handle((SignoutEvent) event);
        }
        else {
            System.out.println("Unhandled event of class " + event.getClass());
        }
    }

}

在这段代码中，Java 可以看到强制转换是安全的。无论外观是在每次需要处理程序时都创建处理程序（如代码中所示），还是保留处理程序并重用它们，它都将起作用。

您可能还想用自己拥有的东西生活。泛型用于编译时类型检查。当您在编译时不知道 event 的实际（运行时）类型时，编译时类型检查没有多大意义。所以你可以选择放弃它。

Answer 3

一个可能的解决方案是将 Handler 中的 handle(E); 替换为 handle(Event);，然后将 Event 转换为从 Handler 继承的类中想要的子类。我不认为有一种方法可以完全满足您的要求，因为编译器不知道 Handler 的泛型类型。

Answer 4

我们可以尝试这样的事情。
问题是 void handle(E event); 无法捕获类型参数。所以我们通过像 <T extends Event> void handle(T event) 一样声明它来捕获它。其余的解释在以下代码之后。

public class Test{
    
    public static void main(final String[] args) throws Exception {
        Event e =new SignupEvent();
        Handler<? extends Event> handler = HandlerFactory.getHandler(e);
        handler.handle(e); //yeppie!

        e = new SignoutEvent();
        handler = HandlerFactory.getHandler(e);
        handler.handle(e);
  }
}

class HandlerFactory {
    public static Handler<? extends Event> getHandler(Event event) {
        if(event instanceof SignupEvent)
            return new SignupEventHandler();
        return new SignoutEventHandler();
    }
}

abstract class Handler<E extends Event> {
    @SuppressWarnings("unchecked") //<-- we know what we are doing
    final <T extends Event>  void handle(T event) {
        handlerHelper((E)event);
    }
    
    protected abstract void handlerHelper(E event);
}

class SignupEventHandler extends Handler<SignupEvent> {
    @Override
    protected void handlerHelper(SignupEvent event) {
        System.out.println("Handling sign UP");
    }
}

class SignoutEventHandler extends Handler<SignoutEvent> {
    @Override
    protected void handlerHelper(SignoutEvent event) {
        System.out.println("Handling sign OUT");
    }
}

interface Event{
    
}
class SignupEvent implements Event{
    
}
class SignoutEvent implements Event{
    
}

输出：

Handling sign UP
Handling sign OUT

现在 handle 方法能够使用方法的类型参数捕获类型。但是我们想要强制执行 Handler 类的类型参数。所以我们创建单独的 handlerHelper 方法，它使用类的类型参数。并从我们的 handle 方法中调用它。现在我们会收到未经检查的强制转换警告，我们将取消该警告。
我们将使 handle 方法最终化。这样子类就不会覆盖它。由于接口不能有最终方法，我们必须使 Handler 类抽象。
此外，我们对 handlerHelper 方法进行了保护，以便 API 用户将仅调用 handle 方法。

通用接口工厂

4 个答案: