Зацикленное отображение GuiScreen

[Unky]

Здравствуйте, начал писать мод на gui ПДА, пошел искать туториалы по отрисовке изображений, сделал все как по туториалу, в туториале изображение только одно, а у меня вот это.

Вот код:

package by.Unky.pda;

import org.lwjgl.opengl.GL11;

import net.minecraft.client.gui.GuiScreen;
import net.minecraft.util.ResourceLocation;

public class pdaGui extends GuiScreen {

    int guiWidth = 990;
    int guiHeight = 650;

    //Minecraft mc = Minecraft.getMinecraft();
    
    @Override
    public void drawScreen(int x, int y, float ticks){
        
        int guiX = (width - guiWidth);
        int guiY = (height - guiHeight);
        GL11.glColor4f(1, 1, 1, 1);
        drawDefaultBackground();
        mc.renderEngine.bindTexture(new ResourceLocation(pdamod.MODID, "textures/guipda/guiPda.png"));
        drawTexturedModalRect(guiX, guiY, 0, 0, guiWidth, guiHeight);
        super.drawScreen(x, y, ticks);
    }

Если кто то знает как решить проблему, то дайте подсказку пожалуйста

 

[fukkivdan]

int xOrigin = (width - xSize) / 2;
int yOrigin = (height - ySize) / 2;
drawTexturedModalRect(xOrigin, yOrigin, 0, 0, xSize, ySize);

 

[tox1cozZ]

У него картинка не 256х256, надо юзать метод из Gui и передавать кастомный размер

 

[rogue_one]

mc.renderEngine.bindTexture(new ResourceLocation(pdamod.MODID, "textures/guipda/guiPda.png"));

Вроде так делать не хорошо, лучше создай переменную РесурсЛокейшена.

Лично я делаю так

GL11.glPushMatrix();
this.mc.renderEngine.bindTexture(твойРесурслокейшен);
GL11.glTranslatef(super.width / 2 - 190, super.height / 2 - 145, 0.0F);
GL11.glScalef(1.0F,  1.0F, 1.0F);
// par1 - x положение на экране, par2 - аналогично y, par3 и  par4 - координаты на текстуре. par5 и par6 - размер области на текстуре
this.drawTexturedModalRect(par1, par2, par3, par4, par5, par6);
GL11.glPopMatrix();

 

[Icosider]

Вроде так делать не хорошо, лучше создай переменную РесурсЛокейшена.

Да, так как каждый кадр создаётся новый объект класса, и название должно быть не resourseLocation.

 

[rogue_one]

и название должно быть не resourseLocation.

Поменял чтобы было понятно

 

[Icosider]

Ты не понял, ну да ладно xD

 

[rogue_one]

Ты не понял, ну да ладно xD

Ну тада прошу обьяснить и извниняюсь за тупость)

 

[fukkivdan]

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

 

[hohserg]

Один объект в кадр ниче не испортит, на него быстро потеряется ссылка и сборщик мусора очистит

 

[fukkivdan]

визуально удобнее, сразу видишь, какую конкретно текстуру биндишь

 

[tox1cozZ]

Один объект в кадр ниче не испортит, на него быстро потеряется ссылка и сборщик мусора очистит

Где один - там и второй. Вообще создавать объекты в рендере считаю дурным тоном.

 

[hohserg]

Верно. Однако на 1.12 создание нового экземпляра модели предмета каждый кадр не влияет на фпс(по крайней мере я не заметил), а каждая модель тянет за собой много экземпляров BakedQuad

 

[fukkivdan]

создание нового экземпляра модели предмета каждый кадр не влияет на фпс

если у тебя модель не 20к трисов, то может быть