Помогите Ошибка при addPotionEffect

[nikvas2000]

Помогите,
я не знаю почему эффект не накладывается на игрока.
Eclipse ничего не подчеркивает.

@EventHandler
    public void click(PlayerInteractEvent e){
        if(e.getAction() == Action.RIGHT_CLICK_BLOCK || e.getAction() == Action.RIGHT_CLICK_AIR){
            Player p = e.getPlayer();
            if(p.getItemInHand().getType() == Material.GOLD_BLOCK){
                ItemMeta goldblockmeta = p.getItemInHand().getItemMeta();
                if(goldblockmeta.getDisplayName() == "Jump"){
                    p.addPotionEffect(new PotionEffect(PotionEffectType.JUMP, 10, 3));
                }
                if (goldblockmeta.getDisplayName() == "Speed"){
                    p.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 10, 3));
                }
            }
            if (p.getItemInHand().getType() == Material.DIAMOND_BLOCK){
                ItemMeta diamondblockmeta = p.getItemInHand().getItemMeta();
                if(diamondblockmeta.getDisplayName() == "Jump"){
                    p.addPotionEffect(new PotionEffect(PotionEffectType.JUMP, 10, 5));
                }
                if (diamondblockmeta.getDisplayName() == "Speed"){
                    p.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 10, 5));
                }
            }
        }
    }

 

[unlimited]

Вероятно не срабатывают if'ы, поставь после каждого ифа например p.sendMessage('***'); и посмотри какие срабатывают, а какие нет

 

[StealerSlain]

может юзнешь else if?

 

[dark32]

goldblockmeta.getDisplayName() == "Speed"

Строки нельзя сравнивать на равенство. Строки нужно сравнивать на эквиваленцию.
В первом случае сравнивается их ссылки, во втором - содержимое.

goldblockmeta.getDisplayName().equals("Speed")

И, вообще, делайте сообщения для отладки, помогает.

 

[gamelax]

dark32, сравнение строк оператором == вернет true, когда обе переменные указывают на один объект в памяти.

 

[dark32]

Ну, а я что написал?

...В первом случае сравнивается их ссылки...

Если ссылки на объект равны, то это один объект, но ещё есть пул коротких строк, но на него лучше не надеяться.
Но всё равно сравнивать строки с помощью == к хорошему не приводит.

 

[nikvas2000]

Спасибо всем за ответы.