Jump to content

Recommended Posts

Posted

I'm having an issue I can't seem to find a resolution to.

 

I'm wanting Stone to drop Flint if broken with any tool that isn't a pickaxe.

 

public class StoneEventHandler {
@SubscribeEvent
public void onHarvestBlock(BlockEvent.HarvestDropsEvent event){

	Block block = event.block;
	EntityPlayer player = event.harvester;

	if(block == Blocks.stone){
		if (player.getHeldItem().getItem() != Items.stone_pickaxe && player.getHeldItem().getItem() != Items.iron_pickaxe){
			player.inventory.addItemStackToInventory(new ItemStack(Items.flint, 1));
		}

	}
}
}

 

This is my third attempt. I'm assuming it has to do directly with the event I've used except I don't know what one I should be using. It probably isn't being run as it isn't being broken with the designed tool.

 

Okay, I changed the event and it seems to work. I have a new question. How can I compare the item/tool used to break the block?

 

My old line of

if (player.getHeldItem().getItem() != Items.stone_pickaxe && player.getHeldItem().getItem() != Items.iron_pickaxe)

 

no longer works and I'm new to forge. Should I just go digging around in reflection or can someone help me out really fast and give me something that will work?

 

Small steps to progress. I'm now getting Flint with my hand. I'm using BreakEvent now, is that problematic?

 

EDIT: Now I'm looking for a way to check if a players using a pickaxe but will work if the player isn't holding anything.

Posted

Just create a new field of Item. I don't think you can compare like that.

 

Item heldItem = player.getHeldItem().getItem();

if (heldItem != Items.stone_pickaxe ...etc...)

 

Crashes my game when I use my fist which is why I removed it in the first place...

java.lang.NullPointerException: Ticking memory connection

It also points to the line number that uses the if statement.

 

... Oh, if anyone wants to try it themselves.

 

public class StoneEventHandler {
@SubscribeEvent
public void onBlockBreak(BreakEvent event){
	EntityPlayer player = event.getPlayer();
	Block block = event.block;
	Item heldItem = player.getHeldItem().getItem();

	if(block == Blocks.stone){
		if(heldItem != Items.stone_pickaxe){
			player.inventory.addItemStackToInventory(new ItemStack(Items.flint, 1));
		}
	}

}
}

is what I'm using now

Posted

Check first if player.getHeldItem != null.

You can only use player.getHeldItem.getItem if the player.getHeldItem is not null, this will make a crash.

 

I figured that out, thanks! I need a workaround as this is the starting base of my mod. I need an accessible and logical way to get flint... before stone even. So if you can't get it when using your fist I'd have to find a convoluted work around that would forego stone and add a new common resource to the world.

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.