Не получается сравнить два ItemStack'а

[Terraformer5039]

Нужно сравнить предмет в OFFHAND с другим предметом, и если OFFHAND = предмет, то вернуть успешное выполнение.
Я попробовал много разных вариантов, но все же остановился на этом.
Проблема в том, что даже если предмет в левой руке не тот, или его вообще нет - выполнение все равно успешное.

public class BeefCan extends ItemFood implements IHasModel
{
    public BeefCan(String name, int amount, float saturation, boolean isWolfFood)
    {
        super(amount, saturation, isWolfFood);
        this.setRegistryName(name);
        this.setUnlocalizedName(name);
        this.setCreativeTab(CreativeTabs.FOOD);
        ItemsRegister.ITEMS.add(this);
    }
//проверка
    public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer playerIn) {
        ItemStack offhand = playerIn.getHeldItemOffhand();
        ItemStack mainhand = playerIn.getHeldItemMainhand();
        if (offhand!=(new ItemStack(ItemsRegister.OPENER))) {
        return new ActionResult<ItemStack>(EnumActionResult.FAIL, mainhand);
        } else {
        return new ActionResult<ItemStack>(EnumActionResult.SUCCESS, mainhand);
    } 
}
//-------
    @Override
    protected void onFoodEaten(ItemStack stack, World worldIn, EntityPlayer player)
    {  
        super.onFoodEaten(stack, worldIn, player);
        ItemStack emptycan = new ItemStack(ItemsRegister.EMPTY_CAN);
        player.addItemStackToInventory(emptycan);
    }
    @Override
    public void registerModels() {
        Main.proxy.registerItemRenderer(this, 0, "inventory");
    }
}

 

[hohserg]

В джава объекты надо сравнивать при помощи equals, а не ==,!=.
А стаки надо сравнивать при помощи ItemStack.areItemStacksEqual

 

[Terraformer5039]

В джава объекты надо сравнивать при помощи equal

Я знаю, но это работало только если убрать SUCCESS-результат.

А стаки надо сравнивать при помощи ItemStack.areItemStacksEqual

Попробую, спасибо.

 

[Eifel]

Ну или речь идет просто про предметы, то можно через stack.getItem() != ItemsRegister.OPENER, но перед этим лучше проверить, что !stack.isEmpty()

 

[hohserg]

Я знаю, но это работало только если убрать SUCCESS-результат.

А ты попробуй выполнить new ItemStack(Items.APPLE) == new ItemStack(Items.APPLE) и посмотри чему равен результат.
ActionResult тут не при чем

@Eifel, ага, можно юзать, если не важны нбт и мета. Проверять на пустоту не обязательно

 

[Terraformer5039]

Всем спасибо за ответы, сейчас испытаю.
_
Оффтоп: почему у меня нет кнопки "Ответить" под сообщениями hohserg1?

 

[Terraformer5039]

Не работает

if (ItemStack.areItemStacksEqual(offhand, new ItemStack(ItemsRegister.OPENER)))
    return new ActionResult<ItemStack>(EnumActionResult.FAIL, mainhand);
    } else {
    return new ActionResult<ItemStack>(EnumActionResult.SUCCESS, mainhand);
    }

 

[BlesseNtumble]

offhand.isItemEqual(new ItemStack(ItemsRegister.OPENER))
пробуй

 

[Terraformer5039]

Тоже не работает, к сожалению

 

[hohserg]

Подебажь, чему равен offhand? Может, количество не совпадает?

 

[Terraformer5039]

Запустил с дебагом (с помощью жука), в игре ничего не поменялось (F3), в логи ничего не добавилось, в IDE ничего не изменилось.

 

[hohserg]

Погугли "как пользоваться дебагом". Попробуй совет Ейфеля, #4

 

[Terraformer5039]

Я попробовал одну вещь.
Заккоментил все, кроме ItemStack mainhand и return new.... FAIL
Получилось вот так:

     public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer playerIn) {
        //ItemStack offhand = playerIn.getHeldItemOffhand();
        ItemStack mainhand = playerIn.getHeldItemMainhand();
        //if (ItemStack.areItemStacksEqual(offhand, new ItemStack(ItemsRegister.OPENER, 1))) {
        //return new ActionResult<ItemStack>(EnumActionResult.SUCCESS, mainhand);
        //} else {
        return new ActionResult<ItemStack>(EnumActionResult.FAIL, mainhand);
        //}

Однако, еду все равно есть можно. Но я ведь по сути, запрещаю это делать, или нет?

 

[Terraformer5039]

Погугли "как пользоваться дебагом"

Забыл про брейкпоинты..

 

[Terraformer5039]

В eclipse у меня было выключено окно дебага.
После включения, дебаг мне ничем не помог. Там был список брейкпоинтов, запущенные потоки (или что-то другое), Variables и Extensions. В последних двух было пусто (

 

[Eifel]

Е-е-е, погоди, в методе #onItemRightClick другие параметры. Он у тебя просто не переопределен, поэтому ничего не имеет результата. Для справки:
public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer playerIn, EnumHand handIn)

 

[hohserg]

Ох, точно! Чтобы так не ошибаться позволяй ide генерировать override-ы. Idea, например при написании названия метода в теле класса предлагает в автодополнении переопределения

 

[will0376]

Про сравнение. equals да, может помочь. Но в качестве дебага - попробуй получить отдельно мету и отдельно айтем и сравнивать уже их.

 

[Terraformer5039]

Спасибо, сегодня попробую

 

[Стаканчик]

Вот типа так как то:

        ItemStack stack = player.getHeldItemMainhand();
        ItemStack stack2 = player.getHeldItemOffhand();
         if(!(stack2.getItem() == ItemsRegister.NameYourClass)) {
             //действие когда предмет в offHand
         }
            if(!(stack.getItem() == ItemsRegister.NameYourClass)) {
            //действие когда предмет в mainHand
        }

.
Работает, сам пользуюсь