SA-MP Forums Archive
Weapon Menu problem - 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: Weapon Menu problem (/showthread.php?tid=360072)



Weapon Menu problem - RenSoprano - 16.07.2012

Hello guys I made Weapons Menu and I take some errors can you help me please

Here is my code

pawn Код:
case 9:
            {
                if(GetPlayerMoney(playerid) < 2000) /*HERE IS LINE 1433*/ return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a Rifle!");
                GivePlayerWeapon(playerid, 33, 110);
                GivePlayerMoney(playerid, -2000);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a Rifle for 2000$");
            }
And here is the error

Код:
C:\Users\Mario\Desktop\American Dream Roleplay\gamemodes\U1.pwn(1433) : error 079: inconsistent return types (array & non-array)



Re: Weapon Menu problem - StrangeLove - 16.07.2012

Show the line 1433


Re: Weapon Menu problem - RenSoprano - 16.07.2012

I showed it


Re: Weapon Menu problem - Captain_Mani - 16.07.2012

What's the line 1433? And show the lines from 1425 to 1440


Re: Weapon Menu problem - Frosty_LV - 16.07.2012

I guess there are some fails made in other lines. Show them please!


Re: Weapon Menu problem - RenSoprano - 16.07.2012

This is my dialog


pawn Код:
if(dialogid == 22 && response)
    {
        switch(listitem)
        {
            case 0:
            {
                ShowPlayerDialog(playerid, 23, DIALOG_STYLE_LIST, "Weapons Show", "9mm(100$)\nSDPistol(150$)\nDeagle(2000$)\nShotgun(250$)\nSpas-12(12500$\nAK-47(4500$)\nM4A1(5000$)\nMP5(450$)\nSniper Rifle(10000$\nRifle(2000$)\nKatana(75$)\nBaseBallBat(75$)\nHelath(Free)\nArmour(1000$)","Buy","Close");
            }
        }
    }
    if(dialogid == 23 && response)
    {
        switch(listitem)
        {
            case 0:
            {
                if(GetPlayerMoney(playerid) < 100) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a 9mm!");
                GivePlayerWeapon(playerid, 22, 150);
                GivePlayerMoney(playerid, -100);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a 9mm for 100$");
            }
            case 1:
            {
                if(GetPlayerMoney(playerid) < 150) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a SDPistol!");
                GivePlayerWeapon(playerid, 23, 130);
                GivePlayerMoney(playerid, -150);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a SDPistol for 150$");
            }
            case 2:
            {
                if(GetPlayerMoney(playerid) < 1300) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a Deagle!");
                GivePlayerWeapon(playerid, 24, 107);
                GivePlayerMoney(playerid, -1300);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a Deagle for 1300$");
            }
            case 3:
            {
                if(GetPlayerMoney(playerid) < 650) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a Shoutgun!");
                GivePlayerWeapon(playerid, 25, 100);
                GivePlayerMoney(playerid, -650);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a Shoutgun for 650$");
            }
            case 4:
            {
                if(GetPlayerMoney(playerid) < 5000) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a Spas-12!");
                GivePlayerWeapon(playerid, 27, 350);
                GivePlayerMoney(playerid, -5000);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a Spas-12 for 5000$");
            }
            case 5:
            {
                if(GetPlayerMoney(playerid) < 1700) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a AK-47!");
                GivePlayerWeapon(playerid, 30, 450);
                GivePlayerMoney(playerid, -1700);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a AK-47 for 1700$");
            }
            case 6:
            {
                if(GetPlayerMoney(playerid) < 5000) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a M4A1!");
                GivePlayerWeapon(playerid, 31, 450);
                GivePlayerMoney(playerid, -5000);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a M4A1 for 5000$");
            }
            case 7:
            {
                if(GetPlayerMoney(playerid) < 600) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a MP5!");
                GivePlayerWeapon(playerid, 29, 350);
                GivePlayerMoney(playerid, -600);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a MP5 for 600$");
            }
            case 8:
            {
                if(GetPlayerMoney(playerid) < 7000) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a Sniper Rifle!");
                GivePlayerWeapon(playerid, 34, 110);
                GivePlayerMoney(playerid, -7000);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a Sniper Rifle for 7000$");
            }
            case 9:
            {
                if(GetPlayerMoney(playerid) < 2000) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a Rifle!");
                GivePlayerWeapon(playerid, 33, 110);
                GivePlayerMoney(playerid, -2000);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a Rifle for 2000$");
            }
            case 10:
            {
                if(GetPlayerMoney(playerid) < 75) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a Katana!");
                GivePlayerWeapon(playerid, 8, 99999);
                GivePlayerMoney(playerid, -75);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a Katana for 75$");
            }
            case 11:
            {
                if(GetPlayerMoney(playerid) < 75) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a Baseballbat!");
                GivePlayerWeapon(playerid, 5, 99999);
                GivePlayerMoney(playerid, -75);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a Baseballbat for 75$");
            }
            case 12:
            {
                SetPlayerHealth(playerid, 100);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a First Aid Kid for free");
            }
            case 13:
            {
                if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a Protect!");
                SetPlayerArmour(playerid, 100);
                GivePlayerMoney(playerid, -1000);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a Protect for your body for 1000$");
            }
        }
    }



Re: Weapon Menu problem - Captain_Mani - 16.07.2012

Try this:

Код:
case 9:
            {
                if(GetPlayerMoney(playerid) < 2000) /*HERE IS LINE 1433*/
                SendClientMessage(playerid, COLOR_YELLOW, "You don't have enoung money to buy a Rifle!");
                else
                if(GetPlayerMoney(playerid) > 2000)
                GivePlayerWeapon(playerid, 33, 110);
                GivePlayerMoney(playerid, -2000);
                SendClientMessage(playerid, COLOR_YELLOW, "You have bought a Rifle for 2000$");
            }



Re: Weapon Menu problem - leonardo1434 - 16.07.2012

Dude , there's no erro in the line you pointed out, post here 2 - 3 lines before this one.


Re: Weapon Menu problem - StrangeLove - 16.07.2012

How about doing it something like this

pawn Код:
case 0:
                {
                    if(GetPlayerMoney(playerid) >= 1500)
                    {
                        // code here
                        SendClientMessage(playerid, COLOR_YELLOW, "You have bought something for $1500.");
                    } else {
                    SendClientMessage(playerid, COLOR_RED, "You do not have the enough money.");
                    }
             }



Re: Weapon Menu problem - RenSoprano - 16.07.2012

Nevermind thanks you guys I fixed it

CLOSED