Inaccurate lightning strikes?

[TLHPoE]

I just semi-finished my lightning sword. It basically shoots lightning where the player is pointing when they right click. Only problem is that the flames from the lightning strike spawn right next to the player. Is there anyway to stop this?

 

    public ItemStack onItemRightClick(ItemStack par1ItemStack, World world, EntityPlayer player)
    {
    	
    	Random r = new Random();
    	
        player.setItemInUse(par1ItemStack, this.getMaxItemUseDuration(par1ItemStack));
        
        par1ItemStack.damageItem(3, player);
        
        Vec3 vec3 = player.getPosition(1.0F);
        Vec3 vec3a = player.getLook(1.0F);
        Vec3 vec3b = vec3.addVector(vec3a.xCoord * 64, vec3a.yCoord * 64, vec3a.zCoord * 64);

        MovingObjectPosition movingobjectposition = world.clip(vec3, vec3b);
        
        if(movingobjectposition != null) //null check
        {
        	
            world.spawnEntityInWorld(new EntityLightningBolt(world, movingobjectposition.blockX, movingobjectposition.blockY, movingobjectposition.blockZ));
        	
        }
        
        return par1ItemStack;
        
    }

[Mew]

I believe that that is set in the actual entity file itself. You could always just go through all the fire blocks within a certain radius around the player and clear the them every time you click... But that is a bit inefficient. So I am not to sure on this...

[Draco18s]

Huh.  I'm not having that problem.

 

Though I did have a weird problem where the item onRightClick method would only be partially functional; as if it was running client-side only (that is, it was running on the server side as well, but that changes to entities wouldn't stick,* as if they'd been done client side).

 

*My one test before I moved over to a packet based system dealt with healing the player with that function.  I didn't try other effects, like lightning and so on, simply went "fine, this isn't working as expected" and shoved everything into packets.

[f1rSt1kChannel]

My sword:

 

public ItemStack onItemRightClick(ItemStack par1ItemStack, World par2World, EntityPlayer par3EntityPlayer)
    {
    	 float f = 1.0F;
         float f1 = par3EntityPlayer.prevRotationPitch + (par3EntityPlayer.rotationPitch - par3EntityPlayer.prevRotationPitch) * f;
         float f2 = par3EntityPlayer.prevRotationYaw + (par3EntityPlayer.rotationYaw - par3EntityPlayer.prevRotationYaw) * f;
         
         double d = par3EntityPlayer.prevPosX + (par3EntityPlayer.posX - par3EntityPlayer.prevPosX) * (double)f;
         double d1 = (par3EntityPlayer.prevPosY + (par3EntityPlayer.posY - par3EntityPlayer.prevPosY) * (double)f + 1.6200000000000001D) - (double)par3EntityPlayer.yOffset;
         double d2 = par3EntityPlayer.prevPosZ + (par3EntityPlayer.posZ - par3EntityPlayer.prevPosZ) * (double)f;
         
         Vec3 vec3d = Vec3.createVectorHelper(d, d1, d2);
         
         float f3 = MathHelper.cos(-f2 * 0.01745329F - 3.141593F);
         float f4 = MathHelper.sin(-f2 * 0.01745329F - 3.141593F);
         float f5 = -MathHelper.cos(-f1 * 0.01745329F);
         float f6 = MathHelper.sin(-f1 * 0.01745329F);
         float f7 = f4 * f5;
         float f8 = f6;
         float f9 = f3 * f5;
         
         double d3 = 5000D;
        
         Vec3 vec3d1 = vec3d.addVector((double)f7 * d3, (double)f8 * d3, (double)f9 * d3);
         
         MovingObjectPosition movingobjectposition = par2World.rayTraceBlocks_do_do(vec3d, vec3d1, false, true);
         
         if (movingobjectposition == null){
         
        	 return par1ItemStack;
         }
         
         if (movingobjectposition.typeOfHit == EnumMovingObjectType.TILE){
         
        	 int i = movingobjectposition.blockX;
             int j = movingobjectposition.blockY;
             int k = movingobjectposition.blockZ;
             
             for(int m = 0; m < 5; m++){
             
            	 par2World.spawnEntityInWorld(new EntityLightningBolt(par2World, i, j, k));
             }
         
             par1ItemStack.damageItem(20, par3EntityPlayer);
         }
         return par1ItemStack;
     }

[Draco18s]

Looks just about the same as what I do.