help with admins command
#3

You've done some poor coding there, lad. You should decide whether to show a dialog or not AFTER your player loop there. Other way it'll show you a message upon first user you get (ID 0) if he's not an admin since you won't get count++ done.
Here's how I'd do it, maybe:

Код:
CMD:admins(playerid,params[])
{
	for(new i=0;i<MAX_PLAYERS;i++)
	{
	    new string[1000],astring[50],count,admin[80];
	    if(PlayerInfo[i][pAdmin] >= 1 && PlayerInfo[i][pAdminHide] == 0)
	    {
	        if(PlayerInfo[i][pAdmin] == 1) { admin = "Admin level [ 1 ]"; }
        	else if(PlayerInfo[i][pAdmin] == 4) { admin = "Admin level [ 4 ]"; }
        	else if(PlayerInfo[i][pAdmin] == 2) { admin = "Admin level  [ 2 ]"; }
        	else if(PlayerInfo[i][pAdmin] == 3) { admin = "Admin level  [ 3 ]"; }
        	else if(PlayerInfo[i][pAdmin] == 6) { admin = "[ CO-OWNER ]";}
        	else if(PlayerInfo[i][pAdmin] == 7) { admin = "[ OWNER]";}
			else if(PlayerInfo[i][pAdmin] == 5) { admin = "[ HEAD-ADMIN]";}
			format(string,1000,"%s {FFFFFF}%s {FF0000}%s[ID: %d]\n",string,admin,PlayerName(i),i);
			count++;
			format(astring,sizeof(astring),"Online admins(%d):",count);
	    }
	}
        //I do all these count checks AFTER the loop; it barely makes sense other way around.
        if(count >=1)
	    {
	        ShowPlayerDialog(playerid, 21323123, DIALOG_STYLE_MSGBOX, astring, string, "{FFFFFF}Close","");
	    }
     	else
	    {
	        SendClientMessage(playerid,0xFF0000FF,"there are no online admins!" );
			return 1;
	    }
	return 1;
}
Reply


Messages In This Thread
help with admins command - by scripter112 - 15.04.2018, 15:09
Re: help with admins command - by UFF - 15.04.2018, 17:05
Re: help with admins command - by Maxandmov - 15.04.2018, 20:11
Re: help with admins command - by Mugala - 15.04.2018, 20:40
Re: help with admins command - by scripter112 - 16.04.2018, 07:47

Forum Jump:


Users browsing this thread: 1 Guest(s)