[Outcast]

I am currently learning to script. I made a /stats command but instead of showing:
-------Current stats: Player_Name------
blah blah blah...

it shows me:
-------Current stats: 32----------

32 is the ammount of money I have.

Here's the code:

Код:

public OnPlayerCommandText(playerid, cmdtext[])
{
	if (strcmp("/stats", cmdtext, true, 10) == 0)
	{
		new statsname[128];
		new statsmsg[256];
		format(statsname, 128, "----------Current stats: %s----------", PLAYER_NAME);
		SendClientMessage(playerid, COLOR_YELLOW, statsname);
		PlayerInfo[playerid][pCash] = GetPlayerMoney(playerid);
 		format(statsmsg, 256, "Level: [%b] Admin level: [%i] Money: [$%i]", PlayerInfo[playerid][pScore], PlayerInfo[playerid][pAdminLevel], PlayerInfo[playerid][pCash]);
		SendClientMessage(playerid, COLOR_YELLOW, statsmsg);
		return 1;
	}
	return 1;
}

PLAYER_NAME is defined as MAX_PLAYER_NAME


I just don't get it.

[Doom8890]

How about this?

pawn Код:

public OnPlayerCommandText(playerid, cmdtext[])
{
    if (strcmp("/stats", cmdtext, true, 10) == 0)
    {
        new statsname[128];
        new statsmsg[256];
                new pName[MAX_PLAYER_NAME];
                GetPlayerName(playerid,pName,sizeof(pName));
        format(statsname, 128, "----------Current stats: %s----------", pName);
        SendClientMessage(playerid, COLOR_YELLOW, statsname);
        PlayerInfo[playerid][pCash] = GetPlayerMoney(playerid);
        format(statsmsg, 256, "Level: [%b] Admin level: [%i] Money: [$%i]", PlayerInfo[playerid][pScore], PlayerInfo[playerid][pAdminLevel], PlayerInfo[playerid][pCash]);
        SendClientMessage(playerid, COLOR_YELLOW, statsmsg);
        return 1;
    }
    return 1;
}

[Joe Staff]

A player's name is a string(text), MAX_PLAYER_NAME is '24' which is a number(integer). There's a function to place a player's name onto a variable designated as a string.

pawn Код:

new pname[24];
GetPlayerName(playerid,pname,24);
format(mystring,sizeof(mystring),"PlayerID %d's name is %s",playerid,pname);

[Outcast]

Quote:

Originally Posted by Doom8890 How about this? pawn Код: public OnPlayerCommandText(playerid, cmdtext[]) {     if (strcmp("/stats", cmdtext, true, 10) == 0)     {         new statsname[128];         new statsmsg[256];                 new pName[MAX_PLAYER_NAME];                 GetPlayerName(playerid,pName,sizeof(pName));         format(statsname, 128, "----------Current stats: %s----------", pName);         SendClientMessage(playerid, COLOR_YELLOW, statsname);         PlayerInfo[playerid][pCash] = GetPlayerMoney(playerid);         format(statsmsg, 256, "Level: [%b] Admin level: [%i] Money: [$%i]", PlayerInfo[playerid][pScore], PlayerInfo[playerid][pAdminLevel], PlayerInfo[playerid][pCash]);         SendClientMessage(playerid, COLOR_YELLOW, statsmsg);         return 1;     }     return 1; }

It works, thanks.
Fala lik

[Haydz]

also instead of using

pawn Код:

PlayerInfo[playerid][pScore]

you could use

pawn Код:

GetPlayerScore(playerid)

You could also do the same with money.