Помогите Как можно проверять какое зелье пьет игрок?

[Rion_GM]

Мне нужно что бы если игрок пил именно зелье исцеления выполнялся код, как это можно сделать?

 

[imDaniX]

На PlayerItemConsumeEvent проверяй предмет PotionMeta#getBasePotionData().getType() == INSTANT_HEAL.

 

[Rion_GM]

На PlayerItemConsumeEvent проверяй предмет PotionMeta#getBasePotionData().getType() == INSTANT_HEAL.

а можно пример если есть?

 

[imDaniX]

а можно пример если есть?

Ну, я вроде уже описал, что нужно. На java 17 можно в одну строку написать так

@EventHandler
public void onConsume(PlayerItemConsumeEvent event) {
    if (event.getItem().getItemMeta() instanceof PotionMeta potionMeta && potionMeta.getBasePotionData().getType() == PotionType.INSTANT_HEAL) {
        // TODO
    }
}

 

[Rion_GM]

Ну, я вроде уже описал, что нужно. На java 17 можно в одну строку написать так

@EventHandler
public void onConsume(PlayerItemConsumeEvent event) {
    if (event.getItem().getItemMeta() instanceof PotionMeta potionMeta && potionMeta.getBasePotionData().getType() == PotionType.INSTANT_HEAL) {
        // TODO
    }
}

говорит
шаблоны в instanceof не поддерживаются на уровне языка 8
что делать?

 

[imDaniX]

говорит
шаблоны в instanceof не поддерживаются на уровне языка 8
что делать?

Как уже сказал, это фича Java 17.

if (meta instanceof PotionMeta potionMeta) {
    // TODO
}

тоже самое, что

if (meta instanceof PotionMeta) {
    PotionMeta potionMeta = (PotionMeta) meta;
}

 

[Rion_GM]

Как уже сказал, это фича Java 17.

if (meta instanceof PotionMeta potionMeta) {
    // TODO
}

тоже самое, что

if (meta instanceof PotionMeta) {
    PotionMeta potionMeta = (PotionMeta) meta;
}

спасибо

 

[Rion_GM]

Как уже сказал, это фича Java 17.

if (meta instanceof PotionMeta potionMeta) {
    // TODO
}

тоже самое, что

if (meta instanceof PotionMeta) {
    PotionMeta potionMeta = (PotionMeta) meta;
}

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

 

[imDaniX]

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

Ты вторую половину проверки из моего первого сообщения то посмотри.

 

[0xLairon1]

дак ты проверку допиши на тип зелья potionMeta.getBasePotionData().getType()

 

[0xLairon1]

Советую изначально выучить яп java а потом уже кодить плагины, не повторяй моих ошибок)

 

[Rion_GM]

Ты вторую половину проверки из моего первого сообщения то посмотри.

я так и пытался

if (meta instanceof PotionMeta && potionMeta.getBasePotionData().getType() == PotionType.INSTANT_HEAL){
potionMeta = (PotionMeta) meta;
}

 

[Rion_GM]

Советую изначально выучить яп java а потом уже кодить плагины, не повторяй моих ошибок)

я его уже почти дописал так что поздно ты сказал)

 

[imDaniX]

я так и пытался

if (meta instanceof PotionMeta && potionMeta.getBasePotionData().getType() == PotionType.INSTANT_HEAL){
potionMeta = (PotionMeta) meta;
}

Ну дак potionMeta у тебя недоступен во время проверки. Тебе нужно написать эту проверку отдельно уже после получения меты.

 

[Rion_GM]

Ну дак potionMeta у тебя недоступен во время проверки. Тебе нужно написать эту проверку отдельно уже после получения меты.

if (meta instanceof PotionMeta){
potionMeta = (PotionMeta) meta;
potionMeta.getBasePotionData().getType() == PotionType.INSTANT_HEAL;
}
так?)

 

[imDaniX]

if (meta instanceof PotionMeta){
potionMeta = (PotionMeta) meta;
potionMeta.getBasePotionData().getType() == PotionType.INSTANT_HEAL;
}
так?)

if (...) {} забыл.

 

[Rion_GM]

if (...) {} забыл.

может я тут где то накосячил?

public void onPotionDrink(PlayerItemConsumeEvent event) {
            Player player = (Player) event.getPlayer();

            Object meta = null;
            PotionMeta potionMeta;
            if (meta instanceof PotionMeta){
                potionMeta = (PotionMeta) meta;
                if (potionMeta.getBasePotionData().getType() == PotionType.SPEED){
                    player.sendMessage(ChatColor.GOLD + "Текст ");
                }
            }
        }

 

[imDaniX]

может я тут где то накосячил?

public void onPotionDrink(PlayerItemConsumeEvent event) {
            Player player = (Player) event.getPlayer();

            Object meta = null;
            PotionMeta potionMeta;
            if (meta instanceof PotionMeta){
                potionMeta = (PotionMeta) meta;
                if (potionMeta.getBasePotionData().getType() == PotionType.SPEED){
                    player.sendMessage(ChatColor.GOLD + "Текст ");
                }
            }
        }

Да. meta должна быть взята с употребляемого предмета.

 

[Rion_GM]

Да. meta должна быть взята с употребляемого предмета.

можешь исправить и скинуть код если не сложно пожалуйста?

 

[imDaniX]

можешь исправить и скинуть код если не сложно пожалуйста?

public void onPotionDrink(PlayerItemConsumeEvent event) {
    Player player = event.getPlayer();
    ItemMeta meta = event.getItem().getItemMeta();
    if (meta instanceof PotionMeta) {
        PotionMeta potionMeta = (PotionMeta) meta;
        if (potionMeta.getBasePotionData().getType() == PotionType.SPEED){
            player.sendMessage(ChatColor.GOLD + "Текст ");
        }
    }
}

Ну, и если будешь расширять плагин, настоятельно советую прислушаться к @0xLairon1