BarAPI help

Hey guys I'm still not getting why this doesn't work, it's in a repeating scheduler for every 20L but it doesn't change.

Code:java

if(bardisplay == 1){
BarAPI.removeBar(Player onlineplayer);
BarAPI.setMessage(Player onlineplayer, "Message 1");
bardisplay --;
}else{
if(bardisplay == 0){
BarAPI.removeBar(Player onlineplayer);
BarAPI.setMessage(Player onlineplayer, "Message 2");
bardisplay ++;
}

(bardisplay is an int)
When I join the server it displays "Message 1" but stays like that.

Any help? Thanks

 

Code:java

 
if(bardisplay == 1) {
    BarAPI.removeBar(Player onlineplayer);
    BarAPI.setMessage(Player onlineplayer, "Message 1");
    bardisplay--;
}
if(bardisplay == 0) {
    BarAPI.removeBar(Player onlineplayer);
    BarAPI.setMessage(Player onlineplayer, "Message 2");
    bardisplay++;
}
 

Try that.

 

Wolfey
You didn't really modify any functionality in the code, and the proper way to write it would be

Code:java

if(bardisplay == 1) {
BarAPI.removeBar(Player onlineplayer);
BarAPI.setMessage(Player onlineplayer, "Message 1");
bardisplay--;
}
else if(bardisplay == 0) {
BarAPI.removeBar(Player onlineplayer);
BarAPI.setMessage(Player onlineplayer, "Message 2");
bardisplay++;
}

KevyPorter
Umm.. is that really your code? It should have a compile error.

 

I got confused at his code, why is he putting Player onlineplayer in removeBar()? Makes no sense whatsoever.

 

Wolfey

 

Wolfey Because otherwise it stays as the same bar

The_Doctor_123 No compile errors but else if made it work

Code:java

        Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
            public void run() {
                if(bardisplay == 5){
                    BarAPI.removeBar(player);
                    BarAPI.setMessage(player, "" + ChatColor.WHITE + ChatColor.BOLD + "WELCOME TO " + ChatColor.DARK_PURPLE + ChatColor.BOLD + "KEVYANDAZZA'S" + ChatColor.WHITE + ChatColor.BOLD + " SERVER!");
                    bardisplay --;
                }else if(bardisplay == 4){
                    BarAPI.removeBar(player);
                    BarAPI.setMessage(player, "" + ChatColor.WHITE + ChatColor.BOLD + "WELCOME TO " + ChatColor.GOLD + ChatColor.BOLD + "KEVYANDAZZA'S" + ChatColor.WHITE + ChatColor.BOLD + " SERVER!");
                    bardisplay --;
                }else if(bardisplay == 3){
                    BarAPI.removeBar(player);
                    BarAPI.setMessage(player, "" + ChatColor.WHITE + ChatColor.BOLD + "WELCOME TO " + ChatColor.RED + ChatColor.BOLD + "KEVYANDAZZA'S" + ChatColor.WHITE + ChatColor.BOLD + " SERVER!");
                    bardisplay --;
                }else if(bardisplay == 2){
                    BarAPI.removeBar(player);
                    BarAPI.setMessage(player, "" + ChatColor.WHITE + ChatColor.BOLD + "WELCOME TO " + ChatColor.AQUA + ChatColor.BOLD + "KEVYANDAZZA'S" + ChatColor.WHITE + ChatColor.BOLD + " SERVER!");   
                    bardisplay --;
                }else if(bardisplay == 1){
                    BarAPI.removeBar(player);
                    BarAPI.setMessage(player, "" + ChatColor.WHITE + ChatColor.BOLD + "WELCOME TO " + ChatColor.GREEN + ChatColor.BOLD + "KEVYANDAZZA'S" + ChatColor.WHITE + ChatColor.BOLD + " SERVER!");
                    bardisplay = 5;
                }
            }
        }, 0L, 20L);

 

Use a switch statement. It will make your code more organized

 

Blah1
That's all preference. Yes, they can be used there, but if statements can just as easily be used as well.