Acrogenous 1 Report post Posted December 14, 2017 I have legitimately simplified this code as much as possible to see if this was a parenthesis error, but to no avail @Override public boolean onBlockActivated(World world, BlockPos blockpos, IBlockState p_onBlockActivated_3_, EntityPlayer player, EnumHand hand, EnumFacing p_onBlockActivated_6_, float p_onBlockActivated_7_, float p_onBlockActivated_8_, float p_onBlockActivated_9_) { ItemStack stack = player.getHeldItem(hand); if (stack.hasTagCompound() && stack.getItem() == ModItems.logiCard && player.isSneaking()) { NBTTagCompound card = stack.getTagCompound(); card.setInteger("xpos", blockpos.getX()); card.setInteger("ypos", blockpos.getY()); card.setInteger("zpos", blockpos.getZ()); } if (stack.hasTagCompound() && stack.getItem() == ModItems.logiCard && !player.isSneaking()) { direction(stack, player, world, blockpos); } return super.onBlockActivated(world, blockpos, p_onBlockActivated_3_,player,hand, p_onBlockActivated_6_, p_onBlockActivated_7_, p_onBlockActivated_8_, p_onBlockActivated_9_); } it is fine with detecting if the player is not sneaking, but it does absolutely nothing when the player is sneaking (there is already a compound tag on the item i am using, this was created originally but the code is greatly simplified, as said above) . A further point is when I invert the statements still only the !isSneaking works. thanks in advance ~Acro Share this post Link to post Share on other sites
lethinh 1 Report post Posted December 15, 2017 3 hours ago, Acrogenous said: I have legitimately simplified this code as much as possible to see if this was a parenthesis error, but to no avail @Override public boolean onBlockActivated(World world, BlockPos blockpos, IBlockState p_onBlockActivated_3_, EntityPlayer player, EnumHand hand, EnumFacing p_onBlockActivated_6_, float p_onBlockActivated_7_, float p_onBlockActivated_8_, float p_onBlockActivated_9_) { ItemStack stack = player.getHeldItem(hand); if (stack.hasTagCompound() && stack.getItem() == ModItems.logiCard && player.isSneaking()) { NBTTagCompound card = stack.getTagCompound(); card.setInteger("xpos", blockpos.getX()); card.setInteger("ypos", blockpos.getY()); card.setInteger("zpos", blockpos.getZ()); } if (stack.hasTagCompound() && stack.getItem() == ModItems.logiCard && !player.isSneaking()) { direction(stack, player, world, blockpos); } return super.onBlockActivated(world, blockpos, p_onBlockActivated_3_,player,hand, p_onBlockActivated_6_, p_onBlockActivated_7_, p_onBlockActivated_8_, p_onBlockActivated_9_); } it is fine with detecting if the player is not sneaking, but it does absolutely nothing when the player is sneaking (there is already a compound tag on the item i am using, this was created originally but the code is greatly simplified, as said above) . A further point is when I invert the statements still only the !isSneaking works. thanks in advance ~Acro Dude, you have to first check if the ItemStack is not null first. Then do whatever you want Share this post Link to post Share on other sites
Silly511 9 Report post Posted December 15, 2017 1 hour ago, lethinh said: Dude, you have to first check if the ItemStack is not null first. Then do whatever you want In 1.12 item stacks are never null. Share this post Link to post Share on other sites
jabelar 584 Report post Posted December 15, 2017 To debug this sort of thing is easy -- just put console (system.out.println()) statements in the code in judicious places and then see what gets printed. I personally put a print statement immediately before the if which prints out the value of every term, and then put a print statement in each path so I can confirm which path was actually executed. with that information it is very easy to see what is going wrong. For example one thing in your code is you're not checking which hand, so it is probably running for both hands, and maybe that is somehow screwing things up a bit. Share this post Link to post Share on other sites
lethinh 1 Report post Posted December 15, 2017 8 hours ago, Silly511 said: In 1.12 item stacks are never null. Come on dude, you understand that. Check if the ItemStack is not EMPTY (ItemStack.EMPTY) or by using ItemStack#isEmpty. Share this post Link to post Share on other sites
jabelar 584 Report post Posted December 15, 2017 6 hours ago, lethinh said: Check if the ItemStack is not EMPTY (ItemStack.EMPTY) or by using ItemStack#isEmpty. Using ItemStack#isEmpty is best because it checks for malformed Itemstacks as well (including somehow being null which can still happen in mods if modder uses old habits or code when emptying a stack). Share this post Link to post Share on other sites
Draco18s 1928 Report post Posted December 15, 2017 1 hour ago, jabelar said: Using ItemStack#isEmpty is best because it checks for malformed Itemstacks as well (including somehow being null which can still happen in mods if modder uses old habits or code when emptying a stack). ItemStack#isEmpty does not work on null stacks. It will crash. Share this post Link to post Share on other sites
Acrogenous 1 Report post Posted December 15, 2017 so are we reckoning isSneaking() isn't working because I haven't checked if the ItemStack is null? Share this post Link to post Share on other sites
Acrogenous 1 Report post Posted December 15, 2017 11 hours ago, jabelar said: To debug this sort of thing is easy -- just put console (system.out.println()) statements in the code in judicious places and then see what gets printed. I personally put a print statement immediately before the if which prints out the value of every term, and then put a print statement in each path so I can confirm which path was actually executed. with that information it is very easy to see what is going wrong. For example one thing in your code is you're not checking which hand, so it is probably running for both hands, and maybe that is somehow screwing things up a bit. I tried that, from that I gauged that it was the actual ifSneaking() statment that wasn't working Share this post Link to post Share on other sites
Acrogenous 1 Report post Posted December 15, 2017 Nope, didn't work, I didn't think it had anything to do with the itemstack (otherwise the latter statement wouldn't work either) public boolean onBlockActivated(World world, BlockPos blockpos, IBlockState p_onBlockActivated_3_, EntityPlayer player, EnumHand hand, EnumFacing p_onBlockActivated_6_, float p_onBlockActivated_7_, float p_onBlockActivated_8_, float p_onBlockActivated_9_) { if(player.getHeldItem(hand) != ItemStack.EMPTY) { ItemStack stack = player.getHeldItem(hand); if (stack.hasTagCompound() && stack.getItem() == ModItems.logiCard && player.isSneaking() ) { NBTTagCompound card = stack.getTagCompound(); card.setInteger("xpos", blockpos.getX()); card.setInteger("ypos", blockpos.getY()); card.setInteger("zpos", blockpos.getZ()); } if (stack.hasTagCompound() && stack.getItem() == ModItems.logiCard && !player.isSneaking()) { direction(stack, player, world, blockpos); } } return super.onBlockActivated(world, blockpos, p_onBlockActivated_3_,player,hand, p_onBlockActivated_6_, p_onBlockActivated_7_, p_onBlockActivated_8_, p_onBlockActivated_9_); } Share this post Link to post Share on other sites
Choonster 1593 Report post Posted December 15, 2017 Sneak-right clicking on a Block with an Item will only call Item#onItemUse by default, ignoring Block#onBlockActivated. To change this behaviour, either override Item#doesSneakBypassUse to return true (for your own Items) or subscribe to PlayerInteractEvent.RightClickBlock and call PlayerInteractEvent.RightClickBlock#setUseBlock with Result.ALLOW (for Items from Vanilla or other mods). This is handled in PlayerInteractionManager#processRightClickBlock on the server and PlayerControllerMP#processRightClickBlock on the client. Don't compare to ItemStack.EMPTY directly, it's just one of many possible empty ItemStacks. Use ItemStack#isEmpty instead. 2 Share this post Link to post Share on other sites
jabelar 584 Report post Posted December 16, 2017 (edited) 6 hours ago, Acrogenous said: I tried that, from that I gauged that it was the actual ifSneaking() statment that wasn't working Sorry, you couldn't have done that fully or it would be obvious where the problem happens. As Choonster mentions, onBlockActivated() probably isn't even being called -- which would be obvious in the console statements (since they wouldn't print when you expect them to). Also, your if statements are still not that simple as there are conditions besides the sneaking that need to be satisfied. The thing with debugging is you have to double-check all your assumptions. Of course you expect everything is working because you wrote the code intending for it to work. So you literally have to confirm that every step of code executes exactly as you expect. You need to confirm that the method is called when you expect, that the parameter values are what you expect, that the if statement executes as you expect, and so forth. Edited December 16, 2017 by jabelar 1 Share this post Link to post Share on other sites
Acrogenous 1 Report post Posted December 17, 2017 thank you @Choonster it wasn't being activated, and fair point @jabelar it was not printing anything, that's what I meant to say, I probably didn't word it correctly thank you all that helped! Share this post Link to post Share on other sites