[Tutorial] /saveskin & /delskin
#11

your script should work, but I have a little advice for you.
Instead of making so many if and elses, you can return the client messages. That will make your script shorter.
example with /saveskin...
pawn Code:
CMD:saveskin(playerid,params[])
{
    new skinid = GetPlayerSkin(playerid);
    if(SkinSaving[playerid] == 1) return SendClientMessage(playerid,COLOR_GREY,"You already saved a skin before! to delete it you can use /delskin !");//this is what I am talking about. Simply return it to save some typing mayhem... Esecially in long scripts.
    else if(SkinSaving[playerid] == 0)
    {
    PlayerInfo[playerid][eSkin] = skinid;
    SkinSaving[playerid] = 1;
    SendClientMessage(playerid,COLOR_GREY,"You have saved your skin, Every time you spawn your skin will be %d",skinid);
    }
    return 1;
}
the same goes for /delskin.
And you haven't added the stuff Rittik said, skin id 74 is really a invalid skin id. You shold add it in your tut.
Otherwise great job.
Reply


Messages In This Thread
/saveskin & /delskin - by Ghazal - 22.05.2014, 16:48
Re: /saveskin & /delskin - by Youssef214 - 22.05.2014, 16:54
Re: /saveskin & /delskin - by Ghazal - 22.05.2014, 16:57
Re: /saveskin & /delskin - by iAnonymous - 22.05.2014, 18:26
Re: /saveskin & /delskin - by Ghazal - 22.05.2014, 19:44
Re: /saveskin & /delskin - by Rittik - 23.05.2014, 04:45
Re: /saveskin & /delskin - by Ghazal - 23.05.2014, 18:57
Re: /saveskin & /delskin - by Alex Magaсa - 23.05.2014, 19:02
Re: /saveskin & /delskin - by jihadmeneer - 23.05.2014, 19:05
Re: /saveskin & /delskin - by Ghazal - 23.05.2014, 21:14
Re: /saveskin & /delskin - by NaClchemistryK - 24.05.2014, 08:07
Re: /saveskin & /delskin - by Ghazal - 24.05.2014, 13:01
Re: /saveskin & /delskin - by Dangjai - 26.05.2014, 21:51
Re: /saveskin & /delskin - by Ghazal - 27.05.2014, 08:49

Forum Jump:


Users browsing this thread: 1 Guest(s)