Jump to content

[1.8.9]attackEntityFrom() not working


captaincleric

Recommended Posts

So I'm trying to make an anti-Undead spell that deals direct damage to the Undead, but I'm having a problem actually dealing damage.

attackEntityFrom() seems to fail on the worldObj.isRemote check, indicating that the world is indeed, remote. When I put in a check for it, it fails it and attackEntityFrom() never gets called.

 

What can I do about this?

 

Code posted below.

@Override
public ItemStack onItemUseFinish(ItemStack itemStack, World world, EntityPlayer player)
{
	int mana = 0;
	ItemStack wand = FindWand(player);

	if(wand != null)
	{
		if(wand.getTagCompound() != null)
		{
			mana = wand.getTagCompound().getInteger("mana");
		}
		if(mana >= COST)
		{
			int distance = 64;

			//MovingObjectPosition position = Minecraft.getMinecraft().getRenderViewEntity().rayTrace(distance, 1.0f);
			Entity target = Minecraft.getMinecraft().objectMouseOver.entityHit; 

			if(target != null)
			{					
				if(target instanceof EntityLiving)
				{
					EntityLiving livingTarget = (EntityLiving)target;
					if(livingTarget.getCreatureAttribute() == EnumCreatureAttribute.UNDEAD)
					{
						ItemBaseWand castWand = (ItemBaseWand)wand.getItem();

						livingTarget.attackEntityFrom(DamageSource.inFire, DAMAGE);
						livingTarget.setFire(10);

						if(!world.isRemote)
						{	
							castWand.ModMana(wand, -COST);
						}
					}
				}
			}
		}
	}
	return itemStack;
}

Link to comment
Share on other sites

You code has logical mistakes:

 

1st things 1st - why does "FindWand" starts with capital :C

 

Back to case:

Minecraft class is @SideOnly(Side.CLIENT) - therefore your item code will crash on dedicated.jar startup.

I am refering to this: Minecraft.getMinecraft().objectMouseOver.entityHit - this is for client-only usage and only via proxy code.

This is technical mistake. Based on this mistake you make next one:

You are using Client-sided code from server logical thread (!world.isRemote).

And calling data-changing methods from client logical side (data can be only manipulated on server thread) - I am talking about applying damage.

 

Some reading:

http://www.minecraftforge.net/forum/index.php/topic,33918.msg178740.html#msg178740

http://www.minecraftforge.net/forum/index.php/topic,22764.0.html

 

Now: In common code (Item's methods) you can perform proxy calls that can use Minecraft class - but that won't give you shit, because you don't need and can't do anything from client logical thread (only server can actually edit real data, client only displays it). What you need is to inside (!world.isRemote) call a method that will obtain target entity using server logical code.

 

To do that you could lookup e.g Arrow's code (for hitting targets) or you can almost literally copy client's method that does that, but do it on server thread.

1.7.10 is no longer supported by forge, you are on your own.

Link to comment
Share on other sites

You code has logical mistakes:

 

1st things 1st - why does "FindWand" starts with capital :C

It's just personal preference, and something drilled into me by my work. Variables start with lowercase, functions start with a capital.

 

Thanks! Very helpful.

 

Now: In common code (Item's methods) you can perform proxy calls that can use Minecraft class - but that won't give you shit, because you don't need and can't do anything from client logical thread (only server can actually edit real data, client only displays it). What you need is to inside (!world.isRemote) call a method that will obtain target entity using server logical code.

 

To do that you could lookup e.g Arrow's code (for hitting targets) or you can almost literally copy client's method that does that, but do it on server thread.

There's a problem there; using an axis aligned bounding box needs an Entity. I only have an Item. I'd need to create or get (I don't know from where) an Entity to use that. Could I use the player?

Link to comment
Share on other sites

You code has logical mistakes:

 

1st things 1st - why does "FindWand" starts with capital :C

It's just personal preference, and something drilled into me by my work. Variables start with lowercase, functions start with a capital.

 

Really? For Java? That isn't really typical. Of course the compiler doesn't care and so the code will work fine, but whenever you share code (like how you're asking for help here, it hurts the other human's ability to quickly read your code).

Check out my tutorials here: http://jabelarminecraft.blogspot.com/

Link to comment
Share on other sites

Fixed it, finally. For those hoping to achieve similar results, try the following:

 

A simple function to find a target:

protected MovingObjectPosition FindTarget(EntityPlayer player, World world)
{		
Vec3 playerPosition = new Vec3(player.posX, player.posY + player.getEyeHeight(), player.posZ);
Vec3 normalisedLook = player.getLookVec().normalize();

Vec3 targetPosition = new Vec3(playerPosition.xCoord + (normalisedLook.xCoord * DISTANCE),
		playerPosition.yCoord + player.eyeHeight + (normalisedLook.yCoord * DISTANCE), 
		playerPosition.zCoord + (normalisedLook.zCoord * DISTANCE));

MovingObjectPosition position = world.rayTraceBlocks(playerPosition, targetPosition, true, true, false);

return position;
}

 

If you want to find all entities within an area, simply use the World.getEntitiesWithinAABBExcludingEntity(player, box) (I used player as I had no other entity available), where 'box' is a bounding box assembled from the data you get from your FindTarget call.

 

Hope this helps!

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



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • how to make animated doors?, maybe geckolib, but i don't know how to code it?
    • For crash 1, set max-tick-time to -1 in your server.properties Crash 2 shows a conflict or incompatibility between LuckPerms and the mod boh-0.0.6.1-forge-1.20.1_2.jar
    • Add the crash-report or latest.log (logs-folder) with sites like https://mclo.gs/ and paste the link to it here  
    • so my minecraft crashes when opening my world, i played without any troubles for about 5 days and today it started tweaking.. pls help me
    • Hi guys! I am having some issues with the server crashing over and over and I was hoping to get some guidance.  Thanks in advance! Crash 1: java.lang.Error: ServerHangWatchdog detected that a single server tick took 60.00 seconds (should be max 0.05)     at net.minecraft.server.dedicated.ServerWatchdog.run(ServerWatchdog.java:43) ~[server-1.20.1-20230612.114412-srg.jar%23217!/:?] {re:classloading}     at java.lang.Thread.run(Thread.java:840) ~[?:?] { Crash 2: java.lang.IllegalStateException: Capability missing for eeb7f026-34b4-42f5-9164-e7736461df83     at me.lucko.luckperms.forge.capabilities.UserCapabilityImpl.lambda$get$0(UserCapabilityImpl.java:66) ~[?:?] {re:classloading,re:classloading,re:classloading}     at net.minecraftforge.common.util.LazyOptional.orElseThrow(LazyOptional.java:261) ~[forge-1.20.1-47.3.10-universal.jar%23222!/:?] {re:mixin,re:classloading}     at me.lucko.luckperms.forge.capabilities.UserCapabilityImpl.get(UserCapabilityImpl.java:66) ~[?:?] {re:classloading,re:classloading,re:classloading}     at me.lucko.luckperms.forge.util.BrigadierInjector$InjectedPermissionRequirement.test(BrigadierInjector.java:143) ~[?:?] {}     at me.lucko.luckperms.forge.util.BrigadierInjector$InjectedPermissionRequirement.test(BrigadierInjector.java:129) ~[?:?] {}     at com.mojang.brigadier.tree.CommandNode.canUse(CommandNode.java:65) ~[brigadier-1.1.8.jar%2376!/:?] {}     at com.mojang.brigadier.CommandDispatcher.parseNodes(CommandDispatcher.java:359) ~[brigadier-1.1.8.jar%2376!/:?] {}     at com.mojang.brigadier.CommandDispatcher.parse(CommandDispatcher.java:349) ~[brigadier-1.1.8.jar%2376!/:?] {}     at com.mojang.brigadier.CommandDispatcher.parse(CommandDispatcher.java:317) ~[brigadier-1.1.8.jar%2376!/:?] {}     at net.minecraft.commands.Commands.m_230957_(Commands.java:237) ~[server-1.20.1-20230612.114412-srg.jar%23217!/:?] {re:classloading}     at net.mcreator.boh.procedures.TeleportbenProcedure.lambda$execute$2(TeleportbenProcedure.java:65) ~[boh-0.0.6.1-forge-1.20.1_2.jar%23165!/:?] {re:classloading}     at net.mcreator.boh.BohMod.lambda$tick$2(BohMod.java:96) ~[boh-0.0.6.1-forge-1.20.1_2.jar%23165!/:?] {re:classloading}     at java.util.ArrayList.forEach(ArrayList.java:1511) ~[?:?] {re:mixin}     at net.mcreator.boh.BohMod.tick(BohMod.java:96) ~[boh-0.0.6.1-forge-1.20.1_2.jar%23165!/:?] {re:classloading}     at net.mcreator.boh.__BohMod_tick_ServerTickEvent.invoke(.dynamic) ~[boh-0.0.6.1-forge-1.20.1_2.jar%23165!/:?] {re:classloading,pl:eventbus:B}     at net.minecraftforge.eventbus.ASMEventHandler.invoke(ASMEventHandler.java:73) ~[eventbus-6.0.5.jar%2352!/:?] {}     at net.minecraftforge.eventbus.EventBus.post(EventBus.java:315) ~[eventbus-6.0.5.jar%2352!/:?] {}     at net.minecraftforge.eventbus.EventBus.post(EventBus.java:296) ~[eventbus-6.0.5.jar%2352!/:?] {}     at net.minecraftforge.event.ForgeEventFactory.onPostServerTick(ForgeEventFactory.java:950) ~[forge-1.20.1-47.3.10-universal.jar%23222!/:?] {re:classloading}     at net.minecraft.server.MinecraftServer.m_5705_(MinecraftServer.java:835) ~[server-1.20.1-20230612.114412-srg.jar%23217!/:?] {re:mixin,pl:accesstransformer:B,xf:fml:xaerominimap:xaero_minecraftserver,re:classloading,pl:accesstransformer:B,xf:fml:xaerominimap:xaero_minecraftserver,pl:mixin:A}     at net.minecraft.server.MinecraftServer.m_130011_(MinecraftServer.java:661) ~[server-1.20.1-20230612.114412-srg.jar%23217!/:?] {re:mixin,pl:accesstransformer:B,xf:fml:xaerominimap:xaero_minecraftserver,re:classloading,pl:accesstransformer:B,xf:fml:xaerominimap:xaero_minecraftserver,pl:mixin:A}     at net.minecraft.server.MinecraftServer.m_206580_(MinecraftServer.java:251) ~[server-1.20.1-20230612.114412-srg.jar%23217!/:?] {re:mixin,pl:accesstransformer:B,xf:fml:xaerominimap:xaero_minecraftserver,re:classloading,pl:accesstransformer:B,xf:fml:xaerominimap:xaero_minecraftserver,pl:mixin:A}     at java.lang.Thread.run(Thread.java:840) ~[?:?] {}
  • Topics

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.