What do I add to this exact code to make it so that the pig will only drop a "pig soul" when holding (a.k.a killing the pig with) a certain sword?

 

//Pig Soul
@SubscribeEvent
public void addEntityDrop5 (LivingDropsEvent event) {
	if (event.entity instanceof EntityPig) {
		ItemStack itemstack = new ItemStack(MASouls.PigSoul, 1);
		event.drops.add(new EntityItem(event.entity.worldObj, event.entity.posX, event.entity.posY, event.entity.posZ, itemstack));
	}
}

What do I add to this exact code to make it so that the pig will only drop a "pig soul" when holding (a.k.a killing the pig with) a certain sword?

 

//Pig Soul
@SubscribeEvent
public void addEntityDrop5 (LivingDropsEvent event) {
	if (event.entity instanceof EntityPig) {
		ItemStack itemstack = new ItemStack(MASouls.PigSoul, 1);
		event.drops.add(new EntityItem(event.entity.worldObj, event.entity.posX, event.entity.posY, event.entity.posZ, itemstack));
	}
}

• Do not add to

  event.drops

  directly, simply use the

  Entity#entityDropItem

  method.

 

Unless I'm mistaken, that won't add to the event's drops list.

Entity#entityDropItem

only captures the drop when

Entity#captureDrops

is

true

, this is set to

true

before the entity's loot is dropped and then set back to

false

before

LivingDropsEvent

is fired.

• Do not add to

  event.drops

  directly, simply use the

  Entity#entityDropItem

  method.

 

Unless I'm mistaken, that won't add to the event's drops list.

Entity#entityDropItem

only captures the drop when

Entity#captureDrops

is

true

, this is set to

true

before the entity's loot is dropped and then set back to

false

before

LivingDropsEvent

is fired.

So if D7 was wrong what should I do?

So if D7 was wrong what should I do?

I have this code for a pickaxe where I mine the block and it removes the current drops and adds in another, and this only works if using the specific pickaxe, this is what I want to do for the sword against mobs but can't figure it out?

 

//Blaze Pickaxe Blocks
//Cobblestone Into Stone
@SubscribeEvent
public void addBlockDrop1 (HarvestDropsEvent event) {
	if (event.state.getBlock() == Blocks.cobblestone) {

		ItemStack holding = event.harvester.inventory.getStackInSlot(event.harvester.inventory.currentItem);
		if (holding != null && holding.getItem() instanceof MABlazePickaxeTool) {
			event.drops.clear();
			event.drops.add(new ItemStack(Blocks.stone));
		}
	}
}

I have this code for a pickaxe where I mine the block and it removes the current drops and adds in another, and this only works if using the specific pickaxe, this is what I want to do for the sword against mobs but can't figure it out?

 

//Blaze Pickaxe Blocks
//Cobblestone Into Stone
@SubscribeEvent
public void addBlockDrop1 (HarvestDropsEvent event) {
	if (event.state.getBlock() == Blocks.cobblestone) {

		ItemStack holding = event.harvester.inventory.getStackInSlot(event.harvester.inventory.currentItem);
		if (holding != null && holding.getItem() instanceof MABlazePickaxeTool) {
			event.drops.clear();
			event.drops.add(new ItemStack(Blocks.stone));
		}
	}
}

Your prior code should work. I don't see anything wrong with it.

Your prior code should work. I don't see anything wrong with it.

It does work, but the pig soul always drops no matter what where I want it to drop when only holding a specific sword, but have no clue how to do that?

It does work, but the pig soul always drops no matter what where I want it to drop when only holding a specific sword, but have no clue how to do that?

Oh you didn't check what sword the player is holding.  This line equivalent:

if (holding != null && holding.getItem() instanceof MABlazePickaxeTool) {

Oh you didn't check what sword the player is holding.  This line equivalent:

if (holding != null && holding.getItem() instanceof MABlazePickaxeTool) {

So is that all I have to do is add this in:

 

if (holding != null && holding.getItem() instanceof MABlazePickaxeTool) {

but change "MABlazePickaxeTool" to the swords class? And that's literally it? Because I can see this going towards me trying it like that then it not working asking a stupid question and people moan at me?

So is that all I have to do is add this in:

 

if (holding != null && holding.getItem() instanceof MABlazePickaxeTool) {

but change "MABlazePickaxeTool" to the swords class? And that's literally it? Because I can see this going towards me trying it like that then it not working asking a stupid question and people moan at me?

Yep

Yep

Okay what about the "holding" variable? What would I set that to?

 

New current code:

 

//Cow Soul
	@SubscribeEvent
	public void addEntityDrop50 (LivingDropsEvent event) {
		if (event.entity instanceof EntityCow) {
			if (holding != null && holding.getItem() instanceof MASoulHarvester) {
			ItemStack itemstack = new ItemStack(MASouls.CowSoul, 1);
			event.drops.add(new EntityItem(event.entity.worldObj, event.entity.posX, event.entity.posY, event.entity.posZ, itemstack));
		}
	}
}

Okay what about the "holding" variable? What would I set that to?

 

New current code:

 

//Cow Soul
	@SubscribeEvent
	public void addEntityDrop50 (LivingDropsEvent event) {
		if (event.entity instanceof EntityCow) {
			if (holding != null && holding.getItem() instanceof MASoulHarvester) {
			ItemStack itemstack = new ItemStack(MASouls.CowSoul, 1);
			event.drops.add(new EntityItem(event.entity.worldObj, event.entity.posX, event.entity.posY, event.entity.posZ, itemstack));
		}
	}
}

Never mind I figured it outs, thanks a lot

 

Edit: Actually there's on more question, I'm now using this as my code for the specific sword:

 

//Cow Soul
	@SubscribeEvent
	public void addEntityDrop50 (LivingDropsEvent event) {
		if (event.entity instanceof EntityCow) {
			Object holding = new ItemStack(MAItems.SoulHarvester);
			if (holding  != null && ((ItemStack) holding).getItem() instanceof MASoulHarvester) {
			ItemStack itemstack = new ItemStack(MASouls.CowSoul, 1);
			event.drops.add(new EntityItem(event.entity.worldObj, event.entity.posX, event.entity.posY, event.entity.posZ, itemstack));
		}
	}
}

 

What code do I add, and where do I add it for a drop chance? So like if I wanted the soul to only drop 10% of the time how would I do this?

Never mind I figured it outs, thanks a lot

 

Edit: Actually there's on more question, I'm now using this as my code for the specific sword:

 

//Cow Soul
	@SubscribeEvent
	public void addEntityDrop50 (LivingDropsEvent event) {
		if (event.entity instanceof EntityCow) {
			Object holding = new ItemStack(MAItems.SoulHarvester);
			if (holding  != null && ((ItemStack) holding).getItem() instanceof MASoulHarvester) {
			ItemStack itemstack = new ItemStack(MASouls.CowSoul, 1);
			event.drops.add(new EntityItem(event.entity.worldObj, event.entity.posX, event.entity.posY, event.entity.posZ, itemstack));
		}
	}
}

 

What code do I add, and where do I add it for a drop chance? So like if I wanted the soul to only drop 10% of the time how would I do this?

Do you actually understand what your code is doing?

 

You're creating a new

ItemStack

of

MAItems.SoulHarvester

and then immediately checking if it's an

ItemStack

of

MAItems.SoulHarvester

. This will always be

true

, so it's completely pointless.

 

You need to get the entity that did the killing from the

DamageSource

and then check its held item (like diesieben07 told you in the first reply).

 

For random chances, use

Random#nextInt

to generate a random number, then check if it's in a specific range; e.g.

random.nextInt(100) < 10

will be

true

10% of the time.

Do you actually understand what your code is doing?

 

You're creating a new

ItemStack

of

MAItems.SoulHarvester

and then immediately checking if it's an

ItemStack

of

MAItems.SoulHarvester

. This will always be

true

, so it's completely pointless.

 

You need to get the entity that did the killing from the

DamageSource

and then check its held item (like diesieben07 told you in the first reply).

 

For random chances, use

Random#nextInt

to generate a random number, then check if it's in a specific range; e.g.

random.nextInt(100) < 10

will be

true

10% of the time.

My way still works fine for making it drop when using the sword only though? Why change it? And okay thanks for the chance to drop help I'll have a look at what I can do.