[sim_sima]

Hello guys.
I made a /admins command, and it works just perfect when admins are online.
But when there are no admins online, its a little bit buggy. When no admins are online, its supposed to send this message: "No admins online at the moment" but it shows up multiple times.
Here is the command:

pawn Код:

COMMAND:admins(playerid, params[])
{
    SendClientMessage(playerid, COLOR_SYSTEMRED, "=========== Online Admins ===========");
    for(new i = 0; i < MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i))
        {
            if(PlayerInfo[i][pAdminLevel] >= 1)
            {
                new name[MAX_PLAYER_NAME], string[21 + MAX_PLAYER_NAME];
                GetPlayerName(i,name,sizeof(name));
                format(string, sizeof(string), "(Level:%i) %s (ID:%i)", PlayerInfo[i][pAdminLevel], name, i);
                SendClientMessage(playerid, COLOR_SYSTEMRED, string);
            }
            else if(PlayerInfo[i][pAdminLevel] == 0) SendClientMessage(playerid, COLOR_SYSTEMRED, "No admins online at the moment");
        }
    }
    SendClientMessage(playerid, COLOR_SYSTEMRED, "===================================");
    return 1;
}

Hope you can help me. Thank you.

[Jay.]

Quote:

Originally Posted by sim_sima Hello guys. I made a /admins command, and it works just perfect when admins are online. But when there are no admins online, its a little bit buggy. When no admins are online, its supposed to send this message: "No admins online at the moment" but it shows up multiple times. Here is the command: pawn Код: COMMAND:admins(playerid, params[]) {     SendClientMessage(playerid, COLOR_SYSTEMRED, "=========== Online Admins ===========");     for(new i = 0; i < MAX_PLAYERS; i++)     {         if(IsPlayerConnected(i))         {             if(PlayerInfo[i][pAdminLevel] >= 1)             {                 new name[MAX_PLAYER_NAME], string[21 + MAX_PLAYER_NAME];                 GetPlayerName(i,name,sizeof(name));                 format(string, sizeof(string), "(Level:%i) %s (ID:%i)", PlayerInfo[i][pAdminLevel], name, i);                 SendClientMessage(playerid, COLOR_SYSTEMRED, string);             }             else if(PlayerInfo[i][pAdminLevel] == 0) SendClientMessage(playerid, COLOR_SYSTEMRED, "No admins online at the moment");         }     }     SendClientMessage(playerid, COLOR_SYSTEMRED, "===================================");     return 1; } Hope you can help me. Thank you.

pawn Код:

COMMAND:admins(playerid, params[])
{
    SendClientMessage(playerid, COLOR_SYSTEMRED, "=========== Online Admins ===========");
    for(new i = 0; i < MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i))
        {
            if(PlayerInfo[i][pAdminLevel] >= 1)
            {
                new name[MAX_PLAYER_NAME], string[21 + MAX_PLAYER_NAME];
                GetPlayerName(i,name,sizeof(name));
                format(string, sizeof(string), "(Level:%i) %s (ID:%i)", PlayerInfo[i][pAdminLevel], name, i);
                SendClientMessage(playerid, COLOR_SYSTEMRED, string);
            }
            else if(PlayerInfo[i][pAdminLevel] == 0) SendClientMessage(playerid, COLOR_SYSTEMRED, "No admins online at the moment"); 
            return 1;
        }
    }
    SendClientMessage(playerid, COLOR_SYSTEMRED, "===================================");
    return 1;
}

[sim_sima]

Quote:

Originally Posted by Jay. pawn Код: COMMAND:admins(playerid, params[]) {     SendClientMessage(playerid, COLOR_SYSTEMRED, "=========== Online Admins ===========");     for(new i = 0; i < MAX_PLAYERS; i++)     {         if(IsPlayerConnected(i))         {             if(PlayerInfo[i][pAdminLevel] >= 1)             {                 new name[MAX_PLAYER_NAME], string[21 + MAX_PLAYER_NAME];                 GetPlayerName(i,name,sizeof(name));                 format(string, sizeof(string), "(Level:%i) %s (ID:%i)", PlayerInfo[i][pAdminLevel], name, i);                 SendClientMessage(playerid, COLOR_SYSTEMRED, string);             }             else if(PlayerInfo[i][pAdminLevel] == 0) SendClientMessage(playerid, COLOR_SYSTEMRED, "No admins online at the moment");             return 1;         }     }     SendClientMessage(playerid, COLOR_SYSTEMRED, "===================================");     return 1; }

Whats changed?

[Jay.]

Quote:

Originally Posted by sim_sima Whats changed?

I added

pawn Код:

return 1;

Under your message for no admins. I don't know if it will work..

[WooTFTW]

pawn Код:

COMMAND:admins(playerid, params[])
{
    new count;
    SendClientMessage(playerid, COLOR_SYSTEMRED, "=========== Online Admins ===========");
    for(new i = 0; i < MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i))
        {
            if(PlayerInfo[i][pAdminLevel] >= 1)
            {
                count ++;
                new name[MAX_PLAYER_NAME], string[21 + MAX_PLAYER_NAME];
                GetPlayerName(i,name,sizeof(name));
                format(string, sizeof(string), "(Level:%i) %s (ID:%i)", PlayerInfo[i][pAdminLevel], name, i);
                SendClientMessage(playerid, COLOR_SYSTEMRED, string);
            }
        }
    }
    if(!count)
    {
        SendClientMessage(playerid, COLOR_SYSTEMRED, "No admins online at the moment");
    }
    SendClientMessage(playerid, COLOR_SYSTEMRED, "===================================");
    return 1;
}

[sim_sima]

Thanks everyone