Если игрок соответствует устовиям

[SouthBridge]

А остановить цикл при телепорте ?

[ExLite]

[SRC="java"] if(command.equals("nuboloka8083"))
{
for(ItemInstance item : player.getInventory().getItems())
{
if(item == null)
continue;
if(item.getCrystalType() != null)
{
if(item.getCrystalType().externalOrdinal <= 3)
{
player.teleToClosestTown();
}
else
{
player.sendMessage("Вы не подходите условиям.");
player.sendActionFailed();
return;
}
}
}
}[/SRC]
Сделал так, все проверяет, но теперь телепортирует и пишет вы не подходите..
Естественно у меня в инвентаре дб, если нет ничего или Б грейд шмот, то просто телпортирует. как и должно быть, но почему то еще телепортирует если не подходу условиям.
Где теперь ошибка?

сейчас это работает как:
получаю все вещи
беру вещь #1
она пустая ? продолжаем
тип кристалов не пуст ? идем дальше
тип кристалов меньше или равен 3 ? ТП игрока

в итоге, когда в инвентаре будет хотя бы одна вещь B грейда или ниже, игрока портанет.
Вам нужно создать массив и вносить туда типы кристалов, и потом в проверке если массив содержит запрещенные кри - писать ошибку, не содержит - тп.

ну, или так, как описано ниже

[SouthBridge]

PHP код:

<?php 

for(ItemInstance item : player.getInventory().getItems())

        {

            if(item == null || item.getCrystalType() == null)

                continue;

            if(item.getCrystalType().externalOrdinal > 3)

            {

                player.sendMessage("Вы не подходите условиям.");

                player.sendActionFailed();

                return;

            }

        }

        player.teleToClosestTown();

        return; 

[Kenrix]

private static final int[] _грейдыВаси = { беГрейд, сеГрейд, ноуГрейд };

if(item.getCrystalType().externalOrdinal > _грейдыВаси)

[n3k0nation]

PHP код:

<?php 

for(ItemInstance item : player.getInventory().getItems())

        {

            if(item == null || item.getCrystalType() == null)

                continue;

            if(item.getCrystalType().externalOrdinal > 3)

            {

                player.sendMessage("Вы не подходите условиям.");

                player.sendActionFailed();

                return;

            }

        }

        player.teleToClosestTown();

        return; 

[Ashe]

Разве у игрока в инвентаре может лежать null? o_O l2j такой l2j.

Ты код l2j давно смотрел? Они там всё проверяют на null, вообще всё...

[ANZO]

Ты код l2j давно смотрел? Они там всё проверяют на null, вообще всё...

Наоборот было бы плохо, если бы где то проверяли а где то нет. А так можно спокойно перевести на Optional\Nullable имея все эти проверки во всех местах, где может произойти такой казус.