Solved Loop through again if the name is null

Okay, so I'm making a lottery plugin and I need to check if one of the winners it picks isn't online I need it to repick, but I'm not sure how I would do this.. Any help?

Here's my code:

Code:java

                    }else if(seconds == 0){
 
 
                            int moneypot = getConfig().getInt("DCLottery.MoneyPot");
                            double winnersmoney = moneypot * .5;
 
 
                            List<String> list = Main.ticketbuyers.getStringList("Buyers");
                            Random r = new Random();
 
                            // Getting the random player name from our list
                            String p1 = list.get(r.nextInt(list.size()));
                            String p2 = list.get(r.nextInt(list.size()));
                            String p3 = list.get(r.nextInt(list.size()));
                            String p4 = list.get(r.nextInt(list.size()));
                            String p5 = list.get(r.nextInt(list.size()));
 
 
                            Player winner1 = Bukkit.getPlayer(p1);
                            Player winner2 = Bukkit.getPlayer(p2);
                            Player winner3 = Bukkit.getPlayer(p3);
                            Player winner4 = Bukkit.getPlayer(p4);
                            Player winner5 = Bukkit.getPlayer(p5);
 
                            EconomyResponse r1 = econ.depositPlayer(winner1.getName(), winnersmoney);
                            if(r1.transactionSuccess()){
                                winner1.sendMessage(prefix + ChatColor.BLUE + "$" + ChatColor.RED + winnersmoney + ChatColor.BLUE + " has been deposited in to your account for winning the lottery!");
                            }
 
                            EconomyResponse r2 = econ.depositPlayer(winner2.getName(), winnersmoney);
                            if(r2.transactionSuccess()){
                                winner2.sendMessage(prefix + ChatColor.BLUE + "$" + ChatColor.RED + winnersmoney + ChatColor.BLUE + " has been deposited in to your account for winning the lottery!");
                            }
 
                            EconomyResponse r3 = econ.depositPlayer(winner3.getName(), winnersmoney);
                            if(r3.transactionSuccess()){
                                winner3.sendMessage(prefix + ChatColor.BLUE + "$" + ChatColor.RED + winnersmoney + ChatColor.BLUE + " has been deposited in to your account for winning the lottery!");
                            }
 
                            EconomyResponse r4 = econ.depositPlayer(winner4.getName(), winnersmoney);
                            if(r4.transactionSuccess()){
                                winner4.sendMessage(prefix + ChatColor.BLUE + "$" + ChatColor.RED + winnersmoney + ChatColor.BLUE + " has been deposited in to your account for winning the lottery!");
                            }
 
                            EconomyResponse r5 = econ.depositPlayer(winner5.getName(), winnersmoney);
                            if(r5.transactionSuccess()){
                                winner5.sendMessage(prefix + ChatColor.BLUE + "$" + ChatColor.RED + winnersmoney + ChatColor.BLUE + " has been deposited in to your account for winning the lottery!");
                            }
 
                        Bukkit.broadcastMessage(prefix + ChatColor.BLUE + "Congratulations, " + ChatColor.RED + p1.toString() + ChatColor.BLUE + ", " + ChatColor.RED + p2.toString() + ChatColor.BLUE + ", " + ChatColor.RED + p3.toString() + ChatColor.BLUE + ", " + ChatColor.RED + p4.toString() + ChatColor.BLUE + ", and " + ChatColor.RED + p5.toString()
                                + ChatColor.BLUE + "! You have won the lottery containing $" + ChatColor.RED + moneypot + ChatColor.BLUE + "!");
                        seconds = 241;
                        seconds--;
                    }

Thanks

 

PimpDuck
I guess maybe a while loop will do.

 

PimpDuck
You would use a while loop and have it run as long as the player selected is null.

 

Gater12 @IvanTheFourth I'm not that experienced with while loops. Would it look like

Code:java

while(winner1 == null){
String p1 = list.get(r.nextInt(list.size()));
}

or something? xD If I'm wrong, could I get an example with my code?
Thanks, again!

IvanTheForth *

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

 

PimpDuck
You would want to use the same variable. So, something like:

Code:

Player winner = list.get(r.nextInt(list.size()));
while(winner == null){
  winner = Bukkit.getPlayer(list.get(r.nextInt(list.size())));
}

 

IvanTheForth It wants me to convert to string.. http://prntscr.com/3i9imt

IvanTheForth and when I do that it does this.. http://prntscr.com/3i9kld lol

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

 

PimpDuck
Oh sorry, my bad. Do
Player winner = Bukkit.getPlayer(list.get(r.nextInt(list.size())));

 

That just adds a duplicate error on winner..

IvanTheForth

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

 

PimpDuck
Change type winner to Player. Assign winner Bukkit.getPlayer(String a) and do while loop.

 

Got it like this

Code:java

                    }else if(seconds == 0){
 
 
                            int moneypot = getConfig().getInt("DCLottery.MoneyPot");
                            double winnersmoney = moneypot * .5;
 
 
                            List<String> list = Main.ticketbuyers.getStringList("Buyers");
                            Random r = new Random();
 
                            String player1 = list.get(r.nextInt(list.size()));
                            while(player1 == null){
                              Player winner1 = Bukkit.getPlayer(player1);
 
                              if(winner1 != null){
                                EconomyResponse r1 = econ.depositPlayer(winner1.getName(), winnersmoney);
                                if(r1.transactionSuccess()){
                                    winner1.sendMessage(prefix + ChatColor.BLUE + "$" + ChatColor.RED + (int) winnersmoney + ChatColor.BLUE + " has been deposited in to your account for winning the lottery!");
                                }
                              }
 
                            }
 
                            String player2 = list.get(r.nextInt(list.size()));
                            while(player2 == null){
                              Player winner2 = Bukkit.getPlayer(player2);
 
                              if(winner2 != null){
                                EconomyResponse r2 = econ.depositPlayer(winner2.getName(), winnersmoney);
                                if(r2.transactionSuccess()){
                                    winner2.sendMessage(prefix + ChatColor.BLUE + "$" + ChatColor.RED + (int) winnersmoney + ChatColor.BLUE + " has been deposited in to your account for winning the lottery!");
                                }
                              }
 
                            }
 
                            String player3 = list.get(r.nextInt(list.size()));
                            while(player3 == null){
                              Player winner3 = Bukkit.getPlayer(player3);
 
                              if(winner3 != null){
                                EconomyResponse r2 = econ.depositPlayer(winner3.getName(), winnersmoney);
                                if(r2.transactionSuccess()){
                                    winner3.sendMessage(prefix + ChatColor.BLUE + "$" + ChatColor.RED + (int) winnersmoney + ChatColor.BLUE + " has been deposited in to your account for winning the lottery!");
                                }
                              }
 
                            }
 
                            String player4 = list.get(r.nextInt(list.size()));
                            while(player4 == null){
                              Player winner4 = Bukkit.getPlayer(player4);
 
                              if(winner4 != null){
                                EconomyResponse r2 = econ.depositPlayer(winner4.getName(), winnersmoney);
                                if(r2.transactionSuccess()){
                                    winner4.sendMessage(prefix + ChatColor.BLUE + "$" + ChatColor.RED + (int) winnersmoney + ChatColor.BLUE + " has been deposited in to your account for winning the lottery!");
                                }
                              }
 
                            }
 
                            String player5 = list.get(r.nextInt(list.size()));
                            while(player5 == null){
                              Player winner5 = Bukkit.getPlayer(player5);
 
                              if(winner5 != null){
                                EconomyResponse r2 = econ.depositPlayer(winner5.getName(), winnersmoney);
                                if(r2.transactionSuccess()){
                                    winner5.sendMessage(prefix + ChatColor.BLUE + "$" + ChatColor.RED + (int) winnersmoney + ChatColor.BLUE + " has been deposited in to your account for winning the lottery!");
                                }
                              }
 
                            }
 
 
 
                        Bukkit.broadcastMessage(prefix + ChatColor.BLUE + "Congratulations, " + ChatColor.RED + player1.toString() + ChatColor.BLUE + ", " + ChatColor.RED + player2.toString() + ChatColor.BLUE + ", " + ChatColor.RED + player3.toString() + ChatColor.BLUE + ", " + ChatColor.RED + player4.toString() + ChatColor.BLUE + ", and " + ChatColor.RED + player5.toString()
                                + ChatColor.BLUE + "! You have won the lottery containing $" + ChatColor.RED + moneypot + ChatColor.BLUE + "!");
                        getConfig().set("DCLottery.MoneyPot", 0);
                        seconds = 241;
                        seconds--;
                    }
 
                    seconds--;
                  }
                  }
                },0,20);
    }

Solved.

 

PimpDuck
This should work:

Code:

        Player winner = Bukkit.getPlayer(list.get(new Random().nextInt(list.size())));
        while(winner == null){
            winner = Bukkit.getPlayer(list.get(new Random().nextInt(list.size())));
        }

I forgot that Bukkit switched over to UUID's.

Oh and your list should not be Strings, but of UUID's

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