Posted November 3, 20186 yr Hi, I'm trying to add an item that works like bone meal. For now I tried to add this function but it not work: @Override public boolean canItemEditBlocks() { return true; } public boolean onItemUse(ItemStack str_006, World str_002, EntityPlayer str_003, BlockPos str_005, EnumHand str_007) { return ItemDye.applyBonemeal(str_006, str_002, str_005, str_003, str_007); } EDIT: Thred solved, correct code: @Override public EnumActionResult onItemUse(EntityPlayer str_001, World str_002, BlockPos str_003, EnumHand str_004, EnumFacing str_005, float f_001, float f_002, float f_003) { ItemStack stack = new ItemStack(this); if (!str_001.canPlayerEdit(str_003.offset(str_005), str_005, stack)) { return EnumActionResult.FAIL; } else { if (ItemDye.applyBonemeal(stack, str_002, str_003, str_001, str_004)) { if (!str_002.isRemote) { str_002.playEvent(2005, str_003, 0); } } return EnumActionResult.SUCCESS; } } Edited November 4, 20186 yr by nov4e
November 3, 20186 yr Author 4 minutes ago, diesieben07 said: Your onItemUse method does nothing, since it's not called from anywhere (unless you haven't shown that code). If you intended to override Item#onItemUse then you need to actually match the method signature. This is why you should always use @Override (and if you get an error, the fix is never to remove @Override). Eclipse says: The method onItemUse(ItemStack, World, EntityPlayer, BlockPos, EnumHand) of type IAnimalExcrements must override or implement a supertype method. 4 minutes ago, diesieben07 said: Also, why are you naming your parameters like that? And your answer of "I rename all my strings like that." is not an explanation. This is not an acceptable naming scheme for variables. It makes some difference? no.
November 3, 20186 yr Author 4 minutes ago, diesieben07 said: Stop... what? Telling you to do your homework instead of just doing it for you? No. I am not going to do that. Okok sorry..... I've maked a method in Basic Item: Spoiler BasicItem public class BasicItem extends Item implements IHasModel { public BasicItem(String str_001) { setRegistryName(str_001); setUnlocalizedName(str_001); ItemList.modItems.add(this); } @Override public boolean canDestroyBlockInCreative(World str_002, BlockPos str_003, ItemStack str_004, EntityPlayer str_005) { return true; } @Override public void registerModels() { Dreams.proxy.registerItemRenderer(this, 0, "inventory"); } public boolean onItemUse(ItemStack str_006, World str_002, EntityPlayer str_003, BlockPos str_005, EnumHand str_007) { return true; } } ItemBoneMeal: public IBoneMeal(String str_001) { super(str_001); setCreativeTab(CreativeTabs.MATERIALS); } @Override public boolean canItemEditBlocks() { return true; } @Override public boolean onItemUse(ItemStack str_006, World str_002, EntityPlayer str_003, BlockPos str_005, EnumHand str_007) { return ItemDye.applyBonemeal(str_006, str_002, str_005, str_003, str_007); } It not works. I'm asking onItemUse or onItemRightClick is correct?
November 3, 20186 yr Author 2 hours ago, diesieben07 said: Item#onItemUse. I Have to override it in BasicItem? Override Item#onItemUse requires EnumActionResult but applyBonemeal needs a boolean method.
November 3, 20186 yr Author 6 minutes ago, diesieben07 said: I don't know what BasicItem is, but it sounds like something you should not have. Do not abuse inheritance to re-use code. applyBonemeal indeed returns a boolean, it returns true if it successfully grew something. onItemUse returns an EnumActionResult, which is one of SUCCESS, FAIL or PASS. SUCCESS and FAIL should be pretty clear. PASS means "I did not do anything", which in your case is not applicable, unless your item only sometimes performs it's action. Translating the "yes/no" success from applyBonemeal into EnumActionResult.SUCCESS resp. EnumActionResult.FAIL is quite trivial. Spoiler public EnumActionResult onItemUse(ItemStack stack, EntityPlayer playerIn, World worldIn, BlockPos pos, EnumHand hand, EnumFacing facing, float hitX, float hitY, float hitZ) { if (ItemDye.applyBonemeal(stack, worldIn, pos, playerIn)) { if (!worldIn.isRemote) { worldIn.playEvent(2005, pos, 0); } } return EnumActionResult.SUCCESS; } } Correct? Edited November 3, 20186 yr by nov4e
November 4, 20186 yr Author 17 hours ago, diesieben07 said: Well, now you just always return SUCCESS, even if the bonemeal could not be applied. In fact, you just disregarded what I said entirely and just copied the code from ItemDye. Why do you ask questions in this forum if you then do something else entirely instead of doing what we tell you? I SOLVED: @Override public EnumActionResult onItemUse(EntityPlayer str_001, World str_002, BlockPos str_003, EnumHand str_004, EnumFacing str_005, float f_001, float f_002, float f_003) { ItemStack stack = new ItemStack(this); if (!str_001.canPlayerEdit(str_003.offset(str_005), str_005, stack)) { return EnumActionResult.FAIL; } else { if (ItemDye.applyBonemeal(stack, str_002, str_003, str_001, str_004)) { if (!str_002.isRemote) { str_002.playEvent(2005, str_003, 0); } } return EnumActionResult.SUCCESS; } }
November 4, 20186 yr 57 minutes ago, nov4e said: I SOLVED: @Override public EnumActionResult onItemUse(EntityPlayer str_001, World str_002, BlockPos str_003, EnumHand str_004, EnumFacing str_005, float f_001, float f_002, float f_003) { ItemStack stack = new ItemStack(this); if (!str_001.canPlayerEdit(str_003.offset(str_005), str_005, stack)) { return EnumActionResult.FAIL; } else { if (ItemDye.applyBonemeal(stack, str_002, str_003, str_001, str_004)) { if (!str_002.isRemote) { str_002.playEvent(2005, str_003, 0); } } return EnumActionResult.SUCCESS; } } Yeah, that should work, but those parameter names, really?
November 4, 20186 yr Author 10 minutes ago, desht said: Yeah, that should work, but those parameter names, really? lmao if (ItemDye.applyBonemeal(str_006, str_002, str_003, str_001, str_004)) { if (!str_002.isRemote) { str_002.playEvent(2005, str_003, 0); } str_006.shrink(1); } str_006.shrink(1); this maybe work?
November 4, 20186 yr 2 hours ago, nov4e said: lmao Seriously you should change them to something that has meaning, what you are doing just makes me think your dealing with magic numbers. VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect. Forge and vanilla BlockState generator.
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.