How to make this into one? - Printable Version

How to make this into one? - 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: How to make this into one? (/showthread.php?tid=431263)

How to make this into one? - LeeXian99 - 17.04.2013

Hello guys, I want to do this on OnPlayerSpawn.

pawn Код:

if(GetPlayerScore(playerid) > 0) return SendClientMessage(playerid, 0xFF000000, "ERROR: You have not enough score to use this weaponset.");
        GivePlayerWeapon(playerid, 22, 99999);
        GivePlayerWeapon(playerid, 25, 99999);
        GivePlayerWeapon(playerid, 28, 99999);
        GivePlayerWeapon(playerid, 30, 99999);
        GivePlayerWeapon(playerid, 33, 99999);
        GivePlayerWeapon(playerid, 17, 5);
        
        if(GetPlayerScore(playerid) < 50) return SendClientMessage(playerid, 0xFF000000, "ERROR: You have not enough score to use this weaponset.");
        GivePlayerWeapon(playerid, 23, 99999);
        GivePlayerWeapon(playerid, 27, 99999);
        GivePlayerWeapon(playerid, 29, 99999);
        GivePlayerWeapon(playerid, 34, 99999);
        GivePlayerWeapon(playerid, 31, 99999);
        GivePlayerWeapon(playerid, 18, 5);
        
        if(GetPlayerScore(playerid) < 100) return SendClientMessage(playerid, 0xFFFFFFFF, "ERROR: You have not enough score to use this weaponset.");
        GivePlayerWeapon(playerid, 24, 99999);
        GivePlayerWeapon(playerid, 26, 99999);
        GivePlayerWeapon(playerid, 32, 99999);
        GivePlayerWeapon(playerid, 34, 99999);
        GivePlayerWeapon(playerid, 31, 99999);
        GivePlayerWeapon(playerid, 16, 5);

Re: How to make this into one? - [D]ry[D]esert - 17.04.2013

pawn Код:

stock GivePlayerWeapons(playerid) // change this to whatever you want.
    {
        if(GetPlayerScore(playerid) > 0) return SendClientMessage(playerid, 0xFF000000, "ERROR: You have not enough score to use this weaponset.");
        GivePlayerWeapon(playerid, 22, 99999);
        GivePlayerWeapon(playerid, 25, 99999);
        GivePlayerWeapon(playerid, 28, 99999);
        GivePlayerWeapon(playerid, 30, 99999);
        GivePlayerWeapon(playerid, 33, 99999);
        GivePlayerWeapon(playerid, 17, 5);

        if(GetPlayerScore(playerid) < 50) return SendClientMessage(playerid, 0xFF000000, "ERROR: You have not enough score to use this weaponset.");
        GivePlayerWeapon(playerid, 23, 99999);
        GivePlayerWeapon(playerid, 27, 99999);
        GivePlayerWeapon(playerid, 29, 99999);
        GivePlayerWeapon(playerid, 34, 99999);
        GivePlayerWeapon(playerid, 31, 99999);
        GivePlayerWeapon(playerid, 18, 5);

        if(GetPlayerScore(playerid) < 100) return SendClientMessage(playerid, 0xFFFFFFFF, "ERROR: You have not enough score to use this weaponset.");
        GivePlayerWeapon(playerid, 24, 99999);
        GivePlayerWeapon(playerid, 26, 99999);
        GivePlayerWeapon(playerid, 32, 99999);
        GivePlayerWeapon(playerid, 34, 99999);
        GivePlayerWeapon(playerid, 31, 99999);
        GivePlayerWeapon(playerid, 16, 5);
        return 1;
    }
public OnPlayerSpawn(playerid)
    {
    GivePlayerWeapons(playerid);
    return 1;
    }

Re: How to make this into one? - LeeXian99 - 17.04.2013

It doesn't work, it gives me three weaponset, luckily I tested with score, so it shows up the error message.

Re: How to make this into one? - zxc1 - 17.04.2013

Why is those msgs needed?
It will cause useless spam in my opinion, since you want to place it under OnPlayerSpawn.

pawn Код:

stock GivePlayerWeapons(playerid) // change this to whatever you want.
    {
        if(GetPlayerScore(playerid) > 0 && GetPlayerScore(playerid) < 50)
        {
        GivePlayerWeapon(playerid, 22, 99999);
        GivePlayerWeapon(playerid, 25, 99999);
        GivePlayerWeapon(playerid, 28, 99999);
        GivePlayerWeapon(playerid, 30, 99999);
        GivePlayerWeapon(playerid, 33, 99999);
        GivePlayerWeapon(playerid, 17, 5);
        return 1;
        }
        else if(GetPlayerScore(playerid) > 50 && GetPlayerScore(playerid) < 100)
        {
        GivePlayerWeapon(playerid, 23, 99999);
        GivePlayerWeapon(playerid, 27, 99999);
        GivePlayerWeapon(playerid, 29, 99999);
        GivePlayerWeapon(playerid, 34, 99999);
        GivePlayerWeapon(playerid, 31, 99999);
        GivePlayerWeapon(playerid, 18, 5);
        return 1;
        }
        else if(GetPlayerScore(playerid) > 100)
        {
        GivePlayerWeapon(playerid, 24, 99999);
        GivePlayerWeapon(playerid, 26, 99999);
        GivePlayerWeapon(playerid, 32, 99999);
        GivePlayerWeapon(playerid, 34, 99999);
        GivePlayerWeapon(playerid, 31, 99999);
        GivePlayerWeapon(playerid, 16, 5);
        return 1;
         }
    }