Quote:
Originally Posted by NaClchemistryK
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. |
I will edit it on Tuesday when im back to home.
Anyways thanks.
