! = false
!player.getInventory().containsOneItem(x) = checking if the player doesn't have the items
|
So I'm making this overload NPC, but somehow it doesn't check if the items are in your inventory.
Does anyone see what I'm doing wrong?
This is my NPCHandler.
Code:else if (npc.getId() == 286) { if (!player.getInventory().containsOneItem(15308) || !player.getInventory().containsOneItem(15312) || !player.getInventory().containsOneItem(15316) || !player.getInventory().containsOneItem(269) || !player.getInventory().containsOneItem(15320) || !player.getInventory().containsOneItem(15324)) { player.getInventory().deleteItem(15308, 1); player.getInventory().deleteItem(15312, 1); player.getInventory().deleteItem(15316, 1); player.getInventory().deleteItem(15320, 1); player.getInventory().deleteItem(15324, 1); player.getInventory().deleteItem(269, 1); player.getPackets().sendGameMessage("You've exchanged your ingredients for an Overload."); player.getSkills().addXp(Skills.HERBLORE, 1000); player.getInventory().addItem(23531, 1); } else { player.getPackets().sendGameMessage("You either don't have the required level, or ingredients to make Overloads."); } }
! = false
!player.getInventory().containsOneItem(x) = checking if the player doesn't have the items
Now it will make an overload when you have 1 of the items, I'm trying to have it check for all items.
Code:if (player.getInventory().containsItem(15308) || player.getSkills().getLevelForXp(Skills.HERBLORE) < 96 || player.getInventory().containsItem(15312) || player.getInventory().containsItem(15316) || player.getInventory().containsItem(269) || player.getInventory().containsItem(15320) || player.getInventory().containsItem(15324)) { player.getInventory().deleteItem(15308, 1); player.getInventory().deleteItem(15312, 1); player.getInventory().deleteItem(15316, 1); player.getInventory().deleteItem(15320, 1); player.getInventory().deleteItem(15324, 1); player.getInventory().deleteItem(269, 1); player.getPackets().sendGameMessage("You've exchanged your ingredients for an Overload."); player.getSkills().addXp(Skills.HERBLORE, 1000); player.getInventory().addItem(23531, 1); } else { player.getPackets().sendGameMessage("You either don't have the required level, or ingredients to make Overloads."); } }
« Previous Thread | Next Thread » |
Thread Information |
Users Browsing this ThreadThere are currently 1 users browsing this thread. (0 members and 1 guests) |