Выдаёт сообщение, где выдавать не должно

[clysic]

Привет, всем кодерам!
Есть условный ивент "BlockBreakEvent" , где я написал первое условие if, а после пишу второе if, и мне выдаёт сообщение из первого if, хотя при этом e.setCancelled успешно работает... Пытался после первого if else, но тогда перестаёт работать e.setCancelled.

public void onBlock(BlockBreakEvent e) {
if (Blocks.contains(block))  {
                 e.getPlayer().sendMessage("text");
                 e.getPlayer().damage(3);
                 e.setCancelled(true);
         }
            if (Items.contains(item)) {
            e.setCancelled(false);
            }

 

[Dymeth]

Чего?
Если хочешь завершить выполнение метода - используй return;

 

[clysic]

Чего?
Если хочешь завершить выполнение метода - используй return;

А при чём здесь return?
У меня есть два условия "if" в одном ивенте, где в первом условии: отменяется ивент, отправляется сообщение и игроку наносит урон; а вот во втором "if" ивент должен возобновиться, естественно без отправки сообщения игроку и урона, но почему-то отправляет...

 

[Dymeth]

А при чём здесь return?
У меня есть два условия "if" в одном ивенте, где в первом условии: отменяется ивент, отправляется сообщение и игроку наносит урон; а вот во втором "if" ивент должен возобновиться, естественно без отправки сообщения игроку и урона, но почему-то отправляет...

Что-то ты плохо изъясняешься.
Вот у тебя первое условие true - код внутри выполняется.
И второе условие true - код тоже выполняется.
Какое нафиг возобновление в линейном коде? )
Ты его не прерываешь, чтобы возобновлять.

 

[alexandrage]

Ты наверное перепутал if с else if внизу

 

[clysic]

Ты наверное перепутал if с else if внизу

Неь
Не робит

 

[hyndorik]

Ну попробуй на паскале тогда написать

 

[Ragnok123]

Уже то, что ты делаешь проверки для разных вещей одновременно (блок и предмет у игрока) сказывается тем, что не работает.
По твой логике, ты хочешь проверить можно ли сломать определенный блок определенным предметом, да?

 

[Luminate_]

Мда...
Опиши что тебе надо.
Если тебе надо отменить ломание блока при наличии в руке определенного предмета, то после проверки юзай e.setCancelled(true);
Напиши задачу нужную тебе.

 

[clysic]

В общем, пошёл немного иначе и добился,нужного мне,результата.
Всем спасибо за помощь!