Jump to content

Don't understand TileEntity synchronization


jh62

Recommended Posts

I read this tutorial (http://cazzar.net/tutorials/minecraft/Tile-Entity-Updates-The-Quick-and-Dirty-Method/) and theres something I don't understand:

 

Where and how do I send my data just overriding this two methods in TileEntity?

 

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

 

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

 

I have a field called "InUse" that it should change whenever a player activates a block, but I don't understand how to send it through the block event.

 

I tried onBlockActivated() getting the description Packet and the NBTTag, write the value, but then I don't know how to send it.

 

What Im I Missing?

Link to comment
Share on other sites

Read further in the tutorial, it tells you how to sync your data. Specifically, the 'Now what?' section.

Don't PM me with questions. They will be ignored! Make a thread on the appropriate board for support.

 

1.12 -> 1.13 primer by williewillus.

 

1.7.10 and older versions of Minecraft are no longer supported due to it's age! Update to the latest version for support.

 

http://www.howoldisminecraft1710.today/

Link to comment
Share on other sites

Read further in the tutorial, it tells you how to sync your data. Specifically, the 'Now what?' section.

 

I read that part, but where in all that code i write the value of my custom field?

 

I only see two methods that are overriden in TileEntity and then the block is mark for update.

 

Should I do something like this?

 

@Override
public boolean onBlockActivated(World worldIn, BlockPos pos,
		IBlockState state, EntityPlayer playerIn, EnumFacing side,
		float hitX, float hitY, float hitZ) {

	if (worldIn.isRemote) {
		return true;
	}

	TileEntityTableSaw entity = (TileEntityTableSaw) worldIn
			.getTileEntity(pos);

	if (!entity.isInUse()) {
		entity.use(worldIn);

		S35PacketUpdateTileEntity packet = (S35PacketUpdateTileEntity) entity
				.getDescriptionPacket();
		NBTTagCompound tag = packet.getNbtCompound();
		tag.setBoolean("inUse", true);
		entity.onDataPacket(null, packet);
		worldIn.markBlockForUpdate(pos);
	} else {
		playerIn.addChatMessage(new ChatComponentText(
				"Can't use is it right now!"));
	}

	return true;
}

Link to comment
Share on other sites

No, in your TileEntity, you write the inUse value to NBT in the writeToNBT method. Then whenever you change the onUse variable in the TileEntity, you call

world.markBlockForUpdate(x,y,z)

. That's it.

Don't PM me with questions. They will be ignored! Make a thread on the appropriate board for support.

 

1.12 -> 1.13 primer by williewillus.

 

1.7.10 and older versions of Minecraft are no longer supported due to it's age! Update to the latest version for support.

 

http://www.howoldisminecraft1710.today/

Link to comment
Share on other sites

No, in your TileEntity, you write the inUse value to NBT in the writeToNBT method. Then whenever you change the onUse variable in the TileEntity, you call

world.markBlockForUpdate(x,y,z)

. That's it.

 

Thanks. That's what I thougth and I tried that also, so I must be missunderstanding something or doing something wrong.

 

The goal of the "InUse" boolean is to trigger an animation on the model. In the renderer i read the tileentity's inUse boolean and use it to rotate a cube. Here is some of the code:

 

BlockTableSaw.java:

@Override
public boolean onBlockActivated(World worldIn, BlockPos pos,
		IBlockState state, EntityPlayer playerIn, EnumFacing side,
		float hitX, float hitY, float hitZ) {

	if (worldIn.isRemote) {
		return true;
	}

	ItemStack stack = playerIn.getHeldItem();

	if (stack == null) {
		return true;
	}

	Item item = stack.getItem();
	Block block = Block.getBlockFromItem(item);

	ItemStack drop = null;

	if (block != null) {
		if (block.getMaterial() == Material.wood) {
			drop = new ItemStack(Blocks.log, stack.stackSize);
		}
	} else {
		drop = new ItemStack(Items.stick, stack.stackSize * 2);
	}

	TileEntityTableSaw entity = (TileEntityTableSaw) worldIn
			.getTileEntity(pos);

	if (!entity.isInUse()) {
		playerIn.destroyCurrentEquippedItem();
		entity.use(worldIn, playerIn, drop);
	} else {
		playerIn.addChatMessage(new ChatComponentText("That is beign used!"));
	}

	return true;
}

 

TileEntityTableSaw.java:

package com.pablismod.tileentity;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.Timer;

import net.minecraft.entity.Entity;
import net.minecraft.entity.item.EntityItem;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.network.NetworkManager;
import net.minecraft.network.Packet;
import net.minecraft.network.play.server.S35PacketUpdateTileEntity;
import net.minecraft.tileentity.TileEntity;
import net.minecraft.util.BlockPos;
import net.minecraft.world.World;

public class TileEntityTableSaw extends TileEntity {

private Timer t;
private boolean inUse = false;

@Override
public void readFromNBT(NBTTagCompound compound) {
	super.readFromNBT(compound);
	inUse = compound.getBoolean("inUse");
}

@Override
public void writeToNBT(NBTTagCompound compound) {
	super.writeToNBT(compound);
	compound.setBoolean("inUse", isInUse());
}

public boolean isInUse() {
	return t != null && t.isRunning();
}

public void use(World worldIn, EntityPlayer player, ItemStack stack) {

	if (t != null && t.isRunning()) {
		return;
	}

	inUse = true;
	worldIn.markBlockForUpdate(pos);

	final World world = worldIn;
	final EntityPlayer p = player;
	final ItemStack s = stack;

	t = new Timer(3000, new ActionListener() {

		@Override
		public void actionPerformed(ActionEvent evt) {

			BlockPos pos = TileEntityTableSaw.this.getPos();
			Entity e = new EntityItem(world, pos.getX(), pos.getY() + 1.2f,
					pos.getZ(), s);
			world.spawnEntityInWorld(e);

			inUse = false;
			world.markBlockForUpdate(TileEntityTableSaw.this.getPos());
		}
	});

	t.setRepeats(false);
	t.start();
}

@Override
public Packet getDescriptionPacket() {
	NBTTagCompound tag = new NBTTagCompound();
	this.writeToNBT(tag);
	return new S35PacketUpdateTileEntity(this.getPos(), 0, tag);
}

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

 

Don't mind Im using the javax.swing.Timer. The code is temporary and it works, because if I try to use the table when is in use, it wont let me... just the animation wont play.

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



×
×
  • Create New...

Important Information

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