[maximthepain]

Код:

CMD:admins(playerid, params[])
{
	new online, astring[256],string[50];
	if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command.");
	for(new i = 0, j = GetPlayerPoolSize(); i <= j; i++)
	{
		if(PlayerInfo[i][pAdmin]) online ++;
	}
	if(online == 0) return SendClientMessage(playerid, COLOR_GREY, "- Theres no admins online -");
	format(string, sizeof(string), "- NY:RP Admins (%d Online) -", online);
	SendClientMessage(playerid, COLOR_LIGHTRED, string);
	for(new x = 0, j = GetPlayerPoolSize(); x <= j; x++)
	{
		if(PlayerInfo[x][pAdmin])
		{
			format(astring, sizeof(astring), "{A35346}Name: {FFFFFF}%s{FFFFFF} | {A35346}Admin Level: {FFFFFF}%s (%d){FFFFFF} | {A35346}Admin Duty:{FFFFFF} %s |", RPN(x), RPALN(x), PlayerInfo[x][pAdmin], ADSTAT(x));
			SendClientMessage(playerid, COLOR_WHITE, astring);
		}
	}
	SendClientMessage(playerid, COLOR_YELLOW, "Do not PM admins! Use /report & before reporting use /rules.");
	return 1;
}

I see 1 online admin but astring is not showing. If other admins connect, its showing only them..
Why this is happening? Help..

[Dokins]

This string is of excessive length

pawn Код:

format(astring, sizeof(astring), "{A35346}Name: {FFFFFF}%s{FFFFFF} | {A35346}Admin Level: {FFFFFF}%s (%d){FFFFFF} | {A35346}Admin Duty:{FFFFFF} %s |", RPN(x), RPALN(x), PlayerInfo[x][pAdmin], ADSTAT(x));

It would be better to use string concatenation.

[maximthepain]

Quote:

Originally Posted by Dokins This string is of excessive length pawn Код: format(astring, sizeof(astring), "{A35346}Name: {FFFFFF}%s{FFFFFF} | {A35346}Admin Level: {FFFFFF}%s (%d){FFFFFF} | {A35346}Admin Duty:{FFFFFF} %s |", RPN(x), RPALN(x), PlayerInfo[x][pAdmin], ADSTAT(x)); It would be better to use string concatenation.

what do you mean? can you show me? it will help me on future strings.

[10MIN]

I think he says that you shouldn't have strings that long.
PS:The maximum client message is 128 characters. Measure them with http://www.lettercount.com/. (add some _random_ data to the strings, so you would know how long it need to be)
For example a string like: %s have joined the server! would be transformed to: A_LONG_PLAYER_NAME_12345 have joined the server! before counting the characters.

[Arbico]

Bro, Try this, Rep+ if it works please

PHP код:

CMD:admins(playerid, params[]) {
     new Str[200], Name[MAX_PLAYER_NAME];
     SendClientMessage(playerid, COLOR_GREEN, "-------Online Admins-------");
     for(new i = 0; i <= GetPlayerPoolSize(); i++) {
          if(IsPlayerAdmin(i)) { //Of course, Change IsPlayerAdmin to detect custom admin level.
               GetPlayerName(playerid, Name, sizeof(Name));
               format(Str, sizeof(Str), "%s - ID:%i", Name, playerid);
               SendClientMessage(playerid, COLOR_GREY, Str);
          }
     }
     return 1;
} 


[maximthepain]

Quote:

Originally Posted by Arbico Bro, Try this, Rep+ if it works please PHP код: CMD:admins(playerid, params[]) {      new Str[200], Name[MAX_PLAYER_NAME];      SendClientMessage(playerid, COLOR_GREEN, "-------Online Admins-------");      for(new i = 0; i <= GetPlayerPoolSize(); i++) {           if(IsPlayerAdmin(i)) { //Of course, Change IsPlayerAdmin to detect custom admin level.                GetPlayerName(playerid, Name, sizeof(Name));                format(Str, sizeof(Str), "%s - ID:%i", Name, playerid);                SendClientMessage(playerid, COLOR_GREY, Str);           }      }      return 1; }

not working.

[DarkBr]

PHP код:

CMD:admins(playerid, params[])
{
    new online, astring[256],string[50];
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command.");
    for(new i = 0, j = GetPlayerPoolSize(); i <= j; i++)
    {
        if(IsPlayerConnected(i) && PlayerInfo[i][pAdmin] > 0)
        {
            online ++;
            format(string, sizeof(string), "- NY:RP Admins (%d Online) -", online);
            format(astring, sizeof(astring), "{A35346}Name: {FFFFFF}%s{FFFFFF} | {A35346}Admin Level: {FFFFFF}%s (%d){FFFFFF} | {A35346}Admin Duty:{FFFFFF} %s |\n\r", RPN(i), RPALN(i), PlayerInfo[i][pAdmin], ADSTAT(i));
        }
    }
    if(online)
    {
        SendClientMessage(playerid, COLOR_LIGHTRED, string);
        SendClientMessage(playerid, COLOR_WHITE, astring);
    }
    else
    {
        SendClientMessage(playerid, COLOR_GREY, "- Theres no admins online -");
        SendClientMessage(playerid, COLOR_YELLOW, "Do not PM admins! Use /report & before reporting use /rules.");
    }
    return 1;
}