command problem help rep+ - Printable Version

command problem help rep+ - 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: command problem help rep+ (/showthread.php?tid=568600)

command problem help rep+ - Mijata - 23.03.2015

when player again type /aduty and when admin turn off admin duty he have cj skin and white color

Код:

CMD:aduty(playerid, params[])
{
    if(pInfo[playerid][pLevel] >= 1)
    {
        new skin[MAX_PLAYERS];
        if(OnDuty[playerid] == 0)
        {
            skin[playerid] = GetPlayerSkin(playerid);
            SetPlayerSkin(playerid, 217);
            SetPlayerHealth(playerid, 100000);
            SetPlayerArmour(playerid, 100000);
            SetPlayerColor(playerid, 0x01FCFFFF);
            OnDuty[playerid] = 1;
        }
        else if(OnDuty[playerid] == 1)
        {
            SetPlayerSkin(playerid, skin[playerid]);
            SetPlayerHealth(playerid, 100);
            SetPlayerArmour(playerid, 0);
            SetPlayerColor(playerid, 0xFFFFFFFF);
            OnDuty[playerid] = 0;
        }
    }
    else
    {
        SendClientMessage(playerid, red, "You are not administrator.");
    }
    return true;
}

Re: command problem help rep+ - arjanforgames - 23.03.2015

Of course he will get a white color, because you do SetPlayerColor to white!

Re: command problem help rep+ - Mijata - 23.03.2015

i want color like before and skin like before

Re: command problem help rep+ - CalvinC - 23.03.2015

The skin array should be global, otherwise it gets set as 0 each time you use the command, and won't remember what it was set to.
And for the white color, you should store the players color in a global array as well, like the skin.

Re: command problem help rep+ - Mijata - 23.03.2015

i can't understand give example

Re: command problem help rep+ - arjanforgames - 23.03.2015

Quote:

Originally Posted by Mijata

i can't understand give example

No, we are not going to give you the code. Calvin explained it pretty clear so just go ahead and change the things he said.

Re: command problem help rep+ - CalvinC - 23.03.2015

If you don't understand basic scripting, you should go watch tutorials, and try doing some easier things instead of asking for people to do it for you here.

Re: command problem help rep+ - Aly - 23.03.2015

You can declare that skin variable exactly where you declared "pLevel".Let's say you call it "pSkin" and you're gonna use it like "pInfo[playerid][pSkin]".

Re: command problem help rep+ - MrCallum - 23.03.2015

Код:

#define COLOR_WHITE 0xFFFFFFAA //- If you do not have COLOR_WHITE Defined, or use -1

CMD:aduty(playerid, params[])
{
    if(pInfo[playerid][pLevel] >= 1)
    {
        new skin[MAX_PLAYERS];
        if(OnDuty[playerid] == 0)
        {
            skin[playerid] = GetPlayerSkin(playerid);
            SetPlayerSkin(playerid, 217);
            SetPlayerHealth(playerid, 100000);
            SetPlayerArmour(playerid, 100000);
            SetPlayerColor(playerid, 0x01FCFFFF);
            OnDuty[playerid] = 1;
        }
        else if(OnDuty[playerid] == 1)
        {
            SetPlayerSkin(playerid, GetPlayerSkin[playerid]);
            SetPlayerHealth(playerid, 100);
            SetPlayerArmour(playerid, 0);
            SetPlayerColor(playerid, 0xFFFFFFAA); // OR SetPlayerColor(playerid, -1);
            OnDuty[playerid] = 0;
        }
    }
    else
    {
        SendClientMessage(playerid, red, "You are not administrator.");
    }
    return true;
}

Re: command problem help rep+ - CalvinC - 23.03.2015

Quote:

Originally Posted by MrCallum

Код:

#define COLOR_WHITE 0xFFFFFFAA //- If you do not have COLOR_WHITE Defined, or use -1

CMD:aduty(playerid, params[])
{
    if(pInfo[playerid][pLevel] >= 1)
    {
        new skin[MAX_PLAYERS];
        if(OnDuty[playerid] == 0)
        {
            skin[playerid] = GetPlayerSkin(playerid);
            SetPlayerSkin(playerid, 217);
            SetPlayerHealth(playerid, 100000);
            SetPlayerArmour(playerid, 100000);
            SetPlayerColor(playerid, 0x01FCFFFF);
            OnDuty[playerid] = 1;
        }
        else if(OnDuty[playerid] == 1)
        {
            SetPlayerSkin(playerid, GetPlayerSkin[playerid]);
            SetPlayerHealth(playerid, 100);
            SetPlayerArmour(playerid, 0);
            SetPlayerColor(playerid, 0xFFFFFFAA); // OR SetPlayerColor(playerid, -1);
            OnDuty[playerid] = 0;
        }
    }
    else
    {
        SendClientMessage(playerid, red, "You are not administrator.");
    }
    return true;
}

How's that even supposed to work?
GetPlayerSkin is a function, so use parenthesis, not brackets.
And even doing GetPlayerSkin(playerid) will just set it to the skin he currently has.
As i said, you need to use a global array to store the data in, not a local one.