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

[L2J]

Вот код:

if(command.equalsIgnoreCase("nuboloka8083"))
            {
                player.sendMessage("nuboloka8083");
                ItemInstance item = player.getInventory().getItemByItemId(_objectId);
                int crys = item.getTemplate().getCrystalType().externalOrdinal;
                        if(crys == 5 && item.getEnchantLevel() >= 6)
                        {
                            sendMessage("Это локация только для Б грейда. Уровень заточки не больше 6.");
                        }
                        if(crys == 4 && item.getEnchantLevel() >= 6)
                        {
                            sendMessage("Это локация только для Б грейда. Уровень заточки не больше 6.");
                        }
                        else if(crys <= 3 && item.getEnchantLevel() <= 6 || item == null)
                        {
                            player.sendMessage("Вы подходите условиям и телепортированы.");
                            player.teleToClosestTown();
                        }
                player.sendMessage("nuboloka8083konec");
             }
Code: Java

Подскажите пожалуйста, почему когда я нажимаю на кнопку nuboloka8083, то в чат выходит только сообщение nuboloka8083?
Задумка была такова: Если у чара Б шмот и оружие, в инвентаре или на себе одето, то можно тп, а если А и S, то нельзя тп.
В чем ошибка?

[xolseg]

Я не программист, но... разве так может быть?
сначала if, потом опять if, потом else if?
А не так, if; else if; else?

Погуглил инфу, мой бессмысленный пост.(

[Kenrix]

if (crys == 5 && item.getEnchantLevel() >= 6 && !грейд_недопустимых_к_тп)
{
teleportЁпте());
}
else
{
sendMessage("Это локация только для Б грейда. Уровень заточки не больше 6.");
}

[Main]

               
if(crys <= 3 && item.getEnchantLevel() <= 6 || item == null)
     {
          player.sendMessage("Вы подходите условиям и телепортированы.");
          player.teleToClosestTown();
     }
else
     sendMessage("Это локация только для Б грейда. Уровень заточки не больше 6.");

Упс, не заметил, пост выше правильно написан.

[ExLite]

в двух if нет ничего страшного, другое дело это то, что они проверяют.
Вы проверяете все что нельзя ( хорошо что только A и S не подходят, а если бы их было больше ?), можно ведь сделать 1 проверку на то что разрешено, а все остальное пустить по else ( как выше в постах ).

по теме: почему не доходит до "nuboloka8083konec" - скорее всего где то ошибка.
Попробуйте так и посмотрите какие значения принимают переменные:

ItemInstance item = player.getInventory().getItemByItemId(_objectId);
player.sendMessage("Item: " +item.getEnchantLevel());
int crys = item.getTemplate().getCrystalType().externalOrdinal;
player.sendMessage("CrysID: " +crys);

[L2J]

if(command.equalsIgnoreCase("nuboloka8083"))
            {
                player.sendMessage("nuboloka8083");
                ItemInstance item = player.getInventory().getItemByItemId(_objectId);
                player.sendMessage("Item: " + item.getEnchantLevel());
                int crys = item.getTemplate().getCrystalType().externalOrdinal;
                player.sendMessage("CrysID: " + crys);
                if(crys <= 3 && item.getEnchantLevel() <= 6 || item == null)
                {
                    player.sendMessage("Вы походите условиям и телепортированы.");
                    player.teleToClosestTown();
                }
                else
                {
                    player.sendMessage("Эта локация только для персонажей в Б экипировке и уровень заточки не должен быть больше 6.");
                }
                player.sendMessage("nuboloka8083konec");
             }
Code: Java

Сделал так, в чат все равно выводит nuboloka8083 и не телепортирует.
Скорее все ошибка в этом:

 ItemInstance item = player.getInventory().getItemByItemId(_objectId);
int crys = item.getTemplate().getCrystalType().externalOrdinal;
Code: Java

Но как это исправить я не знаю, помогите пожалуйста.

[ExLite]

ошибка тут:

ItemInstance item = player.getInventory().getItemByItemId(_objectId);

раз не доходит до player.sendMessage("Item: " + item.getEnchantLevel());

_objectId откуда берется ? может он = null ?

[L2J]

А как по другому брать предмет из инвентаря?

[SouthBridge]

PHP код:

getItemByItemId(_objectId) 


Вы уж определитесь. Либо по ID выборку делаете, либо по OID

PHP код:

item.getEnchantLevel() <= 6 || item == null) 


Так же следует правильно проверять на null и не глушить исключения и все будет тип-топ.
Использование отладчика тоже не помешает.

А как по другому брать предмет из инвентаря?

Загляните в исходник Inventory, ответ не заставит себя долго ждать.

[L2J]

  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;
                    }
                    }
                }
             }
Code: Java

Сделал так, все проверяет, но теперь телепортирует и пишет вы не подходите..
Естественно у меня в инвентаре дб, если нет ничего или Б грейд шмот, то просто телпортирует. как и должно быть, но почему то еще телепортирует если не подходу условиям.
Где теперь ошибка?