Снятие HAIR если сняли DHAIR

[L2J]

Как сделать чтобы при снятии нижнего аксессуара снимался и верхний?
Сделал в ItemFunctions

        //СВОЙСТВА
        if(item.getTemplate().isSA())
        {
            //СА можно ностиь только с акссесуаром
            if(!player.getOpenSA())
                return new SystemMessage2(SystemMsg.YOU_CANT_EQP_SA);
            if(player.getOpenSA())
                player.sendPacket(new SystemMessage2(SystemMsg.YOU_CAN_EQP_SA));
            //Если слот закрыт, снимает СА.
            if(!player.getOpenSA() && player.getInventory().setPaperdollItem(Inventory.PAPERDOLL_DHAIR, null) != null)
                return new SystemMessage2(SystemMsg.YOU_CANT_EQP_SA);
        }
Code: Java

Но не работает, точнее работает, но часть, 1 и 2 проверка работает, а 3 нет, или не так как мне надо.

[xTz]

if(!player.getOpenSA())
return new SystemMessage2(SystemMsg.YOU_CANT_EQP_SA);
тут ретурн так какого дальше идет проверка if(!player.getOpenSA() ?

[L2J]

Разве не может быть несколько ретурнов?
Вот например код не мой:

// Плащи
		if(item.getTemplate().isCloak())
		{
			// Can be worn by Knights or higher ranks who own castle
			if(item.getName().contains("Knight") && (player.getPledgeClass() < Player.RANK_KNIGHT || player.getCastle() == null))
				return new SystemMessage2(SystemMsg.YOU_DO_NOT_MEET_THE_REQUIRED_CONDITION_TO_EQUIP_THAT_ITEM);
 
			// Плащи для камаэлей
			if(item.getName().contains("Kamael") && player.getRace() != Race.kamael)
				return new SystemMessage2(SystemMsg.YOU_DO_NOT_MEET_THE_REQUIRED_CONDITION_TO_EQUIP_THAT_ITEM);
 
			// Плащи можно носить только с S80 или S84 сетом
			if(!player.getOpenCloak())
				return new SystemMessage2(SystemMsg.THE_CLOAK_CANNOT_BE_EQUIPPED_BECAUSE_A_NECESSARY_ITEM_IS_NOT_EQUIPPED);
		}
Code: Java

[xTz]

Ретурнов может быть много но одну и тужуе проверк сувать в ретурн и после ретурна немного туповато ибо она всегда в ретурн будет уходить.

Добавлено через 2 минуты
либо последнюю проверку подымите над первой внутри ифа тогда будет логичнее

[L2J]

Проверка не одна и та же, if(!player.getOpenSA())
return new SystemMessage2(SystemMsg.YOU_CANT_EQP_SA);
Вторая
if(player.getOpenSA())
player.sendPacket(new SystemMessage2(SystemMsg.YOU_CAN_EQP_SA));

Добавлено через 9 минут
Может нужно делать проверку не в ItemFunctions?

Добавлено через 21 минуту
Действительно, сделал через Inventory

case ItemTemplate.SLOT_DHAIR:
				pdollSlot = PAPERDOLL_DHAIR;
                setPaperdollItem(Inventory.PAPERDOLL_HAIR, null);
				break;
Code: Java

[Donatte]

Сообщение от L2J

Действительно, сделал через Inventory

Так этот класс и предназначен исключительно для этого, и полностью отвечает за обработку всех событий со слотами в инвентаре.