Solved Toggle Particle Visibilty Command

So I have a simple blood plugin, which on every hit, shows the player bleeding with redstone block particles. I want it so if players do the command "/realblood toggle" it will disable the visibility to only the player, and if the player runs the command again, it will enable.

Here is my code so far..

Code:java

package me.Jaaakee224.RealBlood;
 
import java.util.HashMap;
import java.util.Map;
 
import org.bukkit.Effect;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
        public class RealBlood extends JavaPlugin implements Listener {
                public static Map<Player,Boolean> states = new HashMap<Player, Boolean>();
                @Override
                public void onDisable() {
 
                }
 
                @Override
                public void onEnable() {
                        getServer().getPluginManager().registerEvents(this, this);
                }
 
                public boolean onCommand(CommandSender sender, Command command, String commandLabel, String[] args) {
                    if(command.getName().equalsIgnoreCase("realblood")) {
                          if(args.length == 0) { return false; }
                          if(args[0].equalsIgnoreCase("toggle")) {
 
 
                          }
 
                }
                    return true;
                }
 
                @SuppressWarnings("deprecation")
                @EventHandler
                public void EntityBlood(EntityDamageEvent e){
                        for(Entity entity : e.getEntity().getNearbyEntities(32,32,32)) {
                                if(entity instanceof Player) {
                                        Player target = (Player) entity;
                                        if (states.containsKey(target)) {
                                                if(!states.get(target)) { continue; }
                                        }
                                        target.playEffect(e.getEntity().getLocation().add(0, 1, 0), Effect.STEP_SOUND, 152);
                                }
                        }
                }
        }

Please help me with this.

 

Haha you probably shouldn't be suppressing deprecation warnings, but I guess it doesn't really matter that much.

Also, you should probably just store player names not the player object and retrieve the player object whenever your need it using then name. There can be problems with storing the actual player object.

Now to your code, it looks fine so far. Lists (Map) referencing players are really annoying because you need to have code making sure to add players every time they join, every time the onEnable runs, and you need to check that the player is actually inside the list whenever it is referenced. Alternatively you could try just having a list of player names that want blood disabled so you wouldn't have to make sure every player is inside it.

 

BorisTheTerrible Can you show me an example of this? I haven't worked with HashMap in a long time, and I need a little refresh.

 

bump

Jaaakee224
Pseudo logic:

Command-
If list contains sender name
remove from list
else
add to list

Event-
Check if the player(s) invokved in hte event are in the list
run code

xTrollxDudex Thanks

EDIT by Moderator: merged posts, please use the edit button instead of double posting.