Solved NPE when I'm checking for null?

Here is the code, also tried replacing i with 1 but still same error. No clue why.

Code:Java

 
    @EventHandler
    public void onBottleDrink(PlayerItemConsumeEvent e) {
        if (e.getItem().getType() == Material.POTION) {
            for (int i = 1; i < e.getPlayer().getInventory().getSize(); i++) {
                if (e.getPlayer().getInventory().getItem(i).getType() != null) { // THIS LINE. NO IDEA WHY.
                    if (e.getPlayer().getInventory().getItem(i).getType() == Material.GLASS_BOTTLE) {
                        e.getPlayer().getInventory().setItem(i, new ItemStack(Material.AIR));
                        break;
                    }
                }
            }       
        }
    }
 

thanks in advance.

 

mhm, legit just realised that, thank you.

EDIT: Last thing, I'm sure this should work:

Code:Java

 
if (e.getPlayer().getInventory().getItem(i) == new ItemStack(Material.GLASS_BOTTLE))
 

this is never true (I'm making a plugin that removes glass bottles when you drink them, is it because it's too fast?

 

was also thinking, just didn't do it >.<
thanks again;
solved.

EDIT: found out it's too fast, used a bukkitrunnable to fix, but just as a note for anybody here ;P

 

@MemeplexOwner if you think about it it's kinda logic. If an event should be cancellable in any way the plugin should be notified about it before it really happens. Only that way it can be stopped before happening.