Jump to content
Search In
  • More options...
Find results that contain...
Find results in...

saving custom classes in nbt?


Atijaf
 Share

Recommended Posts

You can always split objects in to basic objects. All basic objects, you can save in nbt. For example BlockPos, can be split into

int x, int y, int z

. And IBlockState can be converted to

int meta

by using get metaFromState.

Link to comment
Share on other sites

Just overwrite the method writeToNBT and readFromNBT to save/load your data.

 

For example:

@Override
    public void writeToNBT(NBTTagCompound nbtTagCompound)
    {
        super.writeToNBT(nbtTagCompound);

        nbtTagCompound.setInteger("WorldID", world.getId())
        ........
    }

Link to comment
Share on other sites

Just overwrite the method writeToNBT and readFromNBT to save/load your data.

 

For example:

@Override
    public void writeToNBT(NBTTagCompound nbtTagCompound)
    {
        super.writeToNBT(nbtTagCompound);

        nbtTagCompound.setInteger("WorldID", world.getId())
        ........
    }

I think he has custom class and no writeToNbt and readToNbt methods (and maybe no super methods). And he asks about how he can save "complex" variables like objects to nbt.

Link to comment
Share on other sites

I have a list of blocks and block locations that are suppose to regenerate in specific areas.  I have a command that I can use to add them to a list and once I save and quit, I would like to store this information so that it may be used the next time I load into the world.

Link to comment
Share on other sites

Alright I think I'm getting somewhere.

 

I've noticed that you didn't use the methods nbt.setTag or nbt.getTag inside readFromNBT and writeToNBT.

I also see that you are using a method inside QuestologyWorldData get(World world), called world.setItemData(IDENTIFIER, data)

 

Why not use nbt.setTag/getTag and what is "world.setItemData"

 

Also, is markDirty() saving the information?

 

Thank you!  Sorry for the load of questions!

Link to comment
Share on other sites

I've noticed that you didn't use the methods nbt.setTag or nbt.getTag inside readFromNBT and writeToNBT.
I am not sure what you mean here...

I also see that you are using a method inside QuestologyWorldData get(World world), called world.setItemData(IDENTIFIER, data)

 

Why not use nbt.setTag/getTag and what is "world.setItemData"

These are completely different things. The NBT is only for saving to disk. At runtime your data is represented by the WorldSavedData instance. The get() method is just a lazy initialization. If the data is not yet loaded, Minecraft will load it from disk. If it does not exist there either, we create it manually.

Also, is markDirty() saving the information?

Basically, yes. It does not immediately save, it tells minecraft "hey, I have changed". Minecraft will then save it when it is appropriate.
Link to comment
Share on other sites

When I use my command to add new information, it runs through the write method and adds the data I want it to inside of the nbt and It also works correctly while reading the information.  But, when I restart the game, it doesn't find anything there.  My only guess would be that the world Data information I am using is different every time I load the game.  Here is some code.

 

Event Handler

 

 

@SubscribeEvent

public void onWorldLoad(WorldEvent.Load event){

PacketDispatcher.sendToAll(new SyncWorldData(event.world));

}

 

 

 

PacketDispatcher / sendTo methods

 

 

public class PacketDispatcher

{

// a simple counter will allow us to get rid of 'magic' numbers used during packet registration

private static byte packetId = 0;

private boolean used;

 

/**

* The SimpleNetworkWrapper instance is used both to register and send packets.

* Since I will be adding wrapper methods, this field is private, but you should

* make it public if you plan on using it directly.

*/

private static final SimpleNetworkWrapper dispatcher = NetworkRegistry.INSTANCE.newSimpleChannel(Reference.MOD_ID);

 

 

/**

* Call this during pre-init or loading and register all of your packets (messages) here

*/

public static final void registerPackets() {

registerMessage(SyncMiningStats.class);

registerMessage(SyncExcavationStats.class);

registerMessage(SyncWoodCuttingStats.class);

registerMessage(SyncWorldData.class);

 

}

 

/**

* Registers an {@link AbstractMessage} to the appropriate side(s)

*/

private static final <T extends AbstractMessage<T> & IMessageHandler<T, IMessage>> void registerMessage(Class<T> clazz) {

// We can tell by the message class which side to register it on by using #isAssignableFrom (google it)

 

// Also, one can see the convenience of using a static counter 'packetId' to keep

// track of the current index, rather than hard-coding them all, plus it's one less

// parameter to pass.

if (AbstractMessage.AbstractClientMessage.class.isAssignableFrom(clazz)) {

PacketDispatcher.dispatcher.registerMessage(clazz, clazz, packetId++, Side.CLIENT);

} else if (AbstractMessage.AbstractServerMessage.class.isAssignableFrom(clazz)) {

PacketDispatcher.dispatcher.registerMessage(clazz, clazz, packetId++, Side.SERVER);

} else {

// hopefully you didn't forget to extend the right class, or you will get registered on both sides

PacketDispatcher.dispatcher.registerMessage(clazz, clazz, packetId, Side.CLIENT);

PacketDispatcher.dispatcher.registerMessage(clazz, clazz, packetId++, Side.SERVER);

}

}

 

//========================================================//

// The following methods are the 'wrapper' methods; again,

// this just makes sending a message slightly more compact

// and is purely a matter of stylistic preference

//========================================================//

 

 

 

/**

* Send this message to the specified player's client-side counterpart.

* See {@link SimpleNetworkWrapper#sendTo(IMessage, EntityPlayerMP)}

*/

public static final void sendTo(IMessage message, EntityPlayerMP player) {

PacketDispatcher.dispatcher.sendTo(message, player);

}

 

/**

* Send this message to everyone.

* See {@link SimpleNetworkWrapper#sendToAll(IMessage)}

*/

public static void sendToAll(IMessage message) {

PacketDispatcher.dispatcher.sendToAll(message);

}

 

/**

* Send this message to everyone within a certain range of a point.

* See {@link SimpleNetworkWrapper#sendToAllAround(IMessage, NetworkRegistry.TargetPoint)}

*/

public static final void sendToAllAround(IMessage message, NetworkRegistry.TargetPoint point) {

PacketDispatcher.dispatcher.sendToAllAround(message, point);

}

 

/**

* Sends a message to everyone within a certain range of the coordinates in the same dimension.

* Shortcut to {@link SimpleNetworkWrapper#sendToAllAround(IMessage, NetworkRegistry.TargetPoint)}

*/

public static final void sendToAllAround(IMessage message, int dimension, double x, double y, double z, double range) {

PacketDispatcher.sendToAllAround(message, new NetworkRegistry.TargetPoint(dimension, x, y, z, range));

}

 

/**

* Sends a message to everyone within a certain range of the player provided.

* Shortcut to {@link SimpleNetworkWrapper#sendToAllAround(IMessage, NetworkRegistry.TargetPoint)}

*/

public static final void sendToAllAround(IMessage message, EntityPlayer player, double range) {

PacketDispatcher.sendToAllAround(message, player.worldObj.provider.getDimensionId(), player.posX, player.posY, player.posZ, range);

}

 

/**

* Send this message to everyone within the supplied dimension.

* See {@link SimpleNetworkWrapper#sendToDimension(IMessage, int)}

*/

public static final void sendToDimension(IMessage message, int dimensionId) {

PacketDispatcher.dispatcher.sendToDimension(message, dimensionId);

}

 

/**

* Send this message to the server.

* See {@link SimpleNetworkWrapper#sendToServer(IMessage)}

*/

public static final void sendToServer(IMessage message) {

PacketDispatcher.dispatcher.sendToServer(message);

}

}

 

 

 

SnycWorldData Class

 

 

public class SyncWorldData extends AbstractClientMessage<SyncWorldData>{

 

private NBTTagCompound world_data;

 

public SyncWorldData(){}

 

public SyncWorldData(World world){

world_data = new NBTTagCompound();

WorldData.get(world).writeToNBT(world_data);

//get method

 

 

public static WorldData get(World world){

WorldData data = (WorldData)world.loadItemData(WorldData.class, Identifier);

if (data == null){

data = new WorldData();

world.setItemData(Identifier, data);

}

return data;

}

 

 

 

//WriteToNBT method

 

 

public void writeToNBT(NBTTagCompound nbt) {

 

NBTTagList tagList = new NBTTagList();

NBTTagCompound tag = new NBTTagCompound();

 

for(Tuple test : currentRespawnLocs.keySet()){

BlockPos pos = test.getPos();

IBlockState state = test.getState();

int blockId = Requirements.getBlockId(state);

int blockMeta = Requirements.getBlockMeta(state);

int worldId = test.getWorldId();

 

                        //STORES INFORMATION IN ARRAY OF INTS

int[] tempPos = new int[]{pos.getX(),pos.getY(),pos.getZ(), blockId, blockMeta, worldId};

posArray.add(tempPos);

}

 

for(int i = 0; i < posArray.size(); i++){

int[] currentMyBlockPos = posArray.get(i);

if(currentMyBlockPos != null && nbt.getIntArray("myBlockPos" + i) != null){

 

tag.setIntArray("myBlockPos" + i, currentMyBlockPos);

                                //APPEND IF TagList HAS NO LENGTH YET

if(tagList.hasNoTags())

tagList.appendTag(tag);

else

tagList.set(0, tag);

}

}

//STORE TagList INSIDE NBT

nbt.setTag(Identifier, tagList);

}

 

 

 

 

}

 

protected void read(PacketBuffer buffer) throws IOException{

world_data = buffer.readNBTTagCompoundFromBuffer();

}

 

protected void write(PacketBuffer buffer) throws IOException{

buffer.writeNBTTagCompoundToBuffer(world_data);

}

 

public void process(EntityPlayer player, Side side){

WorldData.get(player.worldObj).readFromNBT(world_data);

//READ

 

 

/**

This undoes what the write method does.  It takes all the integers and converts them to blockPos, an IBlockState, and worldId.  It then adds them to the hashMap currentRespawnLocs(a public class variable.

*/

public void readFromNBT(NBTTagCompound nbt) {

NBTTagList tagList = nbt.getTagList(Identifier, Constants.NBT.TAG_COMPOUND);

for(int i = 0; tagList.getCompoundTagAt(0).hasKey("myBlockPos" + i); i++){

int[] currentMyBlockPos = tagList.getCompoundTagAt(0).getIntArray("myBlockPos" + i);

BlockPos pos = new BlockPos(currentMyBlockPos[0], currentMyBlockPos[1], currentMyBlockPos[2]);

int blockId = currentMyBlockPos[3];

int blockMeta = currentMyBlockPos[4];

int worldId = currentMyBlockPos[5];

 

Block tempBlock = Block.getBlockById(blockId);

IBlockState state = tempBlock.getStateFromMeta(blockMeta);

 

currentRespawnLocs.put(new Tuple(this.pos, this.state, this.worldId), true);

}

}

 

 

}

 

}

 

 

 

I know that this is a lot of information.  I don't expect you, nor anyone, to read all of it.  I am just curious if you can spot any errors.  The packetDispatcher should be just fine, as I have used it to store information with players.

Link to comment
Share on other sites

There was one more class that I forgot to include in pastebin.

Also, every time i start minecraft and debug.  It goes through the method, WorldData get(World world), and data is null so it creates a new one.  Is that suppose to happen?

I also never call the method, setDirty();

 

I went ahead and changed the static variable to a private class variable.  No change...  I'll leave it a private class variable though

 

Here's my SyncWorldData class

http://pastebin.com/dgNBV6G2

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
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.

 Share



×
×
  • Create New...

Important Information

By using this site, you agree to our Privacy Policy.