[Visio]

i have a problem,i have this

Код:

if(strcmp(cmd,"/licenses",true)==0)
    {
        if(IsPlayerConnected(playerid))
	    {
	        new pzbori = PlayerInfo[playerid][pzbor];
	    	new pvapori = PlayerInfo[playerid][pvapor];
	    	new ppestei = PlayerInfo[playerid][ppeste];
	    	new parmai = PlayerInfo[playerid][parma];
	        for(new i = 0; i < MAX_PLAYERS; i++)
			{
	        	if(PlayerInfo[i][pzbor] == 0)
					{
					    PlayerInfo[playerid][pFlyLic] = 0;
					    format(string, sizeof(string), "** Flying License: Expired[%d/10].",pzbori);
						SendClientMessage(playerid, COLOR_LICENSES, string);
						return 1;
		   			}
					else
					{
						format(string, sizeof(string), "** Flying License: Avaible[%d/10].",pzbori);
						SendClientMessage(playerid, COLOR_LICENSES, string);
					    return 1;
					}
				if(PlayerInfo[ia][ppeste] == 0)
					{
					    PlayerInfo[playerid][pFlyLic] = 0;
					    format(string, sizeof(string), "** Fishing License: Expired[%d/10].",ppestei);
						SendClientMessage(playerid, COLOR_LICENSES, string);
						return 1;
		   			}
					else
					{
						format(string, sizeof(string), "** Fishing License: Avaible[%d/10].",ppestei);
						SendClientMessage(playerid, COLOR_LICENSES, string);
					    return 1;
					}
			}
		}
	    return 1;
 	}

but te problem is that is show me only Flying License,where is the mistake?

[Visio]

pelase help me is urgent

[RedCrossER]

Check string length

[Visio]

i try but nothing

[Visio]

please help

[Scott Zulkifli]

pawn Код:

if(PlayerInfo[ia][ppeste] == 0)

Quote:

[ia]? or [i]?

[Visio]

Quote:

Originally Posted by ****** Why the loop?

just another try...

[Visio]

Quote:

Originally Posted by Scott Zulkifli pawn Код: if(PlayerInfo[ia][ppeste] == 0)

there i try but isn't work

[Threshold]

You have a return 1; in each if statement, so if you have a flying license, the code will stop there and will not continue. Your code should be this:

pawn Код:

if(strcmp(cmd,"/licenses",true)==0)
{
    if(IsPlayerConnected(playerid))
    {
        new pzbori = PlayerInfo[playerid][pzbor];
        new pvapori = PlayerInfo[playerid][pvapor];
        new ppestei = PlayerInfo[playerid][ppeste];
        new parmai = PlayerInfo[playerid][parma];
        if(PlayerInfo[playerid][pzbor] == 0)
        {
            PlayerInfo[playerid][pFlyLic] = 0;
            format(string, sizeof(string), "** Flying License: Expired[%d/10].",pzbori);
            SendClientMessage(playerid, COLOR_LICENSES, string);
        }
        else
        {
            format(string, sizeof(string), "** Flying License: Available[%d/10].",pzbori);
            SendClientMessage(playerid, COLOR_LICENSES, string);
        }
        if(PlayerInfo[playerid][ppeste] == 0)
        {
            PlayerInfo[playerid][pFlyLic] = 0;
            format(string, sizeof(string), "** Fishing License: Expired[%d/10].",ppestei);
            SendClientMessage(playerid, COLOR_LICENSES, string);
        }
        else
        {
            format(string, sizeof(string), "** Fishing License: Avaible[%d/10].",ppestei);
            SendClientMessage(playerid, COLOR_LICENSES, string);
        }
    }
    return 1;
}

[JaKe Elite]

^^

Why need IsPlayerConnected?
If the player is typing is already connected.
Is there any invalid player that can type commands? LoL.
Anyway. Remove the loop. It's useless. You didn't use it in your command.