[HydraX]

pawn Code:

dcmd_admins(playerid, params[])
{
    #pragma unused params

    new
        string[256],count,
        pName[MAX_PLAYER_NAME];
    foreach(Player,i)
    {
        if(PlayerInfo[i][level])
            GetPlayerName(i,pName,24);
        else
            continue;

        count++;
        switch(PlayerInfo[i][level])
        {
            case 1:format(string, 60, "Name: %s | Level: 1 [VIP]",pName);
            case 2:format(string, 60, "Name: %s | Level: 2 [MOD]",pName);
            case 3:format(string, 60, "Name: %s | Level: 3 [ADMIN]",pName);
        }
        if(IsPlayerAdmin(i))
            format(string, 60, "Name: %s | Level: 4 [SERVER OWNER]",pName);

        ShowPlayerDialog(playerid, 21, DIALOG_STYLE_MSGBOX, "{FF0000}--- Admins ---", string, "Close","");
    }
    if(count)
        format(string, 128,"Total Online: %d",count);
    else
        return SendClientMessage(playerid,COLOR_BROWN,"There are currently no admins online."),1;

    SendClientMessage(playerid,COLOR_BROWN,string);
    return 1;
}

The command supposedly shows a dialog with the admins online, however I want the integers (count) to be implemented in the dialog without error, so can you help me?

[HydraX]

The only thing I need is the correct coding, so I can learn from it.

[Shadoww5]

PHP Code:

dcmd_admins(playerid, params[])
{
    #pragma unused params
    new string[512], count, pName[60];
    for(new i = 0; i < MAX_PLAYERS; i ++)
    {
        if(!PlayerInfo[i][level] || !IsPlayerConnected(i)) continue;
        GetPlayerName(i,pName,24);
        count++;
        if(IsPlayerAdmin(i)) { format(pName, 60, "Name: %s | Level: 4 [SERVER OWNER]",pName); }
        else if(PlayerInfo[i][level] == 1) { format(pName, 60, "Name: %s | Level: 1 [VIP]",pName); }
        else if(PlayerInfo[i][level] == 2) { format(pName, 60, "Name: %s | Level: 2 [MOD]",pName); }
        else if(PlayerInfo[i][level] == 3) { format(pName, 60, "Name: %s | Level: 3 [ADMIN]",pName); }
        strcat(string, pName);
    }
    ShowPlayerDialog(playerid, 21, DIALOG_STYLE_MSGBOX, "{FF0000}--- Admins ---", string, "Close","");
    if(count) format(string, 128,"Total Online: %d",count);
    else return SendClientMessage(playerid,COLOR_BROWN,"There are currently no admins online."),1;
    SendClientMessage(playerid,COLOR_BROWN,string);
    return 1;
} 


[HydraX]

No no, I want the integer, aka this.

pawn Code:

if(count) format(string, 128,"Total Online: %d",count);
    else return SendClientMessage(playerid,COLOR_BROWN,"There are currently no admins online."),1;
    SendClientMessage(playerid,COLOR_BROWN,string);

In the dialog also.

[HydraX]

So no one knows?

[MoroDan]

PHP Code:

dcmd_admins(playerid, params[])
{
    #pragma unused params
    new string[256], str[64], count,
             pName[MAX_PLAYER_NAME];
        
    foreach(Player,i)
    {
        if(PlayerInfo[i][level]) GetPlayerName(i,pName,24);
        else continue;
        count++;
        switch(PlayerInfo[i][level])
        {
            case 1: format(string, 60, "Name: %s | Level: 1 [VIP]",pName);
            case 2: format(string, 60, "Name: %s | Level: 2 [MOD]",pName);
            case 3: format(string, 60, "Name: %s | Level: 3 [ADMIN]",pName);
        }
        if(IsPlayerAdmin(i))
            format(string, 60, "Name: %s | Level: 4 [SERVER OWNER]",pName);
    }
    if(count)
    {
        format(str, 64, "There are {FF0000}%d {FFFFFF}admins online !", count);
        strcat(string, str);
        ShowPlayerDialog(playerid, 21, DIALOG_STYLE_MSGBOX, "{FF0000}--- Admins ---", string, "Close","");
    }
    else return SendClientMessage(playerid,COLOR_BROWN,"There are currently no admins online.");
    return 1;
} 


[jameskmonger]

pawn Code:

dcmd_admins(playerid, params[])
{
    #pragma unused params

    new string[256], finalstring[256], count, pName[MAX_PLAYER_NAME];
    foreach(Player,i)
    {
        if(PlayerInfo[i][level]) GetPlayerName(i,pName,24);
        else continue;

        count++;

        switch(PlayerInfo[i][level]) {
            case 1: { format(string, 60, "Name: %s | Level: 1 [VIP]\n",pName), strcat(finalstring, string); }
            case 2: { format(string, 60, "Name: %s | Level: 2 [MOD]\n",pName), strcat(finalstring, string); }
            case 3: { format(string, 60, "Name: %s | Level: 3 [ADMIN]\n",pName), strcat(finalstring, string); }
        }

        if(IsPlayerAdmin(i)) { format(string, 60, "Name: %s | Level: 4 [SERVER OWNER]\n", pName), strcat(finalstring, string); }
    }

    if(count) { format(string, 40,"Total Online: %d",count), strcat(finalstring, string);
    else return SendClientMessage(playerid,COLOR_BROWN,"There are currently no admins online."),1;

    ShowPlayerDialog(playerid, 21, DIALOG_STYLE_MSGBOX, "{FF0000}--- Admins ---", string, "Close","");
    return 1;
}

Try that.

[MadeMan]

pawn Code:

dcmd_admins(playerid, params[])
{
    #pragma unused params

    new
        string[512],count,
        pName[64];
    foreach(Player,i)
    {
        if(PlayerInfo[i][level])
            GetPlayerName(i,pName,24);
        else
            continue;

        count++;
        switch(PlayerInfo[i][level])
        {
            case 1:format(pName, sizeof(pName), "Name: %s | Level: 1 [VIP]",pName);
            case 2:format(pName, sizeof(pName), "Name: %s | Level: 2 [MOD]",pName);
            case 3:format(pName, sizeof(pName), "Name: %s | Level: 3 [ADMIN]",pName);
        }
        if(IsPlayerAdmin(i))
            format(pName, sizeof(pName), "Name: %s | Level: 4 [SERVER OWNER]",pName);
        
        strcat(string, pName); strcat(string, "\n");
    }
    if(count)
        format(pName, sizeof(pName),"Total Online: %d",count);
        strcat(string, pName);
    else
        return SendClientMessage(playerid,COLOR_BROWN,"There are currently no admins online."),1;

    ShowPlayerDialog(playerid, 21, DIALOG_STYLE_MSGBOX, "{FF0000}--- Admins ---", string, "Close","");
    return 1;
}

[jameskmonger]

My code is the only one so far that actually lists the admins :P

[HydraX]

Quote:

Originally Posted by jameskmonger pawn Code: dcmd_admins(playerid, params[]) {     #pragma unused params     new string[256], finalstring[256], count, pName[MAX_PLAYER_NAME];     foreach(Player,i)     {         if(PlayerInfo[i][level]) GetPlayerName(i,pName,24);         else continue;         count++;         switch(PlayerInfo[i][level]) {             case 1: { format(string, 60, "Name: %s | Level: 1 [VIP]\n",pName), strcat(finalstring, string); }             case 2: { format(string, 60, "Name: %s | Level: 2 [MOD]\n",pName), strcat(finalstring, string); }             case 3: { format(string, 60, "Name: %s | Level: 3 [ADMIN]\n",pName), strcat(finalstring, string); }         }         if(IsPlayerAdmin(i)) { format(string, 60, "Name: %s | Level: 4 [SERVER OWNER]\n", pName), strcat(finalstring, string); }     }     if(count) { format(string, 40,"Total Online: %d",count), strcat(finalstring, string);     else return SendClientMessage(playerid,COLOR_BROWN,"There are currently no admins online."),1;     ShowPlayerDialog(playerid, 21, DIALOG_STYLE_MSGBOX, "{FF0000}--- Admins ---", string, "Close","");     return 1; } Try that.

I think your missing a braket, I keep getting the undefined symbols when they are clearly defined/