Помогите По чему это значение не равно этому значению?

Я писал плагин на ключи для сундуков. И столкулся с проблемой. Есть 2 String переменные. В первой переменной хранится имя сундука, во второй переменной хранится имя ключа. Вот пример:

Code:

Chest chest = (Chest) event.getClickedBlock().getState();
String chestname = chest.getCustomName();
String keyname =  player.getItemInHand().getItemMeta().getDisplayName();
if (chestname == keyname){
    player.sendMessage(Подходит.);
}
if (chestname != keyname){
    player.sendMessage(Не подходит.);
}

Когда я переименовываю сундук и ключ в разные имена, он пишет (Не подходит.). Если одинаковые то, он пишет (Не подходит.). Почему? Подходит он пишет, только если оба значения null.

 

chestname.equals(keyname)

 

Потому что сравниваются ссылки на объекты и равны они будут только если это один и тот же объект.

 

+1

 

Code:

Chest chest = (Chest) event.getClickedBlock().getState();
        if(event.getPlayer().getItemInHand().getItemMeta().getDisplayName() == "Название шмотки" &&
                chest.getBlockInventory().getTitle() == "Название инвенторя") {
            player.sendMessage("Подходит.");
        } else {
            player.sendMessage("Не подходит.");
        }

Пробуй.

Ну или

Code:

if(event.getPlayer().getItemInHand().getItemMeta().getDisplayName() == "Название шмотки" &&
                chest.getBlockInventory().getTitle() == "Название инвенторя") {

Замени на

Code:

if(event.getPlayer().getItemInHand().getItemMeta().getDisplayName() == chest.getBlockInventory().getTitle()) {

 

нельзя так делать

 

Извини, я не тестил xd

 

так вообще делать нельзя. строки через equals надо сравнивать.

 

..? Я лично всегда сравнивал через ==. Вообще, в чём отличие? По сути дела, одно и тоже.

 

Оператор == используется для проверки, что две строки указывают на один и тот же объект.

Code:

// Эти строки имеют одно и тоже же значение
new String("test").equals("test") // --> true 

// ...но это разные объекты
new String("test") == "test" // --> false 

// ...эти строки тоже разные объекты
new String("test") == new String("test") // --> false 

// ...но эти строки указывают на один и тот же объект,
// потому что компилятор добавляет все литералы в пул.
"test" == "test" // --> true 

// Конкатенация литералов тоже происходит на стадии компиляции,
// поэтому они указывают на один объект
"test" == "te" + "st" // --> true

// но вызов substring() происходит во время выполнения,
// в результате получаются разные объекты.
"test" == "!test".substring(1) // --> false

// Строки из пула могут быть получены с помощью вызова intern().
"test" == "!test".substring(1).intern() // --> true

Надо отметить, что == заметно быстрее, чем equals (сравнение ссылки вместо вызова метода и посимвольного сравнения, если строки разной длины), поэтому, если вы работаете со строками из пула (или системного, или своего), замена equals на == может привести к заметному ускорению.

 

Воу воу воу xd. Полегче))0[DOUBLEPOST=1482526891,1482525996][/DOUBLEPOST]Крч, люди, разобрался. Хз почему, но предметы, название которых не имеет цвета, не проходят проверку. Установи любой цвет, будь то даже §f. То есть выдавай игроку не "key", а "§fkey". Цвет не поменяется, а вот работать всё будет.

Code:

public void onInteract(PlayerInteractEvent e) {
        if (e.getAction().equals(Action.RIGHT_CLICK_BLOCK) && e.getClickedBlock().getType() == Material.CHEST) {
            Chest chest = (Chest) e.getClickedBlock().getState();
            if (!chest.getBlockInventory().getName().equals("container.chest") && e.getPlayer().getItemInHand().getItemMeta().getDisplayName() == "§fkey") {
                e.getPlayer().sendMessage(chest.getBlockInventory().getName());
            } else {
                e.getPlayer().sendMessage("sosi");
            }
        }
    }

Хотя мне пришлось немного прокакать на это время, но всё же

 

и все таки когда работаешь со строками лучше использовать equals. (ну кроме тех случаев которые нам так ахриненно расписал Ission