Solved send message to all players

i'm trying to send a message to every player (reward system).
but when it sends it send everyone rewards to each player.
so like i got everyone else reward.

Code:

                    for(Player player : Bukkit.getOnlinePlayers()){
                        int cash = points.get(player.getName());
                        int give = cash/10;
                        goldpieces.addGoldPieces(player, give);
                        Bukkit.broadcastMessage("§f▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄");
                        Bukkit.broadcastMessage("                   §eReward Summary");
                        Bukkit.broadcastMessage("    ");
                        Bukkit.broadcastMessage("§eKills: §7(§6" + killCount.get(player.getName()) + "§7)");
                        Bukkit.broadcastMessage("§eGoldpieces:§7(§6" + killCount.get(player.getName()) + "§7)");
                        Bukkit.broadcastMessage("     ");
                        Bukkit.broadcastMessage("    ");
                        Bukkit.broadcastMessage("§f▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄");
                    }

 

You want to everyone recive message when some one get gold ?

 

no at end of game it send message to all players on how many kills they got and gold pieces

 

it is not doing it, i send every player a reward message. but i get everyone reward message (so theirs more than one in my chat.)

 

DERP DELETE PLS

 

@Jumb_1907 wait what

 

Well you're looping through it for every online player. And then you are printing it for every online player.
Instead of broadcasting for every online player, get a for loop for all the online players, and then just print the values you get once for each player. You currently loop for every player.

 

@Christian210x
Your problem is:
You do loop trough all players, but you aren't sending the message only to the selected player, you are sending it to all. (Broadcast always send it to all players who are online)

So, do stmh like that:

Code:

foreach player (maybe not everyone, just the ones who have played?):
    giveReward(player);
    player.sendMessage("You got XXX goldcoins,...");

Edit: i would recommend not to use playernames in your method. Instead, juse the UUID. It's much better and not the big difference in your code.

 

well i want to send it to all players that played even dead ones
And you guys want me to do the same thing aagain read what you said.
@thechrisanator & @Erumeldor

my bad everyone i see what i did i.
i didnt see the bukkit.broadcastmessage...
it all works now.