Saving up CPU (Packets, etc.)

[nielsbwashere]

Hi everyone,

I am working on a mod; which includes a tile entity that updates and sends packets to the player.

The problem that I am having; is that the game is having huge lag since it is updating too frequently, also this block is used like a lot so it should be so it doesn't lag with a lot of them in the world.

 

Do you have any tips and tricks on how to make it so it updates every ? ticks?

This will help a lot.

 

Thanks!

~Nielsbwashere

[Ernio]

Just so you know - ANY packet system that is NOT based on client-request<->server-response (like opening conatiner/gui, walking into area, being in area, some clicking actions, buttons) will be generating a lot of lags when used in too many 'places'.

 

Let's take example of server-data->client-data (no-request-system):

Synchronization of player data!

So compare player to TileEntity - one is from 0 to few hundreds (that would be good server), other one (TE) is infinite.

 

Rethink you design:

Does your Player need to know what's happening in all TE in WHOLE server world?

If not - consider adding entity scanner in your TE, if Entity instanceof Player -> send packet.

Does your Player even need that data at all times?

Maybe you could make this code from server-data->client-data to client-request<->server-response.

 

Also: "Do you have any tips and tricks on how to make it so it updates every ? ticks?"

What you mean? TE is alredy being updated on tick (or not? enlighten me)?

[daafganggdg]

Do you have any tips and tricks on how to make it so it updates every ? ticks?

Uhm, you mean some kind of like that:

	
int x = 0;
public void updateEntity(){
if(x == 40) {
x = 0
/...
} 
x++
}

It would be probably smarter to send packets only if the data actually changed, just set a boolean whenever you are changing the data and check for the boolean when you are sending the packet, wich, I assume you do in updateEntity()

[Draco18s]

TE's already have functions that handle this.

 

	@Override
public Packet getDescriptionPacket() {
	NBTTagCompound nbtTag = new NBTTagCompound();
	writeToNBT(nbtTag);
	return new S35PacketUpdateTileEntity(xCoord, yCoord, zCoord, 1, nbtTag);
}

@Override
public void onDataPacket(NetworkManager net, S35PacketUpdateTileEntity packet) {
	readFromNBT(packet.func_148857_g());
}

[coolAlias]

Guys. You are digging in the dark. Nobody knows what the code looks like.

Always on point - can't slip anything past this guy