Removing ALL Potion Effects

If a player has multiple potion effects, how would I remove them all, rather than one at a time?

Using player.removePotionEffect(), and player.hasPotionEffect().

-Waffle

 

use case and end case.

case{ player haseffect X;
remove effect
}
case{etc
}

 

I see that. Ok.

Now how would I add a command to that? Just FYI, it's a plugin that allows you to add potion effects through commands, and I was just trying to add a new command to remove all the potion effects on that player when he does a command.

 

Code:

if (cmd.getName().equalsIgnoreCase("remove"))
    for (PotionEffect effect : player.getActivePotionEffects())
        player.removePotionEffect(effect.getType());

 

if you cant find a solution you can always remove them 1 at a time with

Code:

targetPlayer.addPotionEffect(new PotionEffect(PotionEffectType.DAMAGE_RESISTANCE, 0, 0),true);

 

thank you sir for saving me a trip to the docs

 

Or.....

Code:

player.removePotionEffect(PotionEffectType.(Name));

done

 

Could you explain this?

 

If someone types /remove
Loop through all of their current potion effects
Remove that potion effect type from them.

 

One more question what does the : do in this?

 

Atakyn
That's how you make/use a for loop... It isn't just Bukkit, that's general Java stuff.

 

When you see the colon ( : ), read it as “in.” Thus, the loop above reads as “for each PotionEffect in the players active potion effects” Note that there is no performance penalty for using the for-each loop, even for arrays. In fact, it may offer a slight performance advantage over an ordinary for loop in some circumstances, as it computes the limit of the array index only once. While you can do this by hand, programmers don’t always do so.

 

Whats wrong with...

Code:

player.getActivePotionEffects().clear()

I see this thread is a few months old, maybe that has been added since.

 

It won't work. getActivePotionEffects() returns a Collection which is not referenced by anything else. So modifying that collection (using clear() - or anything else) wont make a difference.

That's evident from the fact that this code:

Code:

for (PotionEffect effect : player.getActivePotionEffects())
        player.removePotionEffect(effect.getType());

does work and does not throw a concurrent modification exception.