Hi

 

I have this tool that when I right click, it fills a flat area with Blocks. The problem is, the blocks sometimes get placed on the server and client side correctly,

but when the amount of blocks that are changing is too large, they only get placed on the client side.

 

Here's my code:

@Override
public EnumActionResult onItemUse(ItemStack stack, EntityPlayer player, World world, BlockPos pos, EnumHand hand, EnumFacing side, float hitX, float hitY, float hitZ)
{


	matToReplace = world.getBlockState(pos);
	positions.add(pos);
	for(int i = 0; i < 80; i++) {

		for(int j = 0; j < positions.size(); j++) {

			if(side == EnumFacing.NORTH || side == EnumFacing.SOUTH) {
				addPositionsNorth(j,world);
			}
			if(side == EnumFacing.EAST || side == EnumFacing.WEST) {
				addPositionsEast(j,world);
			}	
			if(side == EnumFacing.DOWN || side == EnumFacing.UP) {
				addPositionsDown(j,world);
			}

		}

		positions.addAll(tempPositions);

		tempPositions.clear();
	} 


	for(int k = 0; k < positions.size(); k++) {

		world.setBlockState(positions.get(k), Block.getBlockFromName(stack.getTagCompound().getString("material")).getStateFromMeta(stack.getTagCompound().getInteger("damage")));
		world.scheduleBlockUpdate(positions.get(k), world.getBlockState(positions.get(k)).getBlock(), 1, 1);

	}


	positions.clear();
	tempPositions.clear();




	return EnumActionResult.PASS;
}

 

Any help would be apprechiated

They are fields in my Item class... So how should I do it differently? Forge Events? Packets?

They store the values for BlockPos so that they can get placed by the Item.

Items are singletons (only one instance of them ever exists), so instance fields can't be used to store per-stack data. The ItemStack's NBTTagCompound should be used to store data that is per-stack.