After some debugging I still can't figure out why this code (code below) don't print expected - when print line was outside if, it also didn't work

@Subscribe
public void onLivingDeathEvent (LivingDeathEvent deadEntity){

    if ( deadEntity.getSource() == DamageSource.DROWN) {
        System.out.println("Entity died: drown");
    }
    else{
        System.out.println("Entity died: other reasons");
    }

}

 

This code is in main class, class is registered to MinecraftForge.EVENT_BUS 

Thanks for your time  

How can I avoid it? And second question it should still work? Thanks for reply tho

Ohh sorry , now I reread it, and I tried with same, but static method:

@Subscribe
public static void onLivingDeathEvent (LivingDeathEvent deadEntity){

    if ( deadEntity.getSource() == DamageSource.DROWN) {
        System.out.println("Entity died: drown");
    }
    else{
        System.out.println("Entity died: other reasons");
    }

}

And it still doesn't work - I am confused, why should it be static?

 

 

package com.saltbeard.potc;

import com.google.common.eventbus.Subscribe;
import com.saltbeard.potc.util.RegistryBlocksHandler;
import com.saltbeard.potc.util.RegistryItemsHandler;
import net.minecraft.item.ItemGroup;
import net.minecraft.item.ItemStack;
import net.minecraft.util.DamageSource;
import net.minecraftforge.common.MinecraftForge;
import net.minecraftforge.event.entity.living.LivingDeathEvent;
import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.fml.event.lifecycle.FMLClientSetupEvent;
import net.minecraftforge.fml.event.lifecycle.FMLCommonSetupEvent;
import net.minecraftforge.fml.javafmlmod.FMLJavaModLoadingContext;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;

@Mod("potc")
public class PotC
{
    private static final Logger LOGGER = LogManager.getLogger();
    public static final String MOD_ID = "potc";

    public PotC() {
        FMLJavaModLoadingContext.get().getModEventBus().addListener(this::setup);
        FMLJavaModLoadingContext.get().getModEventBus().addListener(this::doClientStuff);

        MinecraftForge.EVENT_BUS.register(this);

        RegistryItemsHandler.init();
        RegistryBlocksHandler.init();

    }



    private void setup(final FMLCommonSetupEvent event){
    }

    private void doClientStuff(final FMLClientSetupEvent event) {
    }

    public static final ItemGroup MISC_TAB = new ItemGroup("miscTab"){

        @Override
        public ItemStack createIcon(){

            return new ItemStack(RegistryItemsHandler.JADE.get());

        }

    };

    @Subscribe
    public static void onLivingDeathEvent (LivingDeathEvent deadEntity){

        if ( deadEntity.getSource() == DamageSource.DROWN) {
            System.out.println("Entity died: drown");
        }
        else{
            System.out.println("Entity died: other reasons");
        }

    }
}

-

Edited December 29, 20204 yr by SaltBeard

Oh I am an idiot, thank you very much for patience