java - 为什么即使注释有 RetentionPolicy.RUNTIME，我的 isAnnotationPresent 方法也不起作用？

Question

我正在尝试为我的 OpenGL 游戏引擎实现基于注释的事件系统。我在我想要这样调用的方法上应用 @EventListener 注解:

@EventListener(type = Type.COLLISION)
public void OnCollision(CollisionEvent data)
{
    System.out.println("HI");
}

该方法所在的类实现了一个空接口(interface):

public class Sprite implements EventHandler

EventDispatcher 类:

public class EventDispatcher
{
private static List<EventHandler> registered = new ArrayList<EventHandler>();

public static void register(EventHandler EventHandler)
{
    if (!registered.contains(EventHandler))
    {
        registered.add(EventHandler);
    }
}

public static void unregister(EventHandler EventHandler)
{
    if (registered.contains(EventHandler))
    {
        registered.remove(EventHandler);
    }
}

public static List<EventHandler> getRegistered()
{
    return registered;
}

public static void dispatch(final Event event)
{
    new Thread()
    {
        @Override
        public void run()
        {
            call(event);
        }
    }.start();
}

private static void call(final Event event)
{
    for (EventHandler registered : getRegistered())
    {
        Method[] methods = registered.getClass().getMethods();

        for (int i = 0; i < methods.length; i++)
        {
            System.out.println("Annotation Being Checked");
            if (methods[i].isAnnotationPresent(EventListener.class))
            {
                System.out.println("Has Annotation");
                Class<?>[] methodParams = methods[i].getParameterTypes();
                if (methodParams.length < 1)
                {
                    continue;
                }
                if (!event.getClass().getSimpleName().equals(methodParams[0].getSimpleName()))
                {
                    continue;
                }
                try
                {
                    methods[i].invoke(registered.getClass().newInstance(), event);
                } catch (Exception exception)
                {
                    System.err.println(exception);
                }
            } else System.out.println("No Annotation");
        }
    }
}
}

但是当我运行程序时，它总是打印出来

正在检查注释
无注释

多次。

有人可以帮忙吗？如果需要更多信息，请询问，我将编辑问题。

Answer 1

我根据您的示例设置了一个项目，并且运行良好。然而，当您的代码评估 Sprite 事件处理程序的所有方法时，您会看到一些“无注释”消息。即使您没有实现除 OnCollision 之外的任何其他方法，每个类也会从 Object 继承默认方法，例如 equals、hashCode 或 toString。

测试类:

public class SpriteTest {

    public static void main(String[] args) {
        EventDispatcher.register(new Sprite());

        CollisionEvent collisionEvent = new CollisionEvent();
        EventDispatcher.dispatch(collisionEvent);
    }
}

除此之外，您的代码中还存在一些明显的缺陷:

• 不要使用有状态静态成员 (EventDispatcher.registered)，除非您知道自己在做什么并且了解随之而来的多线程方面
• 您存储 EventHandler 的实例，但仅使用类信息并动态创建新实例 - 为什么不直接注册类而不是实例
• 您为每个要分派(dispatch)的事件创建新线程。这是非常糟糕的做法，因为创建线程是一项成本高昂的操作。使用线程池来代替并提交可运行或可调用
• 您检查类的简单名称是否匹配，以查看处理程序方法是否适用。使用继承时这会中断，应替换为 Class.isAssignableFrom
• 这里注释的一般用法是有问题的。您最好为不同的事件类型使用专用接口(interface)。可以使用 CollisionEventHandler 来代替通用的 EventHandler 等等...

粗略的实现思路

public interface CollisionEventHandler extends EventHandler {
  void onCollision(CollisionEvent event);  
}

public class Sprite implements CollisionEventHandler {
  public void onCollision(CollisionEvent data) {
    System.out.println("HI");
  }
}

public class EventDispatcher {
  ...

  static void call(final CollisionEvent event) {
    getRegistered().stream()
          .filter(handler -> handler instanceof CollisionEventHandler)
          .map(handler -> (CollisionEventHandler) handler)
          .forEach(handler -> handler.onCollision(event));
  }
}

要处理不同类型的事件，您将需要不同的调用/调度方法。也许你可以使用 Visitor pattern (虽然我不喜欢它)。