[1.9] How do I get position of a block that the player is looking at?

[23kcarlson]

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);
}

[23kcarlson]

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.)

[Draco18s]

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.

[23kcarlson]

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.

[23kcarlson]

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

 

Edit: I could probably also use @SideOnly

[23kcarlson]

Hmm I guess I misinterpreted what I read previously

[Draco18s]

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.

[Draco18s]

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.