Помогите Почему не работает евент?

IlyaHaker · 22 авг 2019

Вот интерфейс с одним методом, чтобы его можно было использовать в лямбде

Код:

public interface EventProcessor<T extends Event> extends Listener {
    @EventHandler
    public void onEvent(T e);

    default void register(Main plugin) {
        plugin.getServer().getPluginManager().registerEvents(this, plugin);
    }
}

Вот само тело функции и регистрация листенера

Код:

EventProcessor<AsyncPlayerChatEvent> processor = (e) -> e.getPlayer().sendMessage("lambda is correct");
processor.register(this);

Почему это не работает?

MurlikMurlik · 22 авг 2019

IlyaHaker сказал(а):
Вот интерфейс с одним методом, чтобы его можно было использовать в лямбде

Код:

public interface EventProcessor<T extends Event> extends Listener { @EventHandler public void onEvent(T e); default void register(Main plugin) { plugin.getServer().getPluginManager().registerEvents(this, plugin); } }

Вот само тело функции и регистрация листенера

Код:

EventProcessor<AsyncPlayerChatEvent> processor = (e) -> e.getPlayer().sendMessage("lambda is correct"); processor.register(this);

Почему это не работает?

Может быть потому что тебе нужно юзать абстрактный класс а не интерфейс.

CoolBoy · 22 авг 2019

Если хочешь написать что-то подобное, посмотри вот этот репозиторий: https://github.com/lucko/helper/tree/master/helper/src/main/java/me/lucko/helper

То, что написал ты работать не сможет. Если хочешь понять почему, глянь как bukkit работает с листенерами.

IlyaHaker · 22 авг 2019

MurlikMurlik сказал(а):
Может быть потому что тебе нужно юзать абстрактный класс а не интерфейс.

Ну как же? Я таким образом по факту создал объект, реализующий интерфейс EventProcessor<>
Если делать это с помощью анонимного класса:

Код:

EventProcessor<AsyncPlayerChatEvent> processor = new EventProcessor<AsyncPlayerChatEvent>() {
   @Override
   @EventHandler
   public void onEvent(AsyncPlayerChatEvent e) {
      e.getPlayer().sendMessage("anon class is correct");
   }
};

То все будет работать
Судя по всему проблема в том, что переопределенный метод не получает аннотации изначального метода
Сделал я еще один метод интерфейса по умолчанию с этой аннотацией, а переопределенный вставил в тело этого метода
Но теперь сервер думает, что тип параметра T это Event, а не тот, что я вписал
Но как же так получается? В том же C++ шаблоны создают точно такой же класс, но вместо параметра будет выбран конкретный тип

alexandrage · 22 авг 2019

IlyaHaker сказал(а):
Ну как же? Я таким образом по факту создал объект, реализующий интерфейс EventProcessor<>
Если делать это с помощью анонимного класса:

Код:

EventProcessor<AsyncPlayerChatEvent> processor = new EventProcessor<AsyncPlayerChatEvent>() { @Override @EventHandler public void onEvent(AsyncPlayerChatEvent e) { e.getPlayer().sendMessage("anon class is correct"); } };

То все будет работать
Судя по всему проблема в том, что переопределенный метод не получает аннотации изначального метода
Сделал я еще один метод интерфейса по умолчанию с этой аннотацией, а переопределенный вставил в тело этого метода
Но теперь сервер думает, что тип параметра T это Event, а не тот, что я вписал
Но как же так получается? В том же C++ шаблоны создают точно такой же класс, но вместо параметра будет выбран конкретный тип

Тут то и должно работать. А там не должно и не будет.

alexandrage · 22 авг 2019

Вот тебе тестовый код, узнаешь сам почему не должно сработать.

Код:

    private static List<Listener> Handlers = new ArrayList<Listener>();

    public static <T> T callEvent(T event) {
        Method[] methods;
        for (Object o : Handlers) {
            methods = o.getClass().getMethods();
            for (Method menhod : methods) {
                if (menhod.getAnnotation(EventHandler.class) != null) {
                    try {
                        if (menhod.getParameterTypes()[0].isAssignableFrom(event.getClass())) {
                            menhod.invoke(o, event);
                        }
                    } catch (IllegalAccessException ex) {
                        ex.printStackTrace();
                    } catch (IllegalArgumentException ex) {
                        ex.printStackTrace();
                    } catch (InvocationTargetException ex) {
                        ex.printStackTrace();
                    }
                }
            }
        }
        return (T) event;
    }
 
    public static void registerEvent(Listener o) {
        Handlers.add(o);
    }

Код:

        registerEvent(new EventListener());
        PlayerInteractEvent event = new PlayerInteractEvent(null, null, null, null, null);
        callEvent(event);

IlyaHaker · 22 авг 2019

alexandrage сказал(а):

Вот тебе тестовый код, узнаешь сам почему не должно сработать.

Код:

    private static List<Listener> Handlers = new ArrayList<Listener>();

    public static <T> T callEvent(T event) {
        Method[] methods;
        for (Object o : Handlers) {
            methods = o.getClass().getMethods();
            for (Method menhod : methods) {
                if (menhod.getAnnotation(EventHandler.class) != null) {
                    try {
                        if (menhod.getParameterTypes()[0].isAssignableFrom(event.getClass())) {
                            menhod.invoke(o, event);
                        }
                    } catch (IllegalAccessException ex) {
                        ex.printStackTrace();
                    } catch (IllegalArgumentException ex) {
                        ex.printStackTrace();
                    } catch (InvocationTargetException ex) {
                        ex.printStackTrace();
                    }
                }
            }
        }
        return (T) event;
    }
 
    public static void registerEvent(Listener o) {
        Handlers.add(o);
    }

Код:

        registerEvent(new EventListener());
        PlayerInteractEvent event = new PlayerInteractEvent(null, null, null, null, null);
        callEvent(event);

Да, то, что я хотел сделать, сработало бы с шаблонами C++. Но тут сервер думает, что пришел метод типа Event и где-то в ходе работы он будет безопасно скастован. Я пытался в лямбде явно указать, что там AsyncPlayerChatEvent, но появляется ошибка. Может, можно как-то обойти эту ошибку?

alexandrage · 22 авг 2019

IlyaHaker сказал(а):
Может, можно как-то обойти эту ошибку?

Не юзать лямду. Так твой ивент никогда не вызовется. Первая причина, отсутствие анотации.

CoolBoy · 22 авг 2019

IlyaHaker сказал(а):
Да, то, что я хотел сделать, сработало бы с шаблонами C++. Но тут сервер думает, что пришел метод типа Event и где-то в ходе работы он будет безопасно скастован. Я пытался в лямбде явно указать, что там AsyncPlayerChatEvent, но появляется ошибка. Может, можно как-то обойти эту ошибку?

Если бы ты глянул репозиторий что я указал, ты бы понял, что возможно делать примерно вот так:

Код:

Events.subscribe(PlayerJoinEvent.class)
      .expireAfter(2, TimeUnit.MINUTES) // expire after 2 mins
      .expireAfter(1) // or after the event has been called 1 time
      .filter(e -> !e.getPlayer().isOp())
      .handler(e -> e.getPlayer().sendMessage("Wew! You were first to join the server since it restarted!"));

Но делается это не в пару строк, что написал ты и совсем по-другому.

Ission · 22 авг 2019

IlyaHaker сказал(а):
Да, то, что я хотел сделать, сработало бы с шаблонами C++. Но тут сервер думает, что пришел метод типа Event и где-то в ходе работы он будет безопасно скастован. Я пытался в лямбде явно указать, что там AsyncPlayerChatEvent, но появляется ошибка. Может, можно как-то обойти эту ошибку?

То, что в Java имеет похожий синтаксис на шаблоны, называется Generic и они мало имеют общего с настоящими шаблонами, они узкие и не дают почти никакой широты действий. Почитай про них и расстроишься.

IlyaHaker · 23 авг 2019

alexandrage сказал(а):
Не юзать лямду. Так твой ивент никогда не вызовется. Первая причина, отсутствие анотации.

Аннотация это не проблема, но видимо другая проблема, с дженериками, которая, по всей видимости, не решаема

CoolBoy сказал(а):
Если бы ты глянул репозиторий что я указал, ты бы понял, что возможно делать примерно вот так:

Код:

Events.subscribe(PlayerJoinEvent.class) .expireAfter(2, TimeUnit.MINUTES) // expire after 2 mins .expireAfter(1) // or after the event has been called 1 time .filter(e -> !e.getPlayer().isOp()) .handler(e -> e.getPlayer().sendMessage("Wew! You were first to join the server since it restarted!"));

Но делается это не в пару строк, что написал ты и совсем по-другому.

Я зашел в тот репозиторий, посмотрел, ничего не нашел и забил. Спасибо))

Ission сказал(а):
То, что в Java имеет похожий синтаксис на шаблоны, называется Generic и они мало имеют общего с настоящими шаблонами, они узкие и не дают почти никакой широты действий. Почитай про них и расстроишься.

Да, как оказалось и правда, только синтаксис похож. А жаль

Всем спасибо!!

Exception_Prototype · 23 авг 2019

IlyaHaker сказал(а):
не нашел

https://github.com/lucko/helper/blob/master/helper/src/main/java/me/lucko/helper/Events.java

CoolBoy · 23 авг 2019

IlyaHaker сказал(а):
Аннотация это не проблема, но видимо другая проблема, с дженериками, которая, по всей видимости, не решаема

Я зашел в тот репозиторий, посмотрел, ничего не нашел и забил. Спасибо))

Да, как оказалось и правда, только синтаксис похож. А жаль

Всем спасибо!!

Хорошо искал.

IlyaHaker · 23 авг 2019

CoolBoy сказал(а):
Хорошо искал.

Поправочка, я не мог найти нужный класс до твоего примера
А так, репозиторий большой, я минут 15 полистал, ничего не нашел и забил

Luminate_ · 28 авг 2019

alexandrage сказал(а):

Вот тебе тестовый код, узнаешь сам почему не должно сработать.

Код:

    private static List<Listener> Handlers = new ArrayList<Listener>();

    public static <T> T callEvent(T event) {
        Method[] methods;
        for (Object o : Handlers) {
            methods = o.getClass().getMethods();
            for (Method menhod : methods) {
                if (menhod.getAnnotation(EventHandler.class) != null) {
                    try {
                        if (menhod.getParameterTypes()[0].isAssignableFrom(event.getClass())) {
                            menhod.invoke(o, event);
                        }
                    } catch (IllegalAccessException ex) {
                        ex.printStackTrace();
                    } catch (IllegalArgumentException ex) {
                        ex.printStackTrace();
                    } catch (InvocationTargetException ex) {
                        ex.printStackTrace();
                    }
                }
            }
        }
        return (T) event;
    }
 
    public static void registerEvent(Listener o) {
        Handlers.add(o);
    }

Код:

        registerEvent(new EventListener());
        PlayerInteractEvent event = new PlayerInteractEvent(null, null, null, null, null);
        callEvent(event);

multi-catch юзать религия не позволяет?

Dymeth · 28 авг 2019

Luminate_ сказал(а):
multi-catch юзать религия не позволяет?

Это auto-generated код на один раз для теста. Не придирайся ко всякой херне

Luminate_ · 29 авг 2019

Dymeth сказал(а):
Это auto-generated код на один раз для теста. Не придирайся ко всякой херне

Да так, практикуюсь писать более компактный и читабельный код без костылей, вот и придераюсь куда попало
:O

Помогите Почему не работает евент?

IlyaHaker

Активный участник

MurlikMurlik

Активный участник

CoolBoy

Активный участник

IlyaHaker

Активный участник

alexandrage

Старожил

alexandrage

Старожил

IlyaHaker

Активный участник

alexandrage

Старожил

CoolBoy

Активный участник

Ission

Старожил

IlyaHaker

Активный участник

Exception_Prototype

Активный участник

CoolBoy

Активный участник

IlyaHaker

Активный участник

Luminate_

Активный участник

Dymeth

Активный участник

Luminate_

Активный участник