Jump to content

[Solved] On kill of zombie


Betterjakers

Recommended Posts

I did this, and it does not seem to work. I registered it to the event bus, so it should be working. Here is my class:

 

package mymod.quests;

 

import net.minecraft.entity.player.EntityPlayer;

import net.minecraftforge.event.ForgeSubscribe;

import net.minecraftforge.event.entity.living.LivingDeathEvent;

 

public class Test {

 

@ForgeSubscribe

public void onEntityDeath(LivingDeathEvent event) {

if(event.entityLiving instanceof EntityPlayer) {

 

    System.out.println("Hello!");

   

}

   

    }

 

}

Link to comment
Share on other sites

In your initialization method you need to call MinecraftForge.EVENT_BUS.register(new EventHandlerClass());

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

Now that I look at your code you are checking to see if the entity that died is an instance of EntityPlayer not entity Zombie. Sorry I read that post wrong. ANd to check if it was the player that killed it, use the damagesource object.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

Why are you still using 1.6.4? And Instead of MinecraftForge.EVENT_BUS.register(new EventHandlerClass) use FMLCommonHandler.instance().bus().register(new EventHandlerClass);

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

The first version you posted should be printing to the console each time a zombie dies. To test if your event handler is working at all, you can put a println at the very beginning of it:

@ForgeSubscribe
public void onEntityDeath(LivingDeathEvent event) {
  System.out.println("Entity died: " + event.entity);
}

If you're not getting any output at all, make sure you actually call that random static init1 method you posted... it'd be simpler just to register directly during one of the FML events.

 

Edit: No, don't use FMLCommonHandler event bus - that's the wrong bus for this event. Here's a tutorial  - please read it so we don't have to repeat the same information over and over again.

Link to comment
Share on other sites

Oh thank you Cool Alias I completely forgot what goes where when dealing with Events thanks for the refresher, now if you'll excuse me I've got some reading to refresh on.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

No worries - that's the reason they finally consolidated the event buses, it was a mess keeping track of them.

 

@OP As we've said, you have to check if the DamageSource#getEntity is an EntityPlayer, then cast that entity to EntityPlayer so you can call I believe #addChatMessage to send them a chat - it's been a long time since I've looked at 1.6.4 and the chat methods have changed a lot since then, so you may have to look for it yourself.

Link to comment
Share on other sites

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.