Better way of doing attackEntity?

[Gingerbreadman]

I am using player controller to attack like 90 entities in one block, the only problem is that it stops attacking them after a while

						if(Minecraft.getMinecraft().objectMouseOver.typeOfHit == MovingObjectType.ENTITY) {
						if(Minecraft.getMinecraft().objectMouseOver.entityHit instanceof EntityBlaze) {
							Minecraft.getMinecraft().playerController.attackEntity(Minecraft.getMinecraft().thePlayer, Minecraft.getMinecraft().objectMouseOver.entityHit);
							Minecraft.getMinecraft().thePlayer.swingItem();
						}

[coolAlias]

Each time you call playerController.attackEntity it sends a packet, among other things, not to mention objectMouseOver, which is the position of the mouse cursor on the screen, cannot possibly be hovering over 90 mobs at one time.

 

What exactly are you trying to do? Explain the logic of your attack code.

[TheRealMcrafter]

I believe he wants an auto killing mod, where as long as there is an entity in the crosshairs, attack it (like for a blaze farm, in his case)

[Gingerbreadman]

yes

[coolAlias]

So fetch all of the entities in a given radius around the player and kill them, or attackEntityFrom with a player-based damage source. You can fetch all of the entities using World#getEntitiesWithinAABB(EntityBlaze.class, player.getEntityBoundingBox().expand(8, 8, ) or however far you want.

 

Do it on the server and it will be much faster by not eating up tons of bandwidth with packets.

[Gingerbreadman]

I have them all in one spot, and I attack them, for some reason most hits dont register.

[Gingerbreadman]

Show your code.

its in the OP post.

[Gingerbreadman]

Don't do it on the client. Do it on the server and not based on mouseover but based on an AABB.

I don't have access to the server...

[coolAlias]

I don't have access to the server...

How can you not have access to the server side from your mod? Are you trying to make some kind of client hack?

 

Server side is when the world object is not remote, i.e.:

if (!worldObj.isRemote) {
  // you are on the server side right now
}