VIP Kicks Random Player - Able To Bypass Full Server

Hey guys, I am having trouble with my VIP kicking part of my plugin. I do not understand what is going on and the "You cannot join during game" part is working, but the VIP one is not. Everything is checked right, my name is in the VIP arraylist so I do not see why it is not working. Any help? Thanks!

Code:java

@EventHandler
    public void playerPreJoin(PlayerLoginEvent event) {
        Player player = event.getPlayer();
        int count = Bukkit.getOnlinePlayers().length;
        World world = Bukkit.getWorld("world");
        Location Spawn = new Location(world, -4, 50, -4);
        if ((CauldronWars.gameState.getState().equalsIgnoreCase(GameState.IN_GAME))
                || (CauldronWars.hasStarted)) {
            event.disallow(PlayerLoginEvent.Result.KICK_FULL,
                    "§4§lYou Cannot Join During Game!");
        }
        if (CauldronWars.hasStarted && player.isOp()) {
            event.allow();
            player.teleport(Spawn);
            player.setGameMode(GameMode.CREATIVE);
        }
        if (!CauldronWars.hasStarted) {
            if(player.isBanned()){
                event.disallow(PlayerLoginEvent.Result.KICK_BANNED, ChatColor.DARK_RED + "You Have Been Banned. Bye-Bye.");
                return;
            }
            if((count <= plugin.getServer().getMaxPlayers())){
                if(CauldronWars.vips.contains(player.getName())){
                Random r = new Random();
                event.setResult(PlayerLoginEvent.Result.ALLOWED);
                Player[] players = Bukkit.getServer().getOnlinePlayers();
                        Player rndPlayer = players[r.nextInt(0)];
                        rndPlayer.kickPlayer(ChatColor.DARK_RED + "A VIP has signed in!");
                    } else {
                        event.disallow(PlayerLoginEvent.Result.KICK_FULL, "§4§lTHE SERVER IS FULL!");
                    }
                }
        }
     }

 

Code:java

if((count <= plugin.getServer().getMaxPlayers())){

Should be

Code:java

if((count > plugin.getServer().getMaxPlayers())){

 

Still doesn't work :/

 

You do realize that random.nextInt(0) will always return 0, making the random useless

 

That will throw a Exception: "java.lang.IllegalArgumentException: n must be positive".

If you want have random numbers numbers from for example 0-3, then you need add 4 to the method.

 

That's even worse if that's the case.

 

Also, if you want to only kick someone who isn't a VIP just getting a random player could kick another VIP.