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

[L2J]

Вот код:
[SRC="java"]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");
}[/SRC]
Подскажите пожалуйста, почему когда я нажимаю на кнопку 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]

[SRC="java"]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");
}[/SRC]
Сделал так, в чат все равно выводит nuboloka8083 и не телепортирует.
Скорее все ошибка в этом:
[SRC="java"] ItemInstance item = player.getInventory().getItemByItemId(_objectId);
int crys = item.getTemplate().getCrystalType().externalOrdinal;[/SRC]
Но как это исправить я не знаю, помогите пожалуйста.

[ExLite]

ошибка тут:

Код:

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

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

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

[L2J]

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

[SouthBridge]

PHP код:

<?php 

getItemByItemId(_objectId) 

[L2J]

[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]
Сделал так, все проверяет, но теперь телепортирует и пишет вы не подходите..
Естественно у меня в инвентаре дб, если нет ничего или Б грейд шмот, то просто телпортирует. как и должно быть, но почему то еще телепортирует если не подходу условиям.
Где теперь ошибка?