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



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Start with removing: modernfix and FlyWheel
    • Vigor Vita CBD Gummies- Behind every product is a story. Vigor Vita CBD Gummies's journey involves a commitment to quality, customer satisfaction, and a genuine desire to enhance the well-being of individuals. Understanding the ethos of the company adds depth to the product experience.   Official Website==> https://bestlifecaremart.com/vigor-vita-cbd-gummies/   Facebook==> https://www.facebook.com/GetVigorVitaCBDGummies/   Groups Google:   https://groups.google.com/g/mozilla.dev.platform/c/kAoLq82Obfc   https://groups.google.com/g/comp.os.vms/c/MMy0pLUnU40   https://groups.google.com/g/mozilla.dev.platform/c/op2Zyok7iKI   Sites Google==> https://sites.google.com/view/vigor-vita-c-b-d-gummies-price/home   Gamma==> https://gamma.app/public/Is-Vigor-Vita-CBD-Gummies-Scam-Read-Reviews-Best-CBD-Gummies-Pric-x91kh5meqawi99c   Jimdo==> https://vigor-vita-cbd-gummies-us-reviews-1.jimdosite.com/   Get More Info:   https://medium.com/@yvettieany/vigor-vita-cbd-gummies-united-states-us-100-natural-benefits-pain-relief-pros-cons-scam-d6cff170cac5   https://www.facebook.com/KellyClarksonSlimFusionGummiesUSA/   https://www.mid-day.com/lifestyle/infotainment/article/vigor-vita-cbd-gummies-reviews-wellness-peak-cbd-gummies-pure-ease-cbd-gummies-23314946   https://groups.google.com/g/mozilla.dev.platform/c/hG7s3KQI2q8  
    • If anyone can help me with this . I'm creating a command to open the GUI of a book. Supposedly this should work. But it doesn't do anything. Can somebody help me? Forge 1.16.5 - Server Side Mod   dispatcher.register( Commands.literal("dummyCommand") .executes(context -> { ItemStack book = new ItemStack(Items.WRITTEN_BOOK); CompoundNBT nbt = new CompoundNBT(); ListNBT pages = new ListNBT(); pages.add(StringNBT.valueOf("{\"text\":\"Hello World\"}")); nbt.putString("author", "Steve"); nbt.putString("title", "my_book"); nbt.put("pages", pages); nbt.putByte("resolved", (byte) 1); book.setTag(nbt); context.getSource().getPlayerOrException().openItemGui(book, Hand.MAIN_HAND); return 1; }) ); I have tried with "SOpenBookWindowPacket" but it only sends the instruction to open a book that the user has in their hand. ServerPlayerEntity serverPlayerEntity = context.getSource().getPlayerOrException(); SOpenBookWindowPacket packet = new SOpenBookWindowPacket(serverPlayerEntity.getUsedItemHand()); serverPlayerEntity.connection.send(packet); The idea is that this mod is an information command for the user. You should not give a book to the user and have them read it. I have seen the Spigot documentation and have found some examples of how this is done on servers. PacketPlayOutCustomPayload packet = new PacketPlayOutCustomPayload("MC|BOpen", new PacketDataSerializer(buf)); Is there a way to do this with Forge Server Side? I've been dealing with this for 2 days and I can't find a solution. I've searched on Google and can't find anything.
    • Finally got around to fixing this, just removed the use method, and changed  FIERY_ARROW_ONLY = (stack) -> stack.getItem() instanceof InitItems.FIERY_ARROW.get(); to FIERY_ARROW_ONLY = (stack) -> stack.is(InitItems.FIERY_ARROW.get()); which fixes the issue, and also allows you to use regular arrows, which is an ability I want (I scrapped this).
    • Check Mod Versions: Ensure you are using the correct version of the Scorge mod compatible with Minecraft 1.16.5. Update or downgrade the mod accordingly. Remove or Update Problematic Mods: Identify and remove mods causing conflicts. Test the game by removing mods one by one until you find the one causing the issue. Update Forge: Use the latest version of Forge for Minecraft 1.16.5 to resolve compatibility issues. Check for Mod Updates: Ensure the Scorge mod is up to date. Mod developers release updates to address bugs and compatibility issues. If the problem persists, consider seeking help on Minecraft modding forums or contacting the mod developer for specific guidance. Always back up your world before making changes.
  • Topics

×
×
  • Create New...

Important Information

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