Why doesnt this work? 1.7.10

[BoonieQuafter-CrAfTeR]

so as most of the moderators probably know im making a mob that CAN attackoncollide if he is 4 blocks away from the target, being able to edit the combat task to do just so would make my life about 10 times easier, so if anybody would like to tell me why this doesn't work, it would be greatly appreciated, thank you.

 

why doesn't it work?

    public void setCombatTask()
    {
       

        if (this.getAttackTarget().getDistanceSqToEntity(this) > 16.0D)
        {
            this.tasks.addTask(4, this.aiArrowAttack);
        }
        else
        {
            this.tasks.addTask(4, this.aiAttackOnCollide);
        }
    }

 

crash report:

A detailed walkthrough of the error, its code path and all known details is as follows:
---------------------------------------------------------------------------------------

-- Head --
Stacktrace:
at com.OlympiansMod.entity.Zeus.setCombatTask(Zeus.java:150)
at com.OlympiansMod.entity.Zeus.<init>(Zeus.java:76)
at com.OlympiansMod.Item.ZeusSpawn.onItemRightClick(ZeusSpawn.java:18)
at net.minecraft.item.ItemStack.useItemRightClick(ItemStack.java:164)
at net.minecraft.server.management.ItemInWorldManager.tryUseItem(ItemInWorldManager.java:345)
at net.minecraft.network.NetHandlerPlayServer.processPlayerBlockPlacement(NetHandlerPlayServer.java:576)
at net.minecraft.network.play.client.C08PacketPlayerBlockPlacement.processPacket(C08PacketPlayerBlockPlacement.java:74)
at net.minecraft.network.play.client.C08PacketPlayerBlockPlacement.processPacket(C08PacketPlayerBlockPlacement.java:122)
at net.minecraft.network.NetworkManager.processReceivedPackets(NetworkManager.java:241)

-- Ticking connection --
Details:
Connection: net.minecraft.network.NetworkManager@447b1252
Stacktrace:
at net.minecraft.network.NetworkSystem.networkTick(NetworkSystem.java:182)
at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:726)
at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:614)
at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:118)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:485)
at net.minecraft.server.MinecraftServer$2.run(MinecraftServer.java:752)

-- System Details --
Details:
Minecraft Version: 1.7.10
Operating System: Windows 8.1 (amd64) version 6.3
Java Version: 1.8.0_45, Oracle Corporation
Java VM Version: Java HotSpot(TM) 64-Bit Server VM (mixed mode), Oracle Corporation
Memory: 679592152 bytes (648 MB) / 1038876672 bytes (990 MB) up to 1038876672 bytes (990 MB)
JVM Flags: 3 total; -Xincgc -Xmx1024M -Xms1024M
AABB Pool Size: 0 (0 bytes; 0 MB) allocated, 0 (0 bytes; 0 MB) used
IntCache: cache: 0, tcache: 0, allocated: 0, tallocated: 0
FML: MCP v9.05 FML v7.10.85.1230 Minecraft Forge 10.13.2.1230 4 mods loaded, 4 mods active
mcp{9.05} [Minecraft Coder Pack] (minecraft.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
FML{7.10.85.1230} [Forge Mod Loader] (forgeSrc-1.7.10-10.13.2.1230.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
Forge{10.13.2.1230} [Minecraft Forge] (forgeSrc-1.7.10-10.13.2.1230.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
theolympiansmod{0.1} [The Olympians Mod] (bin) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
Profiler Position: N/A (disabled)
Vec3 Pool Size: 0 (0 bytes; 0 MB) allocated, 0 (0 bytes; 0 MB) used
Player Count: 1 / 8; [EntityPlayerMP['Player959'/1478, l='New World', x=1139.40, y=241.78, z=1095.47]]
Type: Integrated Server (map_client.txt)
Is Modded: Definitely; Client brand changed to 'fml,forge'
[18:48:26] [Client thread/INFO] [sTDOUT]: [net.minecraft.client.Minecraft:displayCrashReport:393]: #@!@# Game crashed! Crash report saved to: #@!@# .\crash-reports\crash-2015-07-13_18.48.26-server.txt
[18:48:26] [Client thread/INFO] [FML]: Waiting for the server to terminate/save.
[18:48:28] [server thread/INFO] [FML]: Unloading dimension 0
[18:48:28] [server thread/INFO] [FML]: Unloading dimension -1
[18:48:28] [server thread/INFO] [FML]: Unloading dimension 1
[18:48:28] [server thread/INFO] [FML]: Applying holder lookups
[18:48:28] [server thread/INFO] [FML]: Holder lookups applied
[18:48:28] [server thread/INFO] [FML]: The state engine was in incorrect state SERVER_STOPPING and forced into state SERVER_STOPPED. Errors may have been discarded.
[18:48:28] [Client thread/INFO] [FML]: Server terminated.
AL lib: (EE) alc_cleanup: 1 device not closed
Java HotSpot(TM) 64-Bit Server VM warning: Using incremental CMS is deprecated and will likely be removed in a future release

 

this line of code is erroring it btw,

 if (this.getAttackTarget().getDistanceSqToEntity(this) > 16.0D)

[coolAlias]

You don't need to make a new thread every single time, especially when it is about the exact same topic as in your other thread in which you already asked this exact same question.

[TheRealMcrafter]

As coolAlias pointed out, he already told you in your other thread that was "SOLVED 100%" that you are not handling cases where the entity can be null.

[BoonieQuafter-CrAfTeR]

dude read the dates, I posted this one before the originally thread was completed

[TheRealMcrafter]

Okay, well coolAlias' point still goes, you dont need to make 2 threads about the same thing.