[SOLVED] Дублируется код внутри проверки getItemInMainHand();

[Zemf4you]

Ну, вот да, в общем. Знаю, обсуждалось и не раз, но чёт вариант @alexandrage не помог.

@EventHandler
    public void interact(PlayerInteractEvent event) {
        Action action = event.getAction();
        if(action == Action.RIGHT_CLICK_AIR || action == Action.RIGHT_CLICK_BLOCK) {
            Player player = event.getPlayer();
            ItemStack item = player.getItemInHand();
        }

getItemInHand() выпилили, а getItemInMainHand() и getPlayer().getInventory().getItemInMainHand() не пашет. Подскажите, пожалуйста, метод)

 

[Cool_boy]

player.getInventory().getItemInMainHand() работает?
И PlayerInteractEvent проходит два раза, для обоих рук, будь осторожен.

 

[Zemf4you]

player.getInventory().getItemInMainHand() работает?
И PlayerInteractEvent проходит два раза, для обоих рук, будь осторожен.

ого, спасибо, пашет. И спасибо за предупреждение)

 

[Zemf4you]

player.getInventory().getItemInMainHand() работает?
И PlayerInteractEvent проходит два раза, для обоих рук, будь осторожен.

UPD: Ну. я шот тупой и не понял, что делать - код внутри if дублируется, как ты и предупреждал. Как пофиксить?

@EventHandler
    public void interact(PlayerInteractEvent event) {
        Action a = event.getAction();
        if(a == Action.RIGHT_CLICK_AIR || a == Action.RIGHT_CLICK_BLOCK) {
            Player p = event.getPlayer();
            ItemStack item = p.getInventory().getItemInMainHand();
            if(item.getType() == Material.STICK) { 
                p.sendMessage("§3Условие выполнено");
            }
        }
       
    }

 

[xDark]

UPD: Ну. я шот тупой и не понял, что делать - код внутри if дублируется, как ты и предупреждал. Как пофиксить?

@EventHandler
    public void interact(PlayerInteractEvent event) {
        Action a = event.getAction();
        if(a == Action.RIGHT_CLICK_AIR || a == Action.RIGHT_CLICK_BLOCK) {
            Player p = event.getPlayer();
            ItemStack item = p.getInventory().getItemInMainHand();
            if(item.getType() == Material.STICK) {
                p.sendMessage("§3Условие выполнено");
            }
        }
     
    }

... Потому что ивент один и тот же и вызывается два раза, сначала для правой руки, а потом - для левой
hashCode сравнивай

 

[Zemf4you]

... Потому что ивент один и тот же и вызывается два раза, сначала для правой руки, а потом - для левой
hashCode сравнивай

Можешь подсказать как, пожалуйста?

 

[xDark]

Можешь подсказать как, пожалуйста?

private final Set<Integer> codes= Sets.<Integer>newHashSet();
final int hashCode = event.hashCode();
if (!codes.contains(hashCode)) {
codes.add(hashCode);
//TODO
}

 

[Zemf4you]

private final Set<Integer> codes= Sets.<Integer>newHashSet();
final int hashCode = event.hashCode();
if (!codes.contains(hashCode)) {
codes.add(hashCode);
//TODO
}

а можешь что-нибудь скинуть, где можно об этом почитать?

 

[xDark]

а можешь что-нибудь скинуть, где можно об этом почитать?

https://habrahabr.ru/post/165683/
https://habrahabr.ru/post/168195/

 

[xDark]

Или:

if (e.getHand() == EquipmentSlot.HAND) {
....
}

 

[iD3LSY]

Или:

if (e.getHand() == EquipmentSlot.HAND) {
....
}

Рабочий метод. Хочется подметить, что метод сработает 2 раза только на 1.9+ ядре.

 

[Zemf4you]

Или:

if (e.getHand() == EquipmentSlot.HAND) {
....
}

о, ого, благодарю)

 

[Zemf4you]

Спасибо большое, @xDark! Проблема решена!

@EventHandler
    public void interact(PlayerInteractEvent event) {
        Action a = event.getAction();
        if(a == Action.RIGHT_CLICK_AIR || a == Action.RIGHT_CLICK_BLOCK) {
            Player p = event.getPlayer();
            ItemStack item = p.getInventory().getItemInMainHand();
            if (event.getHand() == EquipmentSlot.HAND && item.getType() == Material.STICK) {
                p.sendMessage("§4Условие выполнено");
            }
           
        }
       
    }