ActionResult в 1.12.2

[Garik]

Блин! Нужен ли ActionResult в 1.12.2?

Я сделал метательное копьё у которого есть "ActionResult" и думаю надо ли оно для него?

Просто вот это глаза мозолит:

    @Override
    public ActionResult<ItemStack> onItemRightClick(World par2World, final EntityPlayer par3EntityPlayer, EnumHand hand) {

        par3EntityPlayer.setActiveHand(hand);

        ItemStack par1ItemStack = par3EntityPlayer.getHeldItem(hand);


        return new ActionResult(EnumActionResult.SUCCESS, par1ItemStack);

    }

И оно в принципе не такое уж и нужное наверное, раз я предметы с ActionResult негде не видел.

 

[tox1cozZ]

Эм, ну как бы нужен. Меня бы больше null раздражал. Да и краш будет, полюбас.

 

[Могучий горгон]

Если EnumActionResult равен SUCCESS, то действие удалось. Если PASS - то в процессе, или возможно удалось. FAIL - не удалось.
Например если вернешь в методе райтклика итема SUCCESS, то майн не будет искать блок, на который ты гипотетически мог бы кликнуть, если бы у тебя в руках не было предмета. Это пример, на самом деле майн после того как проверит EnumActionResult правой руки начнет проверять левую, потом только блок. Если вернешь FAIL то у тебя клик и на итеме и по блоку пройдет, вернешь PASS - у тебя вызовется что-то типа onItemUseFinish(getMaxItemUseDuration еще можно указать), onPlayerStoppedUsing и т.д.

 

[Garik]

Меня бы больше null раздражал.

Зато при нуле в еклипсе жёлтым не выделяется

Да и краш будет, полюбас.

Да вроде бы и нет.

 

[tox1cozZ]

У меня не выделяет ничего... Что он говорит?

 

[Могучий горгон]

Наверное он скачал эклипс для слабовидящих))

 

[Garik]

Что он говорит?

Type safety: The expression of type ActionResult needs unchecked conversion to conform to ActionResult<ItemStack>

(Наверное очередной баг Затмения)

 

[tox1cozZ]

Какой блин баг, все верно. У тебя же дженерика нет...

new ActionResult<ItemStack>

 

[Могучий горгон]

А лол, кстати да. Как-то я криво варнинг прочитал

 

[Garik]

У тебя же дженерика нет...

Ой... Ой... Верно же! Я на это как-то внимания даже не обратил! (Хотя оно и без него прекрасно работало)

 

[Могучий горгон]

Что-то я не очень понял, какой дженерик для нулла? У тебя же подсвечивается когда ты null возвращаешь ?

 

[Garik]

Оно по идей вот так должно быть:

 

[tox1cozZ]

Нет, когда он возвращал ActionResult без дженерика.

 

[Могучий горгон]

Все равно думаю эклипс можно так настроить, чтобы выкидывало предупреждение или даже ошибку когда пытаешься вернуть null не в nullable методе. Хотя это довольно бесполезно, т.к. эти аннотации почти никто не ставит.

 

[Garik]

Хотя это довольно бесполезно

+

 

[ロロロ]

А еще можно сделать так..
return ActionResult.newResult(EnumActionResult.PASS, playerIn.getHeldItem(handIn));

 

[Garik]

А зачем так делать?

 

[tox1cozZ]

Альтернативный вариант. Это тоже самое.