Ошибка в обработке onBlockActivated

[VeniVidiVici]

Преамбула

У меня есть блок "А".

К этому блоку "А" привязано два GUI, контейнер у них одинаковый (один и тот же).

Первое gui меняет значения int или boolean в TileEntity этого блока с отправкой пакетов (по отдельности происходит).
Второе gui показывает текущее состояние этих int и boolean переменных.

Я беру предмет (могут быть совершенно разные), нажимаю им ПКМ по этому блоку, предмет добавляется в блок "Б" (эти gui и контейнеры блок "Б" не трогают), а также у меня пропадает этот предмет.

Фабула

Если я меняю значение переменной в блоке "А" (в gui #1), а потом нажимаю этим предметом ПКМ на блок, у меня вылетает краш, который я прилагаю ниже.

Однако, если я меняю значение в блоке "А" (в gui #1), а перед использованием предмета открываю gui #2 (с обычным просмотром, пустой рукой), либо перезахожу из меню, либо открываю какое-либо другое gui, то последующее использование предмета проходит штатно.

Ошибка возникает непосредственно из-за player.inventory.consumeInventoryItem(item). Без него всё работает, со сменой значений и без. Но, чёрт возьми, не понимаю почему такая шляпа.

Соль в том, что в краше логированы значения математики, выполняемые перед consumeInventoryItem, в них все значения новые (после изменений, значит пакеты проходят).

На форуме почти ничего не нашёл. Есть вот такая тема, но там что-то другое.

@Override
public boolean onBlockActivated(World world, int x, int y, int z, EntityPlayer player, int integ, float i, float u, float j)
{
    if (world.isRemote)
        return true;
    else
    {
        /** Тут ещё разные условия и т.д. **/
        if (MoneyMath.storageCoins(factory.storage) >= cost)
        {
            tileStorage.addContents(k, new ItemStack(item, 1));
            MoneyMath.addCoins(player, cost);
            MoneyMath.factorySpend(factory.storage, cost);
            player.inventory.consumeInventoryItem(item); /** Вот тут проблема **/
            player.inventoryContainer.detectAndSendChanges();
            return true;
        }
        /** Дальше другой код**/
    }
    return false;
}

 

[VeniVidiVici]

Закостылил проблему переоткрытием GUI №2 после закрытия GUI №1. Но если кто-то знает, как решить вопрос не индокодингом, буду признателен.

 

[timaxa007]

как решить вопрос

Думаю мало кто сможет, учитывая маленький код.

/** Вот тут проблема **/

Caused by: java.lang.NullPointerException
at net.minecraft.network.NetHandlerPlayServer.processPlayerBlockPlacement(NetHandlerPlayServer.java:657) ~[NetHandlerPlayServer.class:?]

Ты каким-то образом ломаешь пакет. Тут всякое может быть. Не зарегистрированный твой пакет, не правильный id у твоего пакета, может твой Handler пакетов не правильно написан или не правильно используется какой-то Hanlder пакетов.

player.inventory.consumeInventoryItem(item); /** Вот тут проблема **/

И мы должны быть уверены, что item это наследник Item, а не null.

return false;

Думаю, лучше будет все return делать true.

 

[VeniVidiVici]

Думаю, лучше будет все return делать true.

Всяко пробовал

И мы должны быть уверены, что item это наследник Item, а не null.

Точно не null, чуть выше есть другой метод, который успешно этот item обрабатывает.

Ты каким-то образом ломаешь пакет. Тут всякое может быть. Не зарегистрированный твой пакет, не правильный id у твоего пакета, может твой Handler пакетов не правильно написан или не правильно используется какой-то Hanlder пакетов.

Пакетов там уже много, по накатанной, вроде как все всегда работали

В любом случае, спасибо за советы