[Thrarod]

I made a command that shows your stats but HP and Armor doesnt show right like;

Health is 90 but shows 1, health is 100 but shows 1
Armor is 0 but shows 1, armor is 80 but shows 1

Code

pawn Код:

if(strcmp(cmdtext, "/stats", true) == 0)
    {
      new StatMSG[128];
        new pName[MAX_PLAYER_NAME];
        GetPlayerName(playerid, pName, sizeof(pName));
        new Float:pHealth;
        new Float:pArmour;
      format(StatMSG,sizeof(StatMSG),"[Stats] Name: %s, Health: %d, Armor: %d, Money: %d", pName, GetPlayerHealth(playerid, pHealth), GetPlayerArmour(playerid, pArmour), GetPlayerMoney(playerid));
        if (Job[playerid] == 0) {
        SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
        SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: Cop");
        }else if (Job[playerid] == 1) {
        SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
        SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: Criminal");
        }else if (Job[playerid] == 255) {
        SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
        SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: None");
        }
        return 1;
    }

[Dark_Kostas]

At formats you need to use %f for float, not %d and also it wont work like that

pawn Код:

if(strcmp(cmdtext, "/stats", true) == 0)
{
    new StatMSG[128];
    new pName[MAX_PLAYER_NAME];
    GetPlayerName(playerid, pName, sizeof(pName));
    new Float:pHealth;
    new Float:pArmour;
    GetPlayerHealth(playerid, pHealth);
    GetPlayerArmour(playerid, pArmour);
    format(StatMSG,sizeof(StatMSG),"[Stats] Name: %s, Health: %f, Armor: %f, Money: %d", pName, pHealth, pArmour, GetPlayerMoney(playerid));
    if (Job[playerid] == 0) {
    SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
    SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: Cop");
    }else if (Job[playerid] == 1) {
    SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
    SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: Criminal");
    }else if (Job[playerid] == 255) {
    SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
    SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: None");
    }
    return 1;
}

EDIT: I did some stupid things before, check again the code if you have already.

[Thrarod]

Changed anything?

[DJDhan]

Код:

if(strcmp(cmdtext, "/stats", true,6) == 0)
{
	new StatMSG[128];
	new pName[MAX_PLAYER_NAME];
	GetPlayerName(playerid, pName, sizeof(pName));
	new Float:pHealth;
	new Float:pArmour;
       GetPlayerHealth(playerid, pHealth); 
	GetPlayerArmour(playerid, pArmour);
	
	format(StatMSG,sizeof(StatMSG),"[Stats] Name: %s, Health: %f, Armor: %f, Money: %d", pName, pHealth, pArmour, GetPlayerMoney(playerid));
	
	if (Job[playerid] == 0) 
	{
		SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
		SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: Cop");
	}
	else if (Job[playerid] == 1) 
	{
		SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
		SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: Criminal");
	}
	else if (Job[playerid] == 255) 
	{
		SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
		SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: None");
	}
	return 1;
}

[Dark_Kostas]

Quote:

Originally Posted by Thrarod Changed anything?

Yes look it again.

DJDhan your code wont work. GetPlayerArmour and GetPlayerHealth doesnt return anything if you use it like that.

[TheInnocentOne]

Uhm none of that code will work. You need to get the health and armour into the floats BEFORE formatting the string. GetPlayerArmour and GetPlayerHealth do not return anything..

Replace your command with this code:

pawn Код:

if(strcmp(cmdtext, "/stats", true) == 0)
{
  new StatMSG[128];
     new pName[MAX_PLAYER_NAME];
GetPlayerName(playerid, pName, sizeof(pName));
new Float:pHealth;
new Float:pArmour;
GetPlayerHealth(playerid, pHealth);
GetPlayerArmour(playerid, pArmour);
  format(StatMSG,sizeof(StatMSG),"[Stats] Name: %s, Health: %f, Armor: %f, Money: %d", pName, pHealth, pArmour, GetPlayerMoney(playerid));
if (Job[playerid] == 0) {
SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: Cop");
}else if (Job[playerid] == 1) {
SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: Criminal");
}else if (Job[playerid] == 255) {
SendClientMessage(playerid, COLOR_FLBLUE, StatMSG);
SendClientMessage(playerid, COLOR_FLBLUE, "[Stats] Job: None");
}
return 1;
}

[DJDhan]

Quote:

Originally Posted by Dark_Kostas Quote: Originally Posted by Thrarod Changed anything? Yes look it again. DJDhan your code wont work. GetPlayerArmour and GetPlayerHealth doesnt return anything if you use it like that.

Well I edited it when I found out about my error.

[TheInnocentOne]

I already posted the correct code

[Thrarod]

Yeah yours is right but only %f may go crazy like 91.324123412151 etc. so changed to %0.f,

Thanks all, also TIO, PM me

[Dark_Kostas]

Quote:

Originally Posted by TheInnocentOne Uhm none of that code will work. You need to get the health and armour into the floats BEFORE formatting the string. GetPlayerArmour and GetPlayerHealth do not return anything.. Replace your command with this code:

which is the difference between mine and yours? except that your code is not indicated?