[Sid_Alexander]

i got this little /fine command, but the problem is whenever i /fine someone, they ask me the reaso after it, i was thinking to edit it and add something so i can fine and people can also see the reason
Example : USAGE: /fine [playerid] [amount] [reason]
Right now its : USAGEL /fine [playerid] [amount]

Here is my code

pawn Код:

if(strcmp(cmd, "/fine", true) == 0)
    {
        if(IsPlayerConnected(playerid))
        {
            tmp = strtok(cmdtext, idx);
            if(!strlen(tmp))
            {
                SendClientMessage(playerid, COLOR_GRAD2, "USAGE: /fine [playerid/PartOfName] [amount]");
                return 1;
            }
            new playa;
            new money;
            playa = ReturnUser(tmp);
            tmp = strtok(cmdtext, idx);
            money = strval(tmp);
            if (PlayerInfo[playerid][pAdmin] >= 1337)
            {
                if(IsPlayerConnected(playa))
                {
                    if(playa != INVALID_PLAYER_ID)
                    {
                        SafeResetPlayerMoney(playa);
                        SafeGivePlayerMoney(playa, -money);
                        GetPlayerName(playa, giveplayer, sizeof(giveplayer));
                        GetPlayerName(playerid, sendername, sizeof(sendername));
                        format(string, 256, "AdmCmd: %s has Fined %s  [Amount: $%d]", sendername,giveplayer,money);
                        SendClientMessageToAll(COLOR_LIGHTRED,string);
                    }
                }
   }
            else
            {
                SendClientMessage(playerid, COLOR_GRAD1, "   you are not authorized to use that command!");
            }
        }
        return 1;
    }

[dannyk0ed]

That Ravens? Wonder why.

[Sid_Alexander]

Quote:

Originally Posted by dannyk0ed That Ravens? Wonder why.

, Nope, my complete script and commands are made by me,Lucky & Steve

We not COPY CATS!

[dannyk0ed]

I dont see what the problem is with Command :/ Sorry

[Sid_Alexander]

Quote:

Originally Posted by dannyk0ed I dont see what the problem is with Command :/ Sorry

The problem is, we cant directly provide the reason with the command

The command is right now in this method : USAGE: /fine [playerid] [amount]
We Want to provide the reason so people can see and wont ask example : USAGE: /fine [playerid] [amount] [reason]

Difference we want : USAGE: /fine [playerid] [amount] "TO GET CHANGED INTO" : USAGE: /fine [playerid] [amount] [reason]

[Steven82]

Well you copy the coding style of ravens then...

Anyways, what i think you should do, is switch to ZCMD and sscanf, since well they are better in many reasons. And they just overall destroy strcmp commands.

[Sid_Alexander]

Quote:

Originally Posted by Steven82 Well you copy the coding style of ravens then... Anyways, what i think you should do, is switch to ZCMD and sscanf, since well they are better in many reasons. And they just overall destroy strcmp commands.

About Ravens: We never saw that Script, And if you are talking about USAGE: Than we saw this in other roleplay servers, So we thought we'd keep the same

AGAIN WE NOT COPY CATS!

[Sid_Alexander]

C'mon guys! Someone convert it to DCMD

[Laronic]

Should work

Using sscanf and dcmd

pawn Код:

public OnPlayerCommandText(playerid, cmdtext[])
{
    dcmd(fine, 4, cmdtext);
    return 0;
}

dcmd_fine(playerid, params[])
{
    new targetid, amount, reason[128];
    if(sscanf(params, "uus[128]", targetid, amount, reason)) return SendClientMessage(playerid, 0xFFFFFFAA, "USAGE: /fine [playerid/name] [amount] [reason]");
    else if(PlayerInfo[playerid][pAdmin] != 1337) return SendClientMessage(playerid, COLOR_GRAD1, "   you are not authorized to use that command!");
    else if(!IsPlayerConnected(targetid)) return SendClientMessage(playerid, COLOR_GRAD1, "Error: That player is not connected");
    {
        GetPlayerName(targetid, targetname, sizeof(targetname));
        GetPlayerName(playerid, playername, sizeof(playername));
        format(string, sizeof(string), "AdmCmd: %s has Fined %s  [Amount: $%d] because: %s", playername, targetname, amount, reason);
        SendClientMessageToAll(COLOR_LIGHTRED,string);
        SafeGivePlayerMoney(targetid, -amount);
    }
    return 1;
}

[Sid_Alexander]

Maybe you should paste it without /pawn confusing paste it as /code