[Guitar]

Hello guys,

Sorry, It's me again XD -- Anyway, I scripted this cmd:

pawn Код:

CMD:getplayerstats(playerid, params[])
{
    if(PlayerInfo[playerid][pAdmin] <= 2) return SendClientMessage(playerid, COLOR_RED, "Error Occured: This command is only for Admins.");
    
    new str[256], TargetName[MAX_PLAYER_NAME], IP[16], targetid;
    new Money = GetPlayerMoney(targetid);
    PlayerNameGet(targetid, TargetName, 24);
    GetPlayerIp(targetid,IP,sizeof IP);
    
    if(sscanf(params,"u", targetid)) return SendClientMessage(playerid, 0xFFFFFFF, "[ADMIN]/getplayerstats [playerid/name]");
    
    format(str, 256, ""#COL_WHITE"=======================[%s statistics]=======================", TargetName);
    SendClientMessage(playerid,COLOR_WHITE,str);
    format(str,128, ">"#COL_WHITE"%s: IP[%s] - Money[$%i] - Kills[%i] - Deaths[%i] - Admin[%i] - Skin[%i]", TargetName, IP, Money, PlayerInfo[targetid][pKills], PlayerInfo[targetid][pDeaths], PlayerInfo[targetid][pAdmin], GetPlayerSkin(targetid));
    SendClientMessage(playerid,COLOR_WHITE,str);
    return 1;
}

But it doesn't give me the player's name and the right numbers, can you re-script it for me again? Thanks <3

[Calgon]

You need to let the sscanf code execute first before you get the details about the player. 'targetid' is 0 until the sscanf code detects who the player is you're using the command on.

pawn Код:

CMD:getplayerstats(playerid, params[])
{
    if(PlayerInfo[playerid][pAdmin] <= 2) return SendClientMessage(playerid, COLOR_RED, "Error Occured: This command is only for Admins.");

    new str[128], TargetName[MAX_PLAYER_NAME], IP[16], targetid;

    if(sscanf(params,"u", targetid)) return SendClientMessage(playerid, 0xFFFFFFF, "[ADMIN]/getplayerstats [playerid/name]");

    new Money = GetPlayerMoney(targetid);
    PlayerNameGet(targetid, TargetName, 24);
    GetPlayerIp(targetid,IP,sizeof IP);

    format(str, 128, ""#COL_WHITE"=======================[%s statistics]=======================", TargetName);
    SendClientMessage(playerid,COLOR_WHITE,str);
    format(str,128, ">"#COL_WHITE"%s: IP[%s] - Money[$%i] - Kills[%i] - Deaths[%i] - Admin[%i] - Skin[%i]", TargetName, IP, Money, PlayerInfo[targetid][pKills], PlayerInfo[targetid][pDeaths], PlayerInfo[targetid][pAdmin], GetPlayerSkin(targetid));
    SendClientMessage(playerid,COLOR_WHITE,str);
    return 1;
}

[Guitar]

Oh, thank you I will study your cmd and my cmd and see what's the difference between them and learn from my mistake .

[Calgon]

All I did was move the 3 lines of code for the money, name and IP down below the sscanf line.

[[MM]RoXoR[FS]]

EDIT : late

[Guitar]

Oh, so my code was right? I actually stopped believing in myself :S

[Calgon]

Read what I wrote, it explains everything...