GUI does not stay updated

[Cyan]

Hi,

 

So like the topic says, I seem to be having trouble making my GUI stay updated when I want to give my entity a new name.

 

I have the standard keyTyped function for imputing the characters in the GUI, and I have a datawatcher keeping track of the nickname that results from it.

 

While in the game, it works fine, but the second I log out it reverts back to the regular name, rather than saving the nickname.

 

I figure this is a problem with the client and server not communicating, but I am unsure of why it is happening, and I would be greatly appreciative for some advice.

 

 

If this isn't enough information to go on, someone chew me out for it and I'll post up the code as well

[Draco18s]

You didn't tell the server that the name changed.

 

(Client-Server disparity.  You need a packet handler)

[Cyan]

You didn't tell the server that the name changed.

 

(Client-Server disparity.  You need a packet handler)

 

 

Ah! But I do have a packet handler.

Granted, I am willing to admit that I probably am not using it right.

 

 

Okay, fine, I'm not entirely sure at all what I am doing when it comes to the packet handler.

 

I'm going to assume the packet function goes something along the lines of

try {
					if () {}
}

 

I suppose I am puzzled somewhat on what to add to the if condition.

 

 

 

Edit: Actually, here is just my whole packet handler.

import java.io.ByteArrayInputStream;
import java.io.DataInputStream;
import java.io.IOException;

import net.minecraft.network.INetworkManager;
import net.minecraft.network.packet.Packet250CustomPayload;

import cpw.mods.fml.common.FMLCommonHandler;
import cpw.mods.fml.common.network.IPacketHandler;
import cpw.mods.fml.common.network.Player;

public class ModPacketHandler implements IPacketHandler {

        @Override
        public void onPacketData(INetworkManager manager,
                        Packet250CustomPayload packet, Player player) {
                
                if (packet.channel.equals("TestMod")) {
                        handlePacket(packet);
                }
        }
        
        public void handlePacket(Packet250CustomPayload packet){
    		
    		DataInputStream inputStream = new DataInputStream(new ByteArrayInputStream(packet.data));
    			
    		int randomInt1;
    		int randomInt2;
    	
    		try {
    			
    			if () {
    				
    			}
    		

    			
    		}catch(IOException e){
    			
    			e.printStackTrace();
    			return;
    			
    		}
    		
    		
    	
    	}

    }

[Mew]

You didn't tell the server that the name changed.

 

(Client-Server disparity.  You need a packet handler)

 

That, or he hasn't written the new name to NBT... That might help if he isn't doing that.

[Cyan]

You didn't tell the server that the name changed.

 

(Client-Server disparity.  You need a packet handler)

 

That, or he hasn't written the new name to NBT... That might help if he isn't doing that.

 

The new name should be updated via the dataWatcher, unless I missed something.

 

I'll post those too, incase I am missing something very simple.

dataWatcher.addObject(3, ""); 
//in the constructor

nbt.setString("nickname", dataWatcher.getWatchableObjectString(3));
dataWatcher.updateObject(3, nbt.getString("nickname"));
// Write and Read NBT functions respectively.

public String getNickname()
    {
    	if (dataWatcher.getWatchableObjectString(3).equals(""))
    		return name;		
    	return dataWatcher.getWatchableObjectString(3);
    }
public void setNickname(String nickname)
    {
    	dataWatcher.updateObject(3, nickname);
    }

[Draco18s]

That, or he hasn't written the new name to NBT... That might help if he isn't doing that.

 

I generally assume that people are using their NBTs correctly.  They're so trivially easy to get right (compared to packet handlers and data watchers*).

But yes, it is a possibility.

 

*Still have no idea what these are needed for myself....

[Cyan]

Bump in case anyone can point me in the right direction on the packet handler.

[Yagoki]

Here is how I handle packets for my tile entities, should be similar for other things too.

 

also here is how i'm sending the packets

ByteArrayOutputStream bos =
			new ByteArrayOutputStream(s.length() + 12);
		DataOutputStream outputStream = new DataOutputStream(bos);

		try
		{
			outputStream.writeInt(x);
			outputStream.writeInt(y);
			outputStream.writeInt(z);

			outputStream.writeUTF(s);
		}
		catch (IOException e)
		{
			e.printStackTrace();
		}

		Packet250CustomPayload packet = new Packet250CustomPayload();
		packet.channel = MyPacketHandler.proxDet;
		packet.data = bos.toByteArray();
		packet.length = bos.size();

		PacketDispatcher.sendPacketToServer(packet);

 

full class here

[Cyan]

Here is how I handle packets for my tile entities, should be similar for other things too.

 

also here is how i'm sending the packets

ByteArrayOutputStream bos =
			new ByteArrayOutputStream(s.length() + 12);
		DataOutputStream outputStream = new DataOutputStream(bos);

		try
		{
			outputStream.writeInt(x);
			outputStream.writeInt(y);
			outputStream.writeInt(z);

			outputStream.writeUTF(s);
		}
		catch (IOException e)
		{
			e.printStackTrace();
		}

		Packet250CustomPayload packet = new Packet250CustomPayload();
		packet.channel = MyPacketHandler.proxDet;
		packet.data = bos.toByteArray();
		packet.length = bos.size();

		PacketDispatcher.sendPacketToServer(packet);

 

full class here

 

Thanks! This makes a bit more sense now, and I think I see what I have done wrong.