Jump to content
View in the app

A better way to browse. Learn more.

Forge Forums

A full-screen app on your home screen with push notifications, badges and more.

To install this app on iOS and iPadOS
  1. Tap the Share icon in Safari
  2. Scroll the menu and tap Add to Home Screen.
  3. Tap Add in the top-right corner.
To install this app on Android
  1. Tap the 3-dot menu (⋮) in the top-right corner of the browser.
  2. Tap Add to Home screen or Install app.
  3. Confirm by tapping Install.

Featured Replies

Posted

I can't set Metadata in server in block crops

 

is block

@Override
public boolean onBlockActivated(World world, int x, int y, int z, EntityPlayer player, int t, float fx, float fy, float fz)
{
	Random ran = new Random();
	TileEntityCrops ted = (TileEntityCrops) world.getTileEntity(x, y, z);
	int l = world.getBlockMetadata(x, y, z);

	if(ted.username == player.getCommandSenderName() && l == 7){
		world.setBlockMetadataWithNotify(x, y, z, l -7, 2);
		world.setTileEntity(x, y, z, new TileEntityCrops());
		System.out.println(l);
		this.updateTick(world, x, y, z, ran);
		return true;
	}else{
		System.out.println(l);
		return false;
	}
}

 

is tile

public static String username;

public void processActivate(EntityPlayer player, World world) {
	username = player.getCommandSenderName();
	world.setTileEntity(this.xCoord, this.yCoord, this.zCoord, this);
}

1. You should NEVER call World#setTileEntity yourself - this is done automatically by Minecraft when a block with a TE is placed

 

2. Don't compare String with the '==' operator; use #equals instead

ted.username.equals(player.getCommandSenderName())
instead of
ted.username == player.getCommandSenderName()

 

3. If your TE #username field can be NULL, you'll want to either check that first or swap the order in #2

 

4. You probably don't want your TE's #username field to be static - that means every TE will have the same player name, all the time

 

5. Don't create a new Random every time the Block is activated - the World object already has a Random instance, as does Block. Use one of those instead, OR, if you really want a new Random, add it as a class field.

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

Important Information

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

Configure browser push notifications

Chrome (Android)
  1. Tap the lock icon next to the address bar.
  2. Tap Permissions → Notifications.
  3. Adjust your preference.
Chrome (Desktop)
  1. Click the padlock icon in the address bar.
  2. Select Site settings.
  3. Find Notifications and adjust your preference.