Weapon Menu problem
#1

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)
Reply
#2

Show the line 1433
Reply
#3

I showed it
Reply
#4

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

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

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$");
            }
        }
    }
Reply
#7

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$");
            }
Reply
#8

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

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.");
                    }
             }
Reply
#10

Nevermind thanks you guys I fixed it

CLOSED
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)