Jump to content

GUI does not stay updated


Cyan

Recommended Posts

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 :P

Link to comment
Share on other sites

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

 

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

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

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;
    			
    		}
    		
    		
    	
    	}

    }

Link to comment
Share on other sites

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.

I am Mew. The Legendary Psychic. I behave oddly and am always playing practical jokes.

 

I have also found that I really love making extremely long and extremely but sometimes not so descriptive variables. Sort of like what I just did there xD

Link to comment
Share on other sites

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);
    }

Link to comment
Share on other sites

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

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

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

Link to comment
Share on other sites

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.

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.



×
×
  • Create New...

Important Information

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