Ошибка при использование PlayerInteractEvent

[_Ruleto4ka_]

Есть у меня код:

      @EventHandler
      public void onPlayerInteract(PlayerInteractEvent event) {
        if (event.getAction() == Action.RIGHT_CLICK_BLOCK &&
          event.getClickedBlock().getType().toString().endsWith("SHULKER_BOX") || event.getClickedBlock().getType().toString().endsWith("BARREL"))
          event.setCancelled(true);
      }
    }

И почему-то выводит иногда ошибку
https://pastebin.com/3AShmiZ6

 

[deadanykey]

Клик по воздуху и getClickedBlock() возвращает null? Не?

 

[Todorov26]

Сравнивай через материал, а не по названию./

Клик по воздуху и getClickedBlock() возвращает null? Не?

У него стоит что Action = right click block Клик по воздуху в данном случае игнорируется.
Но возможно ты прав. Стоит проверять event.getClickedBlock на значение null мало ли что)

 

[_Ruleto4ka_]

Сравнивай через материал, а не по названию./



У него стоит что Action = right click block Клик по воздуху в данном случае игнорируется.
Но возможно ты прав. Стоит проверять event.getClickedBlock на значение null мало ли что)

      @EventHandler
      public void onPlayerInteract(PlayerInteractEvent event) {
        Block block = event.getClickedBlock();
        if (block == null)
            return;
        if (block.getType() != Material.SHULKER_BOX)
        /*if (event.getAction() == Action.RIGHT_CLICK_BLOCK &&
            event.getClickedBlock().getType().toString().endsWith("SHULKER_BOX") || event.getClickedBlock().getType().toString().endsWith("BARREL"))*/
          event.setCancelled(true);
      }

Сделал так, только теперь отменяется любое действия

 

[deadanykey]

      @EventHandler
      public void onPlayerInteract(PlayerInteractEvent event) {
        Block block = event.getClickedBlock();
        if (block == null)
            return;
        if (block.getType() != Material.SHULKER_BOX)
        /*if (event.getAction() == Action.RIGHT_CLICK_BLOCK &&
            event.getClickedBlock().getType().toString().endsWith("SHULKER_BOX") || event.getClickedBlock().getType().toString().endsWith("BARREL"))*/
          event.setCancelled(true);
      }

Сделал так, только теперь отменяется любое действия

Даже клик по шалкеру?

 

[_Ruleto4ka_]

Даже клик по шалкеру?

Все действия

 

[deadanykey]

Все действия

Выведите в логгер, какой блок получается по клику.
И версию скажите.

 

[_Ruleto4ka_]

     @EventHandler
     public void onPlayerInteract(PlayerInteractEvent event) {
       Block block = event.getClickedBlock();
       if (block == null) {
           return;
       } else if (block.getType().Material.SHULKER_BOX) {
         event.setCancelled(true);
         } else {
           return;
      }
     }

Та же фигня

 

[_MRDen3000YT_]

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        Material mat = event.getClickedBlock().getType();
        if (mat == null) {
            return;
        } else if (mat.equals(Material.BLACK_SHULKER_BOX)) {
            event.setCancelled(true);
        } else {
            return;
        }
    }

Ступил, сделай так. Теперь нельзя будет кликнуть по черному шалкер-боксу

 

[_Ruleto4ka_]

Можно как-то сделать, чтобы вписать шалкер и оно блочило все шалкера, а не каждого по цвету писать
И в логги пишет ошибку
https://pastebin.com/SCSgHELV

 

[_MRDen3000YT_]

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        if (event.getClickedBlock() == null) {
            return;
        } else {
            Material mat = event.getClickedBlock().getType();
            if (event.getClickedBlock().getState() instanceof ShulkerBox) {
                event.setCancelled(true);
            } else {
                return;
            }
        }

Ошибка должна исчезнуть

 

[LOVEC3327]

Можно как-то сделать, чтобы вписать шалкер и оно блочило все шалкера, а не каждого по цвету писать
И в логги пишет ошибку
https://pastebin.com/SCSgHELV

1.

        if (e.getClickedBlock().getState() instanceof ShulkerBox) {
            //TODO
        }

2. Скинь код через pastebin, чтобы мы могли видеть, что находится в строке, где происходит ошибка.

 

[_MRDen3000YT_]

1.

        if (e.getClickedBlock().getState() instanceof ShulkerBox) {
            //TODO
        }

2. Скинь код через pastebin, чтобы мы могли видеть, что находится в строке, где происходит ошибка.

Ошибка скорее всего в

Material mat = event.getClickedBlock().getType();

была, т.к. нет проверки на null

 

[LOVEC3327]

Ошибка скорее всего в

Material mat = event.getClickedBlock().getType();

была, т.к. нет проверки на null

Так пусть сделает в начале проверку на действие и все.

        if (e.getAction() != Action.RIGHT_CLICK_BLOCK)
            return;
        //TODO

 

[LOVEC3327]

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        if (event.getClickedBlock().getType() == null) {
            return;
        } else {
            Material mat = event.getClickedBlock().getType();
            if (event.getClickedBlock().getState() instanceof ShulkerBox) {
                event.setCancelled(true);
            } else {
                return;
            }
        }

Ошибка должна исчезнуть

Кстати, мне кажется, что тут надо проверять не Material на null, а сам блок, ибо

    @Nullable
    public Block getClickedBlock() {
        return this.blockClicked;
    }

Если блок будет null, то выкинет ту же ошибку NPE.

 

[_MRDen3000YT_]

Кстати, мне кажется, что тут надо проверять не Material на null, а сам блок, ибо

    @Nullable
    public Block getClickedBlock() {
        return this.blockClicked;
    }

Если блок будет null, то выкинет ту же ошибку NPE.

Поправил

 

[LOVEC3327]

Поправил

Кстати, можно еще использовать уже готовый метод hasBlock, который предоставляет класс PlayerInteractEvent

        if (!event.hasBlock())
            return;
        //TODO

 

[_Ruleto4ka_]

Поправил

Я так понимаю когда ломаешь блок это тоже клик и оно блочит ломание блока

 

[deadanykey]

Я так понимаю когда ломаешь блок это тоже клик и оно блочит ломание блока

Сначала Interact, потом BlockBreak.

 

[_Ruleto4ka_]

Сначала Interact, потом BlockBreak.

    @EventHandler
    public void onBreak(BlockBreakEvent event) {
        if (!event.getBlock().getType().equals(Material.BARREL))
            return;
            event.setCancelled(false);
    }