TileEntity bugging out metadata [Solved]

[Lua]

Hi, so I have a cannon which works perfectly fine, reloading, firing all that jazz. However when I leave the game and join back, the cannon 'forgets' its direction. So I thought simple fix, ill just save the meta to the TileEntity ntb methods, so I did, but no avail. So in short, the cannons fires in a position according to its direction after re-opening the game it forgets that direction(meta) and defaults to -1 (tileentites default) which makes it not go through with the else-if statement, causing it not to spawn. Bad times.

 

Anyway, heres my code:

package arcticraft.tile_entities;

import net.minecraft.client.Minecraft;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.tileentity.TileEntity;
import arcticraft.entities.AC_EntityCannonball;

public class AC_TileEntityCannon extends TileEntity
{
//private int this.blockMetadata = this.blockMetadata;
public boolean isLoaded;
private int fuse;

public AC_TileEntityCannon()
{

}

@Override
public void updateEntity()
{
System.out.println(this.blockMetadata);
if(isLoaded)
{
fuse++;
if(fuse == 100)
{
AC_EntityCannonball cannonball = new AC_EntityCannonball(worldObj, this.xCoord, this.yCoord, this.zCoord);
if(this.blockMetadata == 4)
{
cannonball.setPosition(this.xCoord, this.yCoord + 2, this.zCoord + 2);
cannonball.setVelocity(0, 2.0, 0.7);
Minecraft.getMinecraft().sndManager.playSoundFX("ac:misc.cannon", 1.5F, 1.5F);
worldObj.spawnEntityInWorld(cannonball);
isLoaded = false;
fuse = 0;
}
else if(this.blockMetadata == 3)
{
cannonball.setPosition(this.xCoord + 1.5, this.yCoord + 2, this.zCoord - 1);
cannonball.setVelocity(0.625, 2.0, 0.025);
Minecraft.getMinecraft().sndManager.playSoundFX("ac:misc.cannon", 1.5F, 1.5F);
worldObj.spawnEntityInWorld(cannonball);
isLoaded = false;
fuse = 0;
}
else if(this.blockMetadata == 2)
{
cannonball.setPosition(this.xCoord, this.yCoord + 2, this.zCoord - 1);
cannonball.setVelocity(0, 2.0, - 0.7);
Minecraft.getMinecraft().sndManager.playSoundFX("ac:misc.cannon", 1.5F, 1.5F);
worldObj.spawnEntityInWorld(cannonball);
isLoaded = false;
fuse = 0;
}
else if(this.blockMetadata == 1)
{
cannonball.setPosition(this.xCoord - 1.5, this.yCoord + 2, this.zCoord + 1);
cannonball.setVelocity(- 0.625, 2.0, - 0.025);
Minecraft.getMinecraft().sndManager.playSoundFX("ac:misc.cannon", 1.5F, 1.5F);
worldObj.spawnEntityInWorld(cannonball);
isLoaded = false;
fuse = 0;
}
}
}
}

@Override
public void readFromNBT(NBTTagCompound compound)
{
compound.getBoolean("loaded");
compound.getInteger("fuse");
compound.getInteger("meta");
}

public void writeToNBT(NBTTagCompound compound)
{
compound.setBoolean("loaded", isLoaded);
compound.setInteger("fuse", fuse);
compound.setInteger("meta", this.blockMetadata);
}
}

 

On re-entering the game I see that the

System.out.println(this.blockMetadata)

varies between the real direction and -1. But when I first place the cannon its a sold printing of the actual direction.

 

 

*To the people who try get me to tidy my code by placing the spawning of the cannonball and resetting of the fuse and stuff by putting it at the end of the else-if statement. No, because I want it to spawn depending on that meta only, as the positions and velocity correspond to that direction*

 

Thanks ~ Ben

 

P.S Why does using code tags mess up the formatting of your code, its annoying.

[larsgerrits]

What i think is happening, is that your tile entity is desynchronized between client and server.

[GotoLink]

@Override
public void readFromNBT(NBTTagCompound compound)
{
super.readFromNBT(compound);
isLoaded=compound.getBoolean("loaded");
fuse=compound.getInteger("fuse");
blockMetadata=compound.getInteger("meta");
}

Would be better this way.

[Lua]

@Override
public void readFromNBT(NBTTagCompound compound)
{
super.readFromNBT(compound);
isLoaded=compound.getBoolean("loaded");
fuse=compound.getInteger("fuse");
blockMetadata=compound.getInteger("meta");
}

Would be better this way.

 

Still does the same thing

 

What i think is happening, is that your tile entity is desynchronized between client and server.

 

Metadata is client sided and it prints out client side

[MineMaarten]

Don't use the blockMetadata in the TileEntity, and don't save it in NBT (Minecraft will do that for you). Use getBlockMetadata() instead. You can see in getBlockMetadata() why:

 /**
     * Returns block data at the location of this entity (client-only).
     */
    public int getBlockMetadata()
    {
        if (this.blockMetadata == -1)
        {
            this.blockMetadata = this.worldObj.getBlockMetadata(this.xCoord, this.yCoord, this.zCoord);
        }

        return this.blockMetadata;
    }

blockMetadata is just a variable in the TileEntity which holds the metadata, and gets updated when you call getBlockMetadata().

It says in the MCP (?) description that it's client only, but as far as I can see it isn't (and I've used it on server side successfully as well).

 

If you want to set the metadata from your TileEntity use the good old worldObj.setBlockMetadataWithNotify(xCoord, yCoord, zCoord, metadata, updateflags).

 

[Lua]

Don't use the blockMetadata in the TileEntity, and don't save it in NBT (Minecraft will do that for you). Use getBlockMetadata() instead. You can see in getBlockMetadata() why:

 /**
     * Returns block data at the location of this entity (client-only).
     */
    public int getBlockMetadata()
    {
        if (this.blockMetadata == -1)
        {
            this.blockMetadata = this.worldObj.getBlockMetadata(this.xCoord, this.yCoord, this.zCoord);
        }

        return this.blockMetadata;
    }

blockMetadata is just a variable in the TileEntity which holds the metadata, and gets updated when you call getBlockMetadata().

It says in the MCP (?) description that it's client only, but as far as I can see it isn't (and I've used it on server side successfully as well).

 

If you want to set the metadata from your TileEntity use the good old worldObj.setBlockMetadataWithNotify(xCoord, yCoord, zCoord, metadata, updateflags).

 

 

I tried setting the metadata like that before i made this topic. However, using a combination of setting it and using getBlockMetadata() instead seemed to solve the problem. Thanks