Could this be more efficient?

Hello Bukkit, today I wanted to make a method that would split people into 3 teams. For example, I wanted to have 1 person be on the 2 teams (Murderer, Special) , then the rest be on the other team (Bystander). Basically the same teams that are in Garry's mod "Murder". Is this code good enough because I'm worried that it might fail or something. Here is my code:

Code:java

    boolean allowed = false;
 
        while(!allowed){
            System.out.println("Test");
            String s = getPlayersInMap().get(new Random().nextInt(getPlayersInMap().size()));
            if(getPlayersInMap().size() <= 1){
                stop(); // Have to stop the game or the loop will go on forever, and will eventually crash the server.
                return;
            }
 
            if(s == this.murderer){
                continue;
            }else{
                allowed = true;
            }
 
            if(allowed){
                this.special = s;
                //Player sp = Bukkit.getPlayer(s);
                Role.getInstance().setRole(s, PlayerRole.Special);
                System.out.println("Special: " + this.special);
                System.out.println("Reached 1");
 
                }

 

Test it and you know.

 

It works fine, but I was just wondering if I could do it better.

 

thepaperboy99 I mean, you could remove the allowed boolean and put all the code in the check from line 18-23 into the else clause.
Edit: And make it a while(true) loop. It seems dangerous, but it really isn't much worse than what you're doing now.