Weird really weird

Ok so Im making a plugin so that when a player drops an arrow it cancels it out and launches the arrow and then removes one, ok but it removes the arrows just fine until the last one, then it stops but theres another problem it doesnt even make sense to me, first here is the code

Code:java

@SuppressWarnings("deprecation")
    @EventHandler
    public void dropEvent(PlayerDropItemEvent event){
        Player p = event.getPlayer();
 
 
        if(event.getItemDrop().getItemStack().getType() == Material.ARROW) {
            event.setCancelled(true);
            p.launchProjectile(Arrow.class);
            event.getPlayer().getInventory().removeItem(new ItemStack(Material.ARROW, 1));
            p.updateInventory();
            }
    }
}

Anyways ok so I but the arrow in my inventory slot 1 and I click Q and it launches the arrow and removes one, but when I take all the arrows completely out of my inventory when I select slot 1 and I click Q even without the arrow in my inventory it launches an arrow, how is even possible

 

Why do you have a suppress warning?

Also, for when checking if they have an arrow in their hand, try "event.getItemDrop().equals(Material.ARROW);"

Also, don't update their inventory. You don't need it for this situation.

Also, you made a Player variable and you don't even use it.

Player p;
event.getPlayer().getInv().removeItem(fdjsak);

 

You need to add in a statement that will check if the player actually has an arrow in the selected slot. Currently the script doesn't know weather there is or not.

 

XgXXSnipz at the time that the event is fired, the ItemStack is not in their inventory. Thus they won't have any arrows when they drop it, it won't be removed, and after the event the ItemStack will be added back due to the cancellation of the event.