Looping

Hi I have a question about looping. I know there is a for loop but for what I am doing, this will not work. I am making a minigame plugin where the players are stored in String Lists in a config file and the config goes like so:
There is a section called arenas which stores all the arenas as a string list. After, whenever you do /pvc create arenaname, it will create a section called arenaname and inside that, it will create some more sections. The only relevant ones are arenaname.police and arenaname.criminals. Now, when someone does /pvc leave, I want to loop through all the arenas and if they are in any of the arenaname.police or arenaname.criminals, I want to remove them and I have this but I am using a for loop which says the following:
for every arena, if the player is in in the arenaname.police or arenaname.criminals, remove them and send them a message saying they left the game or else send them a message saying they aren't in a game. This is faulty because as I said, it is for every arena so therefore if I create 10 arenas and they are in only 1, It will send the player 9 messages saying they aren't in an arena and 1 saying they left the arena. Is there a way to tell them they left and NOT tell them they are not in 9 arenas and yet still keep the possibility that if they aren't in an arena then tell them so?

Btw here is the code:

Code:java

if (args[0].equalsIgnoreCase("leave")) {
                        for (String s : getConfig().getStringList("arenas")) {
                            if (getConfig().getStringList(s + ".police").contains(player.getName())) {
                                List<String> police = getConfig().getStringList(s + ".police");
                                police.remove(player.getName());
                                getConfig().set(s + ".police", police);
                                saveConfig();
                                for (String p : getConfig().getStringList(s + ".police")) {
                                    Player pol = getServer().getPlayer(p);
                                    pol.sendMessage(ChatColor.YELLOW + player.getName() + " has left the game!");
                                }
                                for (String c : getConfig().getStringList(s + ".criminals")) {
                                    Player crim = getServer().getPlayer(c);
                                    crim.sendMessage(ChatColor.YELLOW + player.getName() + " has left the game!");
                                }
                                player.sendMessage(ChatColor.YELLOW + "You have left the game!");
                                break;
                            }
                            if (getConfig().getStringList(s + ".criminals").contains(player.getName())) {
                                List<String> criminals = getConfig().getStringList(s + ".criminals");
                                criminals.remove(player.getName());
                                getConfig().set(s + ".criminals", criminals);
                                saveConfig();
                                for (String p : getConfig().getStringList(s + ".police")) {
                                    Player pol = getServer().getPlayer(p);
                                    pol.sendMessage(ChatColor.YELLOW + player.getName() + " has left the game!");
                                }
                                for (String c : getConfig().getStringList(s + ".criminals")) {
                                    Player crim = getServer().getPlayer(c);
                                    crim.sendMessage(ChatColor.YELLOW + player.getName() + " has left the game!");
                                }
                                player.sendMessage(ChatColor.YELLOW + "You have left the game!");
                                break;
                            } else {
                                player.sendMessage(ChatColor.RED + "You are not in a game!");
                            }
                        }
                    }

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

 

BillyBobJoe168
This would probably be so much easier if you just had an Arena class with a HashSet to store player names...

 

Alright how would I do this? I've never used HashSets before...I want to be able to have multiple arenas so if you need to make 2 hashsets per arena, I would like to be able to name them like arenaname + "police"

Oh and if you do have a way to solve this I would appreciate it because I have a whole bunch of stuff coded and I don't really want to change my whole code. It's pretty big now

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

 

You don't understand. You wouldn't be using the config in the way you are now.

A Set is sort of like a List. It's a Collection, used to store Objects, except that duplicate Objects aren't allowed. You don't want to store the same Player name twice.

Basically what you will do is create a class called Arena. Here, you will have all the attributes of an Arena- The arena name, the players, the spawn point, etc. as well as the methods- getName, addPlayer, removePlayer, etc. These are kind of basic object oriented programming basics, so I'm not really sure how much detail I need (or am willing) to go into.