How can i get a false if the array list is empty?

Hey Guys what i want to make is a little support command that informs a group of player with a specific permission if someone type in /support but if no one with the permission is online the player shall get a messager that no Supporter is online and the he should open a support ticket.

Code:java

@Override
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
Player p = null;
if(sender instanceof Player){
p = (Player) sender;
}
 
if(p!= null) {
 
if(args.length == 0) {
 
p.sendMessage("A Supporter has ben informed about your request!");
 
for(Player players : Bukkit.getServer().getOnlinePlayers()) {
if(players.hasPermission("supportix.supporter")) {
players.sendMessage("The player" + p.getName() + " needs your help! Have a look on his request");
}
}
}
else {
return false;
}
return true;
}
else {
sender.sendMessage("This command can only executed by a Player!");
return true;
}
}
 
}
 

This works fine. Now i tried to give the player another message when no "supporter" is online.

Code:java

@Override
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
Player p = null;
if(sender instanceof Player){
p = (Player) sender;
}
 
int supporters = 0;
if(p!= null) {
 
if(args.length == 0) {
if(supporters == 0) {
//No supporters online
}
else {
p.sendMessage("A Supporter has ben informed about your request!");
}
for(Player players : Bukkit.getServer().getOnlinePlayers()) {
if(players.hasPermission("supportix.supporter")) {
players.sendMessage("The player" + p.getName() + " needs your help! Have a look on his request");
supporter++
}
}
}
else {
return false;
}
return true;
}
else {
sender.sendMessage("This command can only executed by a Player!");
return true;
}
}
 
}

I'm a beginner in java and i have no clue how to implement this. I hope you can help me

regards Funky

 

Your logic seems right except that you're calculating the number of supporters after checking. Just before if (supporters == 0) is where you should loop through all the players and total up the number of ones with the necessary permission.

Speaking of ArrayLists, if you are using any Collection (which includes all Sets, Lists, etc), there is a convenient size() function which will tell you how many elements are in the collection.

 

Looks good, here is a proper and simple solution

Code:java

 
ArrayList<Player> supporters = new ArrayList<Player>();
for(Player players : Bukkit.getServer().getOnlinePlayers())
{
if(players.hasPermission("supportix.supporter"))
supporters.add(players);
}
for(Player supporter : supporters)
supporter.sendMessage("The player" + p.getName() + " needs your help! Have a look on his request");
if(supporters.size() == 0)
p.sendMessage("Sorry, no supporters online!");
else
p.sendMessage("Supporters have been informed!");
 

 

ohh Thanks! So my problem was that i checked the amount of supporters before the loop can increase the number of the supporter variable? And i looked up the size() function and i will use it next time Thanks for your supporting !!

Thank you very much for your help I'll gettin better and better )