[SOLVED]Flint from Stone

[Zelpher]

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.

[coolboy4531]

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

[Zelpher]

  On 9/13/2014 at 5:06 AM, coolboy4531 said:

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

[coolboy4531]

You got to check if heldItem is null in the first place. (*insert null check*)

 

if (heldItem != null)
--continue with code--

[Zelpher]

Edit: Is there another way to check if the player is using a pickaxe without crashing the game?

 

Also, null didn't remedy the situation.

[knokko]

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.

[Zelpher]

  On 9/13/2014 at 6:10 AM, knokko said:

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.

[knokko]

Just use if(player.getHeldItem == null){

Do something

}