Solved Help With Cycling Through List

Here's my code:

Bukkit.getScheduler().scheduleSyncRepeatingTask(this, new Runnable(){

@Override
public void run() {
for (int tip = 1; tip > 5; tip++){
if (tip == 5){
tip = 1;
}
if (tip == 1){
BarAPI.setMessage(ChatColor.GREEN + prefix + " " + ChatColor.GOLD + tip1);
Bukkit.getServer().broadcastMessage(ChatColor.GREEN + prefix + " " + ChatColor.GOLD + tip1);
}
if (tip == 2){
BarAPI.setMessage(ChatColor.GREEN + prefix + " " + ChatColor.GOLD + tip2);
Bukkit.getServer().broadcastMessage(ChatColor.GREEN + prefix + " " + ChatColor.GOLD + tip2);
}
if (tip == 3){
BarAPI.setMessage(ChatColor.GREEN + prefix + " " + ChatColor.GOLD + tip3);
Bukkit.getServer().broadcastMessage(ChatColor.GREEN + prefix + " " + ChatColor.GOLD + tip3);
}
if (tip == 4){
BarAPI.setMessage(ChatColor.GREEN + prefix + " " + ChatColor.GOLD + tip4);
Bukkit.getServer().broadcastMessage(ChatColor.GREEN + prefix + " " + ChatColor.GOLD + tip4);
}
if (tip == 5){
BarAPI.setMessage(ChatColor.GREEN + prefix + " " + ChatColor.GOLD + tip5);
Bukkit.getServer().broadcastMessage(ChatColor.GREEN + prefix + " " + ChatColor.GOLD + tip5);
}


}
}

}, 20, 20 * delay);

I have everything registered, the .yml's are good, etc. The problem is this piece of code.

I want the system to display tips in order from 1 - 5 and when it gets to five, it restarts back to one.

Thanks

 

Post it in code tags please ReadySetPawn

 

that's because you have this loop in a repeating task

 

k9rosie

How do I make it cycle through in a repeating task then?

 

how often is your task supposed to repeat?

 

k9rosie

Well it's configurable in seconds but I usually keep it at 300 seconds (5 minutes).

 

nevermind my last comment. is the loop restarting back at one the problem you're having? or is that what you want it to do

 

k9rosie

When I log onto the server, it doesn't display any text. And I know the methods that display the text work because they worked when I used the random method.

I actually found part of the problem. I accidentally had "<" rather than ">". But now it just spams all the messages every 5 minutes =/

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

 

well that's what you programmed it to do. what are you trying to do?

 

To do tips, you wouldn't use a for loop. If you use a for loop, it would just consistently spam it, as there is no delay, and since it is inside a repeating task, it would just get very awkward. What you would want to do is outside of the run method, create the tip variable, and then check which number it is inside the run statement. Then you would want to do whatever, and if it goes to 5, set it back to 1. Then just make the delay however long you want in ticks.

 

Alright I fixed it now thanks to the help of you two

Thanks <3