Doesnt give me the online list?

Hello, I tried to make a /who command to show who is online unfortunately when i write /who it only gives me the amount online ( not who is online )

Its like this:

Code:

    @Override
    public boolean onPlayerCommand(Player p, Command command, String label, String[] args) {       
        StringBuilder players = new StringBuilder();
        for (Player player : Bukkit.getOnlinePlayers()) {
            if (players.length() > 0) {
                players.append(", ");       
            players.append(player.getDisplayName());
            }
            if (args.length == 0) { 
        p.sendMessage(ChatColor.BLUE + "============{" + ChatColor.GOLD + " Spillere online: "+ ChatColor.BLUE + "}============");
            p.sendMessage(players.toString());
            p.sendMessage(ChatColor.BLUE + "=====================================");
            p.sendMessage(ChatColor.GREEN + "Pålogget: " + ChatColor.WHITE + plugin.getServer().getOnlinePlayers().length);
            return true;
        } else if (args.length == 1) {
            Player v = this.plugin.playerMatch(args[0]);
            if (v != null) {
            p.sendMessage(this.userHandler.getNameColor(v) +  ChatColor.GREEN + " er pålogget akuratt nå.");
                return true;       
            } else {
                p.sendMessage(ChatColor.GOLD + "[Alarm] " + ChatColor.RED + "Fant ingen match.");
            return true; 
            }
        }    }
        return false;    }    }

And I often get Socket Write error, anyone know something about that?

 

I guess it's this line:
p.sendMessage(ChatColor.GREEN + "Pålogget: " + ChatColor.WHITE + plugin.getServer().getOnlinePlayers().length);
change it to
p.sendMessage(ChatColor.GREEN + "Pålogget: " + ChatColor.WHITE + players.toString());

 

W

Well, the pålogget message tells me how many is online, the real thing is there it say's players.toString.

bump

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

 

Did you change the line I showed you?

edit: nvm, found the line with players.toString()

 

Code:

@Override
public boolean onPlayerCommand(Player p, Command command, String label, String[] args) {
    StringBuilder players = new StringBuilder();
 
    for (Player player : Bukkit.getOnlinePlayers()) {
        players.append(player.getName() + ", ");
    }
 
    String online = players.toString().substring(0,players.toString().length() - 2);
 
    if (args.length == 0) {
        p.sendMessage(ChatColor.BLUE + "============{" + ChatColor.GOLD + " Spillere online: " + ChatColor.BLUE + "}============");
        p.sendMessage(online);
        p.sendMessage(ChatColor.BLUE + "=====================================");
        p.sendMessage(ChatColor.GREEN + "Pålogget: " + ChatColor.WHITE + Bukkit.getOnlinePlayers().length);
        players.delete(0, players.length());
        return true;
    } else if (args.length == 1) {
        Player v = Bukkit.getServer().getPlayer(args[0]);
        if (v != null) {
            p.sendMessage(v.getDisplayName() +  ChatColor.GREEN + " er pålogget akuratt nå.");
            return true;
        } else {
            p.sendMessage(ChatColor.GOLD + "[Alarm] " + ChatColor.RED + "Fant ingen match.");
            return true;
        }
    }
return false;
}

 

Thanks