Jump to content

[Solved][1.16.1] One time initializing for entities


DavidQF555

Recommended Posts

I was told that the onInitialSpawn() method was for one time initializing for things like armor, but I also thought that was a good place to initialize many lists and maps that are my own fields for my own custom entity. However, I'm getting loads of nullpointerexceptions because apparently it initializes the fields after ticks and goals are run. I don't want to put it in the constructor because then the lists will be constantly cleared. Is there a way to initialize them once before the ticks and goals run? I really don't want to put null checkers in every tick method. 

Edited by DavidQF555
Link to comment
Share on other sites

4 hours ago, diesieben07 said:

The constructor should be fine. What do you mean by "constantly cleared"? show your code.

Someone told me that the constructor runs every time an entity spawns, so I should use onInitialSpawn for 1 time events. So if I did list = new ArrayList<>() in the construtor, wouldn't it just reset every time an entity spawns? Is that not how the constructor runs?

Link to comment
Share on other sites

15 minutes ago, diesieben07 said:

The constructor runs whenever a new instance of your entity is created.

This happens when it spawns, but also when it is loaded from disk (e.g. when the chunk got unloaded). It also happens when your entity is created on the client, that is when a player is in or comes into range.

onInitialSpawn fires only when the entity is first spawned, not when it is e.g. loaded from disk.

 

As for your list: you need to of course create it in the constructor, that is how objects work (basic Java knowledge is needed, read up on how objects work if you don't know this stuff).

Then you can use onInitialSpawn to set the intial data of your entity. You then need to use the NBT read and write function to actually make your data persist.

for reading and writing NBT for entities, do you use the read/write additional methods?

Link to comment
Share on other sites

@Override
    public void readAdditional(@Nonnull CompoundNBT nbt) {
        super.readAdditional(nbt);
        System.out.println("run");
        if (nbt.contains("testmod.customentity")) {
            System.out.println("has nbt");
            CompoundNBT reg = (CompoundNBT) nbt.get("testmod.customentity");
            personality = Personality.get(reg.getString("personality"));
            level = nbt.getInt("level");
        }
    }

    @Override
    public void writeAdditional(@Nonnull CompoundNBT nbt) {
        super.writeAdditional(nbt);
        CompoundNBT reg = new CompoundNBT();
        reg.putString("personality", personality.name());
        reg.putInt("level", level);
        nbt.put("testmod.customentity", reg);
    }

It seems like some of my fields were just left as null because the nbt apparently doesn't have "testmod.customentity". Did I do something wrong? "has nbt" does not appear, but "run" does.

Link to comment
Share on other sites

If you had entities in the world already that were saved before you added that code, of course they won't have it.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

I'm confused as to the order that the constructor runs. I set a default value in the constructor and had it randomized to the actual value in the onInitalSpawn method. It keeps on changing back to the default value though. 

public CustomEntity(World worldIn, int level) {
        super(RegistryHandler.CUSTOM_ENTITY.get(), worldIn);
        this.level = level;
        personality = Personality.NEUTRAL;
        System.out.println("constructor");
    }

public ILivingEntityData onInitialSpawn(IWorld worldIn, DifficultyInstance difficultyIn, SpawnReason reason, ILivingEntityData spawnDataIn, CompoundNBT dataTag) {
  		Personality[] personalities = Personality.values();
        personality = personalities[(int) (Math.random() * personalities.length)];
    }

I checked the personality value by printing in livingTick() and also because the entity is rendered with a texture based on personality. In livingTick(), the personality constantly switches between NEUTRAL and another value. The rendered texture is always just the default one. 

Link to comment
Share on other sites

22 minutes ago, DavidQF555 said:

constantly switches between NEUTRAL and another value.

See also: Threads.

$100 says that the client thread (which doesn't matter) prints NEUTRAL while the server thread prints the randomized value.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

1 hour ago, Draco18s said:

See also: Threads.

$100 says that the client thread (which doesn't matter) prints NEUTRAL while the server thread prints the randomized value.

How would I retrieve the server side version of Personality for rendering then? Or how would I make it so that I only retrieve the server side version? 

Edited by DavidQF555
Link to comment
Share on other sites

Usually packets.

There's also a way to specify entity data values (eg. things that don't change, or change rarely), I'm drawing a blank on what they're called at the moment.

Edited by Draco18s

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

On 8/13/2020 at 9:13 AM, diesieben07 said:

For your own entities you can use EntityDataManager (look at vanilla for examples) for data that can change or IEntityAdditionalSpawnData for data that never changes.

I was able to use data parameters to sync client and server data

Link to comment
Share on other sites

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



×
×
  • Create New...

Important Information

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