onEntityJoinWorldEvent() being buggy with EntityItem

[apuppy120]

I'm trying to get a specific item to kill the player or a nearby player if it exists as an EntityItem in world.

The problem is that, even though I am making a check for it, it is running the kill code for all items.

 

 

 

package com.spectrum.handlers;

import com.spectrum.item.ASItem;

import net.minecraft.entity.EntityLivingBase;
import net.minecraft.entity.item.EntityItem;
import net.minecraft.entity.item.EntityTNTPrimed;
import net.minecraft.init.Items;
import net.minecraft.util.DamageSource;
import net.minecraftforge.event.entity.EntityEvent;
import net.minecraftforge.event.entity.EntityJoinWorldEvent;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;

public class AntiwaveEntityHandler {

@SubscribeEvent
public void onEntityJoinWorldEvent(EntityJoinWorldEvent event)
{

	 if (event.entity instanceof EntityItem){
		 EntityItem entity = (EntityItem)event.entity;
		 if(entity.getEntityItem() != null)
		 {

			 if(entity.getEntityItem().getItem().getUnlocalizedName().equals(ASItem.antiItem.getUnlocalizedName()));
			 {


				 event.setCanceled(true);


				 entity.worldObj.getClosestPlayerToEntity(entity, 10).attackEntityFrom(DamageSource.outOfWorld, 1000.0F);
				 entity.worldObj.getClosestPlayerToEntity(entity, 10).experienceLevel = 0;
				 entity.setDead();



			 }
		 }


	 }

}


}

 

 

 

Any help?

[LexManos]

Why are you using 'unlocalized names'?

 

[apuppy120]

I saw something about it in someone else's similar problem that they ended up fixing, so I thought that I would test it.

Here is the original code:

if(entity.getEntityItem().getItem() == ASItem.antiItem) {...}

 

 

EDIT: Nevermind, I found the problem. There was a semicolon at the end of an if() statement where there shouldn't be.

[Draco18s]

EDIT: Nevermind, I found the problem. There was a semicolon at the end of an if() statement where there shouldn't be.

 

Hehe. I had that exact same problem, but with a loop yesterday.