Помогите Не вызывается InteractEvent, в консоль не выводит ошибки

[Kloper]

Не вызывается ивент, мой код:

    @EventHandler
    public void usageHeal(PlayerInteractEvent e) {
        Player p = e.getPlayer();
        Action a = e.getAction();
        if(a != Action.RIGHT_CLICK_AIR || a != Action.RIGHT_CLICK_BLOCK) {
            return;
        } else {
            String checkRank = plugin.getConfig().getString("players." + p.getName() + ".rank");
            if(p.getInventory().getItemInMainHand().getType().equals(Material.PAPER)) {
                if(p.getInventory().getItemInMainHand().getItemMeta().getDisplayName().contains(ChatColor.WHITE + "Свиток мгновенного лечения")) {
                    int rank = Integer.parseInt(checkRank);
                    if(rank >= 3) {
                        p.getInventory().removeItem(p.getInventory().getItemInMainHand());
                        p.setHealth(p.getHealth() + 2);
                    } else {
                        return;
                    }
                } else {
                    return;
                }
            } else {
                return;
            }
        }
    }

config.yml:

players:
  f3azzy:
    rank: 10

класс в onEnable зарегистрировал.
в консоле ошибок никаких нету

 

[Zlodey_duo]

Если я правильно понимаю у тебя возвращается 'return' (отмена)

Связано это с проверкой:

if(a != Action.RIGHT_CLICK_AIR || a != Action.RIGHT_CLICK_BLOCK) {
            return;

*Если правый-клик не по воздуху и правый-клик не по блоку то завершаем процесс*
Тебе нужно сделать в точности наоборот.

Замени проверку которую я отметил выше, на:

if(a == Action.RIGHT_CLICK_AIR || a == Action.RIGHT_CLICK_BLOCK) {
            return;
        }

*Если действие произведено по воздуху или по блоку, то*
Делай следующую проверку на содержания предмета у него в руках.

(Может я где то и не догоняю, более опытные люди могут меня поправить, ну я пытался помочь) Сильно не чморите уш меня)

 

[Dymeth]

if (a != Action.RIGHT_CLICK_AIR || a != Action.RIGHT_CLICK_BLOCK) return;

А теперь тебе вопрос - какое должно быть действие, чтобы твой код продолжил работу? )

UPD: Упс, опередили.

Вместо логического "или" тебе нужно "и"

 

[Zlodey_duo]

Если я правильно понимаю у тебя возвращается 'return' (отмена)

Связано это с проверкой:

if(a != Action.RIGHT_CLICK_AIR || a != Action.RIGHT_CLICK_BLOCK) {
            return;

*Если правый-клик не по воздуху и правый-клик не по блоку то завершаем процесс*
Тебе нужно сделать в точности наоборот.

Замени проверку которую я отметил выше, на:

if(a == Action.RIGHT_CLICK_AIR || a == Action.RIGHT_CLICK_BLOCK) {
            return;
        }

*Если действие произведено по воздуху или по блоку, то*
Делай следующую проверку на содержания предмета у него в руках.

(Может я где то и не догоняю, более опытные люди могут меня поправить, ну я пытался помочь) Сильно не чморите уш меня)

Да, да, да. Заметил что у тупень, сейчас исправлю
сорри

 

[Zlodey_duo]

Меня свою проверку клика на эту:

if(a == Action.RIGHT_CLICK_AIR || a == Action.RIGHT_CLICK_BLOCK) {

               // твой код

        }else{
      return;
}

 

[Kloper]

не заметил, спасибо
закрыто