Jump to content

Recommended Posts

Posted (edited)

So I need to create a lookup table of several of my custom entities, via the EntityType members of my entity registry. The entities are successfully registered;

 

    public static final EntityType<DemonFangEntity> DEMON_FANG = registerEntity(EntityType.Builder.create(DemonFangEntity::new, EntityClassification.MISC).size(0.7F, 0.7F), "demon_fang");

 

and when I need an entity to spawn I simply call the following method where I try to return an entity via a string key. In this case, I'd need this to return a DemonFangEntity object (it extends WithernautsMagicEntity):

 

    public WithernautsMagicEntity getMagicEntity(World worldIn, String key)
    {
        try {
            for (Field f : EntityRegistry.class.getDeclaredFields())
            {
                Object obj = f.get(null);
                if (obj instanceof EntityType)
                {
                    EntityType type = (EntityType)obj;
                    Entity entity = type.create(worldIn);
                    if (entity instanceof WithernautsMagicEntity)
                    {
                        WithernautsMagicEntity magicEntity = (WithernautsMagicEntity)entity;
                        if (magicEntity.getKey().equals(key))
                            return magicEntity;
                    }
                }
            }
            throw new IllegalArgumentException();
        } catch (IllegalAccessException e)
        {
            throw new RuntimeException(e);
        }
    }

 

However, this method currently throws an exception because Entity entity = type.create(worldIn) returns null. What's a way I could correctly return an object of EntityType's template type? Or is this just not possible without reflection?

 

Sidenote: I'm trying to do it this way because I don't want to create a giant conditional statement since it would not scale well at all with my project's scope.

Edited by Turtledove
Posted

Okay...

first thing wrong is with your reflection code: you are passing null into the .get() method, that's not how it works.

you need to pass it an instance of the Object that you got the Field from, in this case you need the instance of EntityRegistry.

 

But there's a way better way of accomplishing what you want, you can just get the Deffered Register from your EntityRegistry class, and it has a method called .getEntries(), I think it's pretty self explanatory what it does.

 

This should solve your problem and drastically simplify the code

Posted
1 hour ago, Turtledove said:

Sidenote: I'm trying to do it this way because I don't want to create a giant conditional statement since it would not scale well at all with my project's scope.

I don't see in which scenario not using reflection would result in a giant conditional statement

 

1 hour ago, Turtledove said:

and when I need an entity to spawn I simply call the following method where I try to return an entity via a string key.

Trying to get things by string is very error-prone, you should at a minimun store the Entity Id String in your EntityRegistry as a static final, and then use it when needed

Posted (edited)
30 minutes ago, kiou.23 said:

I don't see in which scenario not using reflection would result in a giant conditional statement

 

Trying to get things by string is very error-prone, you should at a minimun store the Entity Id String in your EntityRegistry as a static final, and then use it when needed

Ugly conditionals, as in:

switch key:
	case 'blah':
		return BlahEntity object
	case 'foo':
		return FooEntity object
	case 'bar':
		return BarEntity object
	.
	.
	.
	case 'blah':
		return BlahEntity object

Where I'd need to add to this whenever I add new relevant Entities. This is what I'm avoiding.

 

35 minutes ago, kiou.23 said:

Okay...

first thing wrong is with your reflection code: you are passing null into the .get() method, that's not how it works.

you need to pass it an instance of the Object that you got the Field from, in this case you need the instance of EntityRegistry.

 

This part works just fine, we pass null to field.get() in this case because the members it's looking for are static. It correctly retreives the EntityTypes, the problem is that I don't know how to get an Entity object out of it.

 

Edited by Turtledove
Posted

Howdy

 

 

I'm not sure I understand the problem you're facing.

 

Because you control the creation of your own types, you should be able to create a map of the entity type and its string name when you create them, and when you get a string name, use that map to decide if it's one of your entities and then spawn it.  Your registered entity type has the factory in it already.

 

Using reflection to search your own registry class seems like a very strange idea to me.

 

I also suggest not to use static initialisers for your entity types; either used DeferredRegistry or use an event, like this

https://github.com/TheGreyGhost/MinecraftByExample/blob/master/src/main/java/minecraftbyexample/mbe81_entity_projectile/StartupCommon.java

 

Cheers

  TGG

 

 

 

 

 

Posted
22 minutes ago, Turtledove said:

It correctly retreives the EntityTypes, the problem is that I don't know how to get an Entity object out of it.

You can use EntityType.create(), this returns a new Entity

 

23 minutes ago, Turtledove said:

This part works just fine, we pass null to field.get() in this case because the members it's looking for are static.

Oh yeah, those are static, my bad. It's that I just spended the whole day doing reflection and getting non static values

Posted (edited)
27 minutes ago, Turtledove said:

Ugly conditionals, as in:


switch key:
	case 'blah':
		return BlahEntity object
	case 'foo':
		return FooEntity object
	case 'bar':
		return BarEntity object
	.
	.
	.
	case 'blah':
		return BlahEntity object

Where I'd need to add to this whenever I add new relevant Entities. This is what I'm avoiding.

Can't you make the method generic? make it receive a EntityType<T> and then return the entity from it

It also seems very redundant, why do you need a method for this, isn't it easier to call EntityRegistry.RANDOM_ENTITY_TYPE.get().create()?

 

I also don't get why you are using reflections. I suppose you're using Deferred Registries, and the Deferred Regsitry has a method that returns all entries. And if you're not using Deferred Registries, you should

Edited by kiou.23

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Hello, I have this same problem. Did you manage to find a solution? Any help would be appreciated. Thanks.
    • log: https://mclo.gs/QJg3wYX as stated in the title, my game freezes upon loading into the server after i used a far-away waystone in it. The modpack i'm using is better minecraft V18. Issue only comes up in this specific server, singleplayer and other servers are A-okay. i've already experimented with removing possible culprits like modernfix and various others to no effect. i've also attempted a full reinstall of the modpack profile. Issue occurs shortly after the 'cancel' button dissapears on the 'loading world' section of the loading screen.   thanks in advance.
    • You would have better results asking a more specific question. What have you done? What exactly do you need help with? Please also read the FAQ regarding posting logs.
    • Hi, this is my second post with the same content as no one answered this and it's been a long time since I made the last post, I want to make a client-only mod, everything is ok, but when I use shaders, none of the textures rendered in RenderLevelStageEvent nor the crow entity model are rendered, I want them to be visible, because it's a horror themed mod I've already tried it with different shaders, but it didn't work with any of them and I really want to add support for shaders Here is how i render the crow model in the CrowEntityRenderer<CrowEntity>, by the time i use this method, i know is not the right method but i don't think this is the cause of the problem, the renderType i'm using is entityCutout @Override public void render(CrowEntity p_entity, float entityYaw, float partialTick, PoseStack poseStack, MultiBufferSource bufferSource, int packedLight) { super.render(p_entity, entityYaw, partialTick, poseStack, bufferSource, packedLight); ClientEventHandler.getClient().crow.renderToBuffer(poseStack, bufferSource.getBuffer(ClientEventHandler.getClient().crow .renderType(TEXTURE)), packedLight, OverlayTexture.NO_OVERLAY, Utils.rgb(255, 255, 255)); } Here renderLevelStage @Override public void renderWorld(RenderLevelStageEvent e) { horrorEvents.draw(e); } Here is how i render every event public void draw(RenderLevelStageEvent e) { for (HorrorEvent event : currentHorrorEvents) { event.tick(e.getPartialTick()); event.draw(e); } } Here is how i render the crow model on the event @Override public void draw(RenderLevelStageEvent e) { if(e.getStage() == RenderLevelStageEvent.Stage.AFTER_ENTITIES) { float arcProgress = getArcProgress(0.25f); int alpha = (int) Mth.lerp(arcProgress, 0, 255); int packedLight = LevelRenderer.getLightColor(Minecraft.getInstance().level, blockPos); VertexConsumer builder = ClientEventHandler.bufferSource.getBuffer(crow); Crow<CreepyBirdHorrorEvent> model = ClientEventHandler .getClient().crow; model.setupAnim(this); RenderHelper.renderModelInWorld(model, position, offset, e.getCamera(), e.getPoseStack(), builder, packedLight, OverlayTexture.NO_OVERLAY, alpha); builder = ClientEventHandler.bufferSource.getBuffer(eyes); RenderHelper.renderModelInWorld(model, position, offset, e.getCamera(), e.getPoseStack(), builder, 15728880, OverlayTexture.NO_OVERLAY, alpha); } } How i render the model public static void renderModelInWorld(Model model, Vector3f pos, Vector3f offset, Camera camera, PoseStack matrix, VertexConsumer builder, int light, int overlay, int alpha) { matrix.pushPose(); Vec3 cameraPos = camera.getPosition(); double finalX = pos.x - cameraPos.x + offset.x; double finalY = pos.y - cameraPos.y + offset.y; double finalZ = pos.z - cameraPos.z + offset.z; matrix.pushPose(); matrix.translate(finalX, finalY, finalZ); matrix.mulPose(Axis.XP.rotationDegrees(180f)); model.renderToBuffer(matrix, builder, light, overlay, Utils .rgba(255, 255, 255, alpha)); matrix.popPose(); matrix.popPose(); } Thanks in advance
  • Topics

  • Who's Online (See full list)

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.