Рендер tile entity

Rulet · 30 Июн 2022

Пытался сделать блоки декора, используя один класс рендера для всех блоков. В целом всё работает исправно, но ModelWrapperDisplayList работает только если назначить саму модель за пределами метода, то бишь как в учебнике, иначе оптимизация пропадает. Проблема в том, что если так сделать, то как мне указать имя своего блока в пути к модельке? ( tile.getBlockType().getLocalizedName() ). Я надеюсь, всё понятно написал.

Java:

public class DecorTileRender extends TileEntitySpecialRenderer {

    @Override
    public void renderTileEntityAt(TileEntity tile, double x, double y, double z, float f) {
        render((TileDecor)tile, x, y, z);
    }

    private void render(TileDecor tile, double x, double y, double z) {
        IModelCustom model = new ModelWrapperDisplayList((WavefrontObject) AdvancedModelLoader.loadModel(new ResourceLocation(Main.MODID, "models/decor/" + tile.getBlockType().getLocalizedName() + ".obj")));
            bindTexture(new ResourceLocation(Main.MODID, "models/decor/textures/" + tile.getBlockType().getLocalizedName() + ".png"));
                GL11.glPushMatrix();
                GL11.glTranslated(x, y, z);
                GL11.glTranslatef(0.5F, 0.0F, 0.5F);
                model.renderAll();
                GL11.glPopMatrix();
        }
    }

mts2200 · 30 Июн 2022

Java:

    FMLClientHandler.instance().getClient().getTextureManager().bindTexture(new ResourceLocation(Main.MOD_ID, "models/decor/" + block.getUnlocalizedName().replaceFirst("tile.", "") + ".obj"));

И к слову, нельзя в render делать AdvancedModelLoader#loadModel, у тебя таким образом каждый кадр будет пытаться подгрузиться твоя OBJ модель, из-за этого будут большие просадки FPS. Лучше ещё раз пройдись по теме: Ускорение рендера моделей

Rulet · 30 Июн 2022

mts2200 написал(а):
И к слову, нельзя в render делать AdvancedModelLoader#loadModel

Так в этом и суть вопроса: как сделать loadmodel и получить имя блока за пределами render.

Netrus · 30 Июн 2022

Выучить джаву

Dioxide_ · 1 Июл 2022

как вариант

Java:

IModelCustom model;
private void render(TileDecor tile, double x, double y, double z) {
        if(model == null) {
            model = new ModelWrapperDisplayList((WavefrontObject) AdvancedModelLoader.loadModel(new ResourceLocation(Main.MODID, "models/decor/" + tile.getBlockType().getLocalizedName() + ".obj"))); 
       }
                bindTexture(new ResourceLocation(Main.MODID, "models/decor/textures/" + tile.getBlockType().getLocalizedName() + ".png"));
                GL11.glPushMatrix();
                GL11.glTranslated(x, y, z);
                GL11.glTranslatef(0.5F, 0.0F, 0.5F);
                model.renderAll();
                GL11.glPopMatrix();
        }
}

Rulet · 1 Июл 2022

Тогда на все блоки накладывается одна модель

Kitoglav · 1 Июл 2022

ну ты можешь тип декора хранить как строку или интегер в своём TileDecor, а модели хранить в мапе. почитай как работает мемоизация, и по этому принципу легко сможешь черпать модели, не нагружая память

Rulet · 1 Июл 2022

Kitoglav написал(а):
ну ты можешь тип декора хранить как строку или интегер в своём TileDecor

Я пытался. NullPointerException

Kitoglav · 1 Июл 2022

а ты с клиентом синхронизировал?

PeTcHeNyShOhKa · 1 Июл 2022

Dioxide_ написал(а):

как вариант

Java:

IModelCustom model;
private void render(TileDecor tile, double x, double y, double z) {
        if(model == null) {
            model = new ModelWrapperDisplayList((WavefrontObject) AdvancedModelLoader.loadModel(new ResourceLocation(Main.MODID, "models/decor/" + tile.getBlockType().getLocalizedName() + ".obj")));
       }
                bindTexture(new ResourceLocation(Main.MODID, "models/decor/textures/" + tile.getBlockType().getLocalizedName() + ".png"));
                GL11.glPushMatrix();
                GL11.glTranslated(x, y, z);
                GL11.glTranslatef(0.5F, 0.0F, 0.5F);
                model.renderAll();
                GL11.glPopMatrix();
        }
}

Костыльный бред..... Лучше вызывать модель из ClientProxy.

mts2200 · 1 Июл 2022

Java:

@SideOnly(Side.CLIENT)
public class TileExampleRenderer extends TileEntitySpecialRenderer {
    private final int list;
    private final String name;

    public TileExampleRenderer(Block block) {
        this.name = block.getUnlocalizedName().replaceFirst("tile.", "");
        IModelCustom model = new ModelWrapperDisplayList((WavefrontObject) AdvancedModelLoader.loadModel(new ResourceLocation(Main.MOD_ID, "models/decor/" + name + ".obj")));
        list = GL11.glGenLists(1);
        GL11.glNewList(list, GL11.GL_COMPILE);
        model.renderAll();
        GL11.glEndList();
    }

    @Override
    public void renderTileEntityAt(TileEntity tile, double x, double y, double z, float renderPartialTicks) {
        FMLClientHandler.instance().getClient().getTextureManager().bindTexture(new ResourceLocation(Main.MOD_ID, "textures/decor/" + name + ".png"));
        GL11.glPushMatrix();
        GL11.glTranslated(x, y, z);
        GL11.glTranslatef(0.5F, 0.0F, 0.5F);
        GL11.glCallList(list);
        GL11.glPopMatrix();
    }
}

Ну и регистрация: ClientRegistry.bindTileEntitySpecialRenderer(TileEntityFurnace.class, new TileExampleRenderer(Blocks.furnace));

Dioxide_ · 1 Июл 2022

PeTcHeNyShOhKa написал(а):
Костыльный бред

Может быть это и костыль, но бредом как таковым он не является. Сам часто такое использую, очень удобно и быстро

Rulet · 1 Июл 2022

mts2200 написал(а):

Java:

@SideOnly(Side.CLIENT)
public class TileExampleRenderer extends TileEntitySpecialRenderer {
    private final int list;
    private final String name;

    public TileExampleRenderer(Block block) {
        this.name = block.getUnlocalizedName().replaceFirst("tile.", "");
        IModelCustom model = new ModelWrapperDisplayList((WavefrontObject) AdvancedModelLoader.loadModel(new ResourceLocation(Main.MOD_ID, "models/decor/" + name + ".obj")));
        list = GL11.glGenLists(1);
        GL11.glNewList(list, GL11.GL_COMPILE);
        model.renderAll();
        GL11.glEndList();
    }

    @Override
    public void renderTileEntityAt(TileEntity tile, double x, double y, double z, float renderPartialTicks) {
        FMLClientHandler.instance().getClient().getTextureManager().bindTexture(new ResourceLocation(Main.MOD_ID, "textures/decor/" + name + ".png"));
        GL11.glPushMatrix();
        GL11.glTranslated(x, y, z);
        GL11.glTranslatef(0.5F, 0.0F, 0.5F);
        GL11.glCallList(list);
        GL11.glPopMatrix();
    }
}

Ну и регистрация: ClientRegistry.bindTileEntitySpecialRenderer(TileEntityFurnace.class, new TileExampleRenderer(Blocks.furnace));

Одна модель применяется для всех блоков с этим кодом(

Kitoglav · 2 Июл 2022

Kitoglav написал(а):
ну ты можешь тип декора хранить как строку или интегер в своём TileDecor, а модели хранить в мапе. почитай как работает мемоизация, и по этому принципу легко сможешь черпать модели, не нагружая память

еще раз напомню. заводишь мапу <String, IModelCustom> и модель определяешь так:

Java:

public static final HashMap<String, ICustomModel> CACHE = new HashMap<>();

@Override
private void render(TileDecor tile, double x, double y, double z) {
    ICustomModel model = null;   
    String key = tile.getBlockType().getLocalizedName()
    if(!CACHE.contains(key)) {
        model = /**код создания модели, лень копипастить твой **/
        CACHE.put(key, model);
    } else {
        model = CACHE.get(key);
    }
    if(model != null) {
        //рендеришш
    }
}

без костылей, без проблем.

mts2200 · 2 Июл 2022

Боже, какой раз уже кидаю код для получения ключа имени блока, но всё равно во всех других ответах вижу:

Kitoglav написал(а):
tile.getBlockType().getLocalizedName()

Dioxide_ написал(а):
tile.getBlockType().getLocalizedName()

Rulet написал(а):
tile.getBlockType().getLocalizedName()

Этот метод возвращает локализированное имя блока, то есть, чтобы сделать адекватную работу с ним, то в ресурсах мода придётся делать название блока на КАЖДОМ языке, вопрос, серьёзно собираетесь? Будет весело, если вдруг захочется изменить локализированное название. А касательно этого:

Kitoglav написал(а):
public static final HashMap<String, ICustomModel> CACHE = new HashMap<>();

Лучше сделать интерфейс. Тем более, хранить в памяти ICustomModel неправильно, об этом рассказывается в теме: Ускорение рендера моделей

Выше я кидал код, который чутка переделать под интерфейс по типу:

Java:

import net.minecraft.util.ResourceLocation;

public interface IDecoTile {
    ResourceLocation getModelLocation();
    ResourceLocation getTextureLocation();
  
}

И всё будет нормально работать

TheAndrey · 2 Июл 2022

Kitoglav написал(а):
без костылей, без проблем.

С computeIfAbsent() будет поинтереснее. И откуда там null должен взяться?

Kitoglav · 4 Июл 2022

TheAndrey написал(а):
И откуда там null должен взяться?

не посмотрел как он объект ICustomModel создавал. думал через метод какой, мало ли null вернёт, а там конструктор оказывается. лишнее, соглы.

TheAndrey написал(а):
computeIfAbsent()

не приходилось пользоваться, но теперь видимо буду. за совет спасибо

Icosider · 4 Июл 2022

mts2200 написал(а):
Лучше сделать интерфейс. Тем более, хранить в памяти ICustomModel неправильно, об этом рассказывается в теме

А ничего что объект всё равно придётся создать и он будет храниться в памяти? В теме рассказывается немного о другом.

mts2200 написал(а):

Java:

@SideOnly(Side.CLIENT)
public class TileExampleRenderer extends TileEntitySpecialRenderer {
    private final int list;
    private final String name;

    public TileExampleRenderer(Block block) {
        this.name = block.getUnlocalizedName().replaceFirst("tile.", "");
        IModelCustom model = new ModelWrapperDisplayList((WavefrontObject) AdvancedModelLoader.loadModel(new ResourceLocation(Main.MOD_ID, "models/decor/" + name + ".obj")));
        list = GL11.glGenLists(1);
        GL11.glNewList(list, GL11.GL_COMPILE);
        model.renderAll();
        GL11.glEndList();
    }

    @Override
    public void renderTileEntityAt(TileEntity tile, double x, double y, double z, float renderPartialTicks) {
        FMLClientHandler.instance().getClient().getTextureManager().bindTexture(new ResourceLocation(Main.MOD_ID, "textures/decor/" + name + ".png"));
        GL11.glPushMatrix();
        GL11.glTranslated(x, y, z);
        GL11.glTranslatef(0.5F, 0.0F, 0.5F);
        GL11.glCallList(list);
        GL11.glPopMatrix();
    }
}

Ну и регистрация: ClientRegistry.bindTileEntitySpecialRenderer(TileEntityFurnace.class, new TileExampleRenderer(Blocks.furnace));

1. Для бинда текстур есть метод TileEntitySpecialRenderer#bindTexture(ResourceLocation)
2. Нет никакого смысла постоянно создавать новый объект ResourceLocation
3. Получение названия блока у тебя также неправильно, есть метод GameRegistry#findUniqueIdentifierFor. В случае, если надо получить название тайла есть такие перемененные: TileEntity#classToNameMap и TileEntity#nameToClassMap
4. Щепотка занудства: оба glTranslate можно объединить в один

Открытая Математика

Рендер tile entity

Rulet

Kitoglav

mts2200

Rulet

Netrus

Dioxide_

Rulet

Kitoglav

Rulet

Kitoglav

PeTcHeNyShOhKa

mts2200

Dioxide_

Rulet

Kitoglav

mts2200

TheAndrey

Kitoglav

Icosider

Kotliner