Loop show all player job - Printable Version

Loop show all player job - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: Loop show all player job (/showthread.php?tid=622300)

Loop show all player job - Cerealguy - 20.11.2016

waaaaaa help me :P pls

Screen:

pawn Код:

CMD:pjobs(playerid, params[])
{
    new str[50], STR[512];
    strcat(STR, "Name(ID)\tJob\n");
    for(new i; i < MAX_PLAYERS; i++)
    {
        if(pInfo[i][USER_LOGGED_IN] == true)
        format(str, sizeof(str),"%s(%i)\t\%s\n", GetPlayerName(i), i, GetClassName(i));
        if(strlen(str) < 1) strdel(str, 0, 50);
        strcat(STR, str);
    }
    ShowPlayerDialog(playerid, DIALOG_PJOBS, DIALOG_STYLE_TABLIST_HEADERS, ""COL_BLUE"Players Jobs", STR, "Exit", "");
    return 1;
}

Re: Loop show all player job - Konstantinos - 20.11.2016

https://sampwiki.blast.hk/wiki/GetPlayerName

The function does not return the name but it is passed by reference.

https://sampwiki.blast.hk/wiki/GetPlayerPoolSize

Player pool is very useful to reduce the iterations. foreach/y_iterate is even better.

The problem is not using brackets to make it a block of code and strcat is called again and again.

Re: Loop show all player job - Kaliber - 20.11.2016

Here you go:

PHP код:



CMD:pjobs(playerid, params[])

{

    new str[512] = "Name(ID)\tJob";

    for(new i=GetPlayerPoolSize(),n[MAX_PLAYER_NAME]; i != -1; i--)

    {

        if(!IsPlayerConnected(i) || !pInfo[i][USER_LOGGED_IN]) continue;

        GetPlayerName(i,n,MAX_PLAYER_NAME);

        format(str, sizeof(str),"%s\n%s (%i)\t%s",str,n, i, GetClassName(i));

    }

    ShowPlayerDialog(playerid, DIALOG_PJOBS, DIALOG_STYLE_TABLIST_HEADERS, ""COL_BLUE"Players Jobs", str, "Exit", "");

    return 1;

}

Show us your GetClassName

Respuesta: Re: Loop show all player job - Cerealguy - 20.11.2016

Quote:

Originally Posted by Konstantinos

https://sampwiki.blast.hk/wiki/GetPlayerName

The function does not return the name but it is passed by reference.

https://sampwiki.blast.hk/wiki/GetPlayerPoolSize

Player pool is very useful to reduce the iterations. foreach/y_iterate is even better.

The problem is not using brackets to make it a block of code and strcat is called again and again.

Ohh ok, i redefined GetPlayeName

pawn Код:

stock GetPlayerNameEx(playerid)
{
    new pname[MAX_PLAYER_NAME];
    GetPlayerName(playerid, pname, MAX_PLAYER_NAME);
    return pname;
}
#define GetPlayerName(%0)   GetPlayerNameEx(%0)

I had time without scripting, I did not know that function(GetPlayerPoolSize), thanks!

Quote:

Originally Posted by Kaliber

Here you go:

PHP код:



CMD:pjobs(playerid, params[])

{

    new str[512] = "Name(ID)\tJob";

    for(new i=GetPlayerPoolSize(),n[MAX_PLAYER_NAME]; i != -1; i--)

    {

        if(!IsPlayerConnected(i) || !pInfo[i][USER_LOGGED_IN]) continue;

        GetPlayerName(i,n,MAX_PLAYER_NAME);

        format(str, sizeof(str),"%s\n%s (%i)\t%s",str,n, i, GetClassName(i));

    }

    ShowPlayerDialog(playerid, DIALOG_PJOBS, DIALOG_STYLE_TABLIST_HEADERS, ""COL_BLUE"Players Jobs", str, "Exit", "");

    return 1;

}

Show us your GetClassName

Thanks friend,

pawn Код:

stock GetClassName(playerid)
{
    new ClassName[MAX_PLAYERS];
    
    if(GetPlayerTeam(playerid) == TEAM_CIVILIAN)
    {
        if(pInfo[playerid][Job] == 0) SetPVarString(playerid, "JobName", "Terrorist");
        if(pInfo[playerid][Job] == 1) SetPVarString(playerid, "JobName", "Hitman");
        if(pInfo[playerid][Job] == 2) SetPVarString(playerid, "JobName", "Weapon Dealer");
        if(pInfo[playerid][Job] == 3) SetPVarString(playerid, "JobName", "Rapist");
    }
    
    if(GetPlayerTeam(playerid) == TEAM_POLICE) SetPVarString(playerid, "JobName", ""COL_POLICE"Police");
    if(GetPlayerTeam(playerid) == TEAM_SWAT) SetPVarString(playerid, "JobName", ""COL_FBI"S.W.A.T");
    if(GetPlayerTeam(playerid) == TEAM_FBI) SetPVarString(playerid, "JobName", ""COL_FBI"F.B.I");
    if(GetPlayerTeam(playerid) == TEAM_CIA) SetPVarString(playerid, "JobName", ""COL_CIA"C.I.A");
    if(GetPlayerTeam(playerid) == TEAM_ARMY) SetPVarString(playerid, "JobName", ""COL_ARMY"ARMY");

    GetPVarString(playerid, "JobName", ClassName, sizeof(ClassName));
    return ClassName;
}