I have tried searching, but everything is outdated, and I cannot figure out how to get it to work.

 

I think this is as close as I got:

public ActionResult<ItemStack> onItemRightClick(ItemStack itemStackIn, World worldIn, EntityPlayer playerIn,
		EnumHand hand) {
         Vec3d vec3 = playerIn.getPositionEyes(1.0F);
         Vec3d lookVec = playerIn.getLook(1.0F);
         Vec3d addedVector = vec3.addVector(lookVec.xCoord * 50.0D, lookVec.yCoord * 50.0D, lookVec.zCoord * 50.0D);
         RayTraceResult lookPos= new  RayTraceResult(playerIn, lookVec);
         int x =lookPos.getBlockPos().getX();
         int y =lookPos.getBlockPos().getY();
         int z =lookPos.getBlockPos().getX();

         EntityLightningBolt lightning = new EntityLightningBolt(worldIn, x,y,z, false);
	worldIn.addWeatherEffect(lightning);
	return new ActionResult(EnumActionResult.PASS, itemStackIn);
}

1. I just modified the code from the Item class? What should I return? Nothing, and make it a void method?

 

2. I forgot about that, I'll fix that. Should it only be run on the server? or the client?

 

 

 

(I just started learning how to mod btw.)

I have tried searching, but everything is outdated, and I cannot figure out how to get it to work.

 

I think this is as close as I got

 

Your code looks 1 block away from where the player is and goes "yep, spawn there. good enough."

 

You never attempt to raytrace farther out.

I got it to work

 

Here is what I have now:

public ActionResult<ItemStack> onItemRightClick(ItemStack itemStackIn, World worldIn, EntityPlayer playerIn,

EnumHand hand) {

if (worldIn.isRemote) {

 

RayTraceResult result = playerIn.rayTrace(100, 1F);

int x = result.getBlockPos().getX();

int y = result.getBlockPos().getY();

int z = result.getBlockPos().getZ();

 

EntityLightningBolt lightning = new EntityLightningBolt(worldIn, x, y, z, false);

worldIn.addWeatherEffect(lightning);

}

return new ActionResult(EnumActionResult.SUCCESS, itemStackIn);

}

 

If this can be improved, please let me know.

I am only spawning it if worldIn.isRemote is true. Wouldn't that be the server side?

 

Edit: I could probably also use @SideOnly

Hmm I guess I misinterpreted what I read previously

isRemote should have been called isClient, but people like Lex think that people should instinctively know that "remote" means "relative to the server."  Or something, I don't recall the reasons only that it wasn't going to change.

 

There was an issue about it on the git, but I can't find it now.

The reason it is not changed is because everyone knows it now and it's so ubiquitously used.

 

That didn't stop some of the other name changes that occurred.

But whatever.