Странная ошибка в Screen

[Coriin]

Уже добавлял в мод меню и экраны, но в этот раз выдаёт странную ошибку, причём эта ошибка возникает не всегда, и экран может открыться. Майнкрафт жалуется на то что поле Screen.minecraft является null и поэтому он не может получить нарратора, подскажите в чём проблема

java.lang.NullPointerException: Cannot invoke "net.minecraft.client.Minecraft.getNarrator()" because "this.minecraft" is null

 

[ATRIK]

Странно :/ В любом случае ты всегда можешь получить Minecraft вот так: Minecraft.getInstance()

 

[Coriin]

Я знаю, уже в паранойе натыкал повсюду this.minecraft = Minecraft.getInstance() и в init и в конструкторе, даже при отрисовке пытался, но это ошибку не решает

 

[ATRIK]

Я вообще всегда вместо this.minecraft использую Minecraft.getInstance()

 

[Coriin]

Да и я так делаю, но есть проблема this.minecraft используется в приватном методе shouldRunNarration. Я же не могу поменять приватный метод

 

[_SupCM_]

Уже добавлял в мод меню и экраны, но в этот раз выдаёт странную ошибку, причём эта ошибка возникает не всегда, и экран может открыться. Майнкрафт жалуется на то что поле Screen.minecraft является null и поэтому он не может получить нарратора, подскажите в чём проблема

java.lang.NullPointerException: Cannot invoke "net.minecraft.client.Minecraft.getNarrator()" because "this.minecraft" is null

Покажи часть, где Screen открываешь.

 

[Coriin]

    @Override
    public InteractionResultHolder<ItemStack> use(Level pLevel, Player pPlayer, InteractionHand pUsedHand) {
        if(!pLevel.isClientSide){
            if(Minecraft.getInstance().player.isShiftKeyDown()){
                NetworkHooks.openScreen((ServerPlayer) pPlayer, ((ChromaticLexicon)pPlayer.getItemInHand(pUsedHand).getItem()));
            }
            else { Minecraft.getInstance().setScreen(new LexiconMainPageScreen(LexiconTitle)); }
        }
        pPlayer.getCooldowns().addCooldown(this, 20);

        return super.use(pLevel, pPlayer, pUsedHand);
    }

 

[ATRIK]

А когда вызывается shouldRunNarration? Думаю с этого стоит начать. Потому что если этот метод вызывается в конструкторе у Screen, то хоть как не открывай оно будет ломаться(если в конструкторе не присваивается значение minecraft)

 

[Coriin]

Пробовал сейчас несколько способов решения проблемы:
1. добавить в конструктор this.minecraft = Minecraft.getInstance()
после этого майнкрафт вылетел и пожаловался, что font is null, я добавил в констрктор ещё this.font = Minecraft.getInstance().font;
после этого игра перестала вылетать, но этот экран стал некорректно открываться, а именно поверх него накладываются несколько пустых слоёв и, пока не закроешь 2-5 пустых экранов нужный не видно, иногда эти экраны не полностью пустые, а являются копиями исходного без заднего фона

2. сделать @Override всех методов где вызывается shouldRunNarration

    @Override
    public void handleDelayedNarration() {

    }

    @Override
    public void triggerImmediateNarration(boolean pOnlyNarrateNew) {

    }

после этого майнкрафт перестал корректно отрисовывать экран, как и в первом случае и стал вылетать при закрытии экрана, это удалось починить, переписав метод onClose, используя вместо Screen.minecraft Minecraft.getInstance()

P.S. Посмотрел в логе: пустые экраны появляются, потому что RenderSystem, по только богу известной причине, пытается вызваться в серверного потока, вместо рендер потока

P.S P.S ну конечно оно вызываетcя на сервере я же дурак даже проверку поставил в методе use. В общем я переделал метод в это:

    @Override
    public InteractionResultHolder<ItemStack> use(Level pLevel, Player pPlayer, InteractionHand pUsedHand) {
        if(!pLevel.isClientSide && Minecraft.getInstance().player.isShiftKeyDown()){
            NetworkHooks.openScreen((ServerPlayer) pPlayer, ((ChromaticLexicon)pPlayer.getItemInHand(pUsedHand).getItem()));
        } else { Minecraft.getInstance().setScreen(new LexiconMainPageScreen(LexiconTitle)); }
        pPlayer.getCooldowns().addCooldown(this, 20);

        return super.use(pLevel, pPlayer, pUsedHand);
    }

И теперь всё работает, вроде как

P.S. P.S. P.S. Я проверил, все проблемы с полем Screen.minecraft были следствием вызова клиентского экрана на серверной стороне

 

[_SupCM_]

(1):

!pLevel.isClientSide && Minecraft.getInstance().player.isShiftKeyDown()

(2):

Minecraft.getInstance().setScreen(new LexiconMainPageScreen(LexiconTitle));

Ага, а теперь запусти в среде сервер, и смотри как он крашится (даже не заходя на него).

По-хорошему, для случая (1) тебе нужно проверять pPLayer на isSneaking(), для случая (2) нужен пакет.
Всё то, что ты делал для решения - попытка вылечить симптомы, а не болезнь. Потому их даже костылём не назовёшь.

 

[Coriin]

Может я туплю, но я не совсем понимаю зачем нужен пакет в случае пользовательского интерфейса

 

[_SupCM_]

зачем нужен пакет в случае пользовательского интерфейса

Потому что Item#use существует и на стороне сервера -> сервер пытается посмотреть этот метод -> в методе где-то есть клиентский пакет (тот же класс Minecraft, через который ты открываешь Screen) -> серверу плохо.
Для того, чтобы такого не было, нужен пакет.

UPD:

pPlayer.getCooldowns().addCooldown(this, 20);

У меня вызывает сомнение, зачем это.

 

[Coriin]

Я сделал открытие экрана через пакеты, спасибо что объяснил. Перезарядку на предмет добавил в качестве эксперимента, просто чтобы попробовать сделать предмет с перезарядкой использования