Sending messages to only certain players

Okay, so on my lotter plugin I'm making a lottostfu command so if someone doesn't wanna see the lottery it doesn't show up for them. I have the arraylist for adding and removing the player set-up but how would I make it only broadcast to the players not on the list???

Main class: http://paste.md-5.net/kiwaqitapu.avrasm

CommandClass: http://paste.md-5.net/qoxadapunu.avrasm

 

PimpDuck
Here's a rough example of what you could do:

Code:java

    private ArrayList<Player> players = new ArrayList<Player>();
 
    public void sendMessage(String message){
        for (Player p : Bukkit.getOnlinePlayers()){
            if (!players.contains(p))
                p.sendMessage(message);
        }
    }

 

EDIT: Well The Gaming Grunts beat me to it.

Make a for-while loop, this will iterate through all people on the server if you loop through Bukkit.getServer().getOnlinePlayes(); Then check if the shhlotto contains player.getName();

Example:

Code:java

List<String> muted = Arrays.asList("JasonDL13", "JasonDL14");
//That's how you make an arraylist in one line of code
 
for(Player p : Bukkit.getOnlinePlayers()) {
//This will loop through everyone online, p being the player
 
if(muted.contains(p.getName())) {
//Muted contains the player that we are looping through
p.sendMessage("You're Muted!");
} else {
//Muted doesn't contain the player that we are looping through
p.sendMessage("You're Not Muted!");
}
 
}

So use something like that and only send people the message if they're not in your shh list.

Also, you're variable names aren't really professional. Instead of "shhlist" I might use "mutedLottoMessages"

 

The Gaming Grunts
I get this error :/

Code:java

[21:48:21 WARN]: [DCLottery] Task #104 for DCLottery v1.0 generated an exception
 
java.lang.NullPointerException
at me.PimpDuck.DCLottery.Main$1.run(Main.java:96) ~[?:?]
at org.bukkit.craftbukkit.v1_7_R3.scheduler.CraftTask.run(CraftTask.java
:53) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
at org.bukkit.craftbukkit.v1_7_R3.scheduler.CraftScheduler.mainThreadHea
rtbeat(CraftScheduler.java:345) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d89
43-b3078jnks]
at net.minecraft.server.v1_7_R3.MinecraftServer.v(MinecraftServer.java:6
00) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
at net.minecraft.server.v1_7_R3.DedicatedServer.v(DedicatedServer.java:2
60) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
at net.minecraft.server.v1_7_R3.MinecraftServer.u(MinecraftServer.java:5
58) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
at net.minecraft.server.v1_7_R3.MinecraftServer.run(MinecraftServer.java
:469) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
at net.minecraft.server.v1_7_R3.ThreadServerApplication.run(SourceFile:6
28) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
>

Main: http://paste.md-5.net/komebagoce.avrasm CommandClass: http://paste.md-5.net/huworegofu.avrasm

 

PimpDuck
There are no players when the server first starts.

 

I'm logged in?

 

PimpDuck
As Gater12 said. You're calling your method on startup, at which time there are no players online, resulting in your NPE.

 

How would I fix this? .-.

The Gaming Grunts

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

 

PimpDuck
By checking if there are players online then loop through them.

 

Gater12

Code:java

public void Lottery()
    {
        taskId = Bukkit.getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
                  @SuppressWarnings({ "deprecation", "unused" })
                public void run() {
                    if(seconds == 14400) {
                        while(Bukkit.getOnlinePlayers() == null){
 
                        }
                        for(Player p : Bukkit.getOnlinePlayers()){
                            if(!CommandClass.shhlist.contains(p.getName())){
                                int moneypot = getConfig().getInt("DCLottery.MoneyPot");
                        p.sendMessage(prefix + ChatColor.BLUE + "The lottery will be drawn in " + ChatColor.RED + "4 hours!");
                        p.sendMessage(prefix + ChatColor.BLUE + "Buy your lottery ticket with " + ChatColor.RED + "'/lottobuy [# of tickets]'");
                        p.sendMessage(prefix + ChatColor.BLUE + "Get information on the lottery by doing " + ChatColor.RED + "'/lottoinfo'");
                        alreadypayed = false;
                        lastCachedTime = 14400;
                        if((int) timers.getInt("LastCachedTime") != lastCachedTime){
                            timers.set("LastCachedTime", lastCachedTime, false);
                        }
                        timers.saveFile();
                    seconds--;
                            }
                        }

Could I use something like the while loop?

Code:java

    public void Lottery()
    {
        taskId = Bukkit.getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
                  @SuppressWarnings({ "deprecation", "unused" })
                public void run() {
                    if(seconds == 14400) {
                        while(Bukkit.getOnlinePlayers() == null){
                            seconds = 14400;
                        }
                        for(Player p : Bukkit.getOnlinePlayers()){
                            if(!CommandClass.shhlist.contains(p.getName())){
                                int moneypot = getConfig().getInt("DCLottery.MoneyPot");
                        p.sendMessage(prefix + ChatColor.BLUE + "The lottery will be drawn in " + ChatColor.RED + "4 hours!");
                        p.sendMessage(prefix + ChatColor.BLUE + "Buy your lottery ticket with " + ChatColor.RED + "'/lottobuy [# of tickets]'");
                        p.sendMessage(prefix + ChatColor.BLUE + "Get information on the lottery by doing " + ChatColor.RED + "'/lottoinfo'");
                        alreadypayed = false;
                        lastCachedTime = 14400;
                        if((int) timers.getInt("LastCachedTime") != lastCachedTime){
                            timers.set("LastCachedTime", lastCachedTime, false);
                        }
                        timers.saveFile();
                    seconds--;
                            }
                          }

I think I fixed it but I still get the error on "if(!CommandClass.shhlist.contains(p.getName())){
"

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