1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help

Threaded Mode
1 error: Undefined symbol "idx"
thomaswilliams
Big Clucker
***
Posts: 151
Threads: 44
Joined: Apr 2013
Reputation: 0
#1

02.12.2013, 19:54
This is the coding

PHP код:
CMD:buybiz(playerid, params[])
{
    new string[128], done;
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command.");
    {
        for(new idx=1; idx<MAX_BIZ; idx++)
         if(IsPlayerInRangeOfPoint(playerid, 2, BizInfo[idx][bX], BizInfo[idx][bY], BizInfo[idx][bZ]))
        {
            if(!strcmp("The State", BizInfo[idx][bOwner], false))
            {
                if(PlayerInfo[playerid][pMoney] < BizInfo[idx][bPrice]) return SendClientMessage(playerid, COLOR_GREY, "You don't have enough money to buy this business.");
                GiveZaiatMoney(playerid, -BizInfo[idx][bPrice]);
                if(PlayerInfo[playerid][pBiz]) PlayerInfo[playerid][pVBiz] = idx;
                else PlayerInfo[playerid][pBiz] = idx;
                format(BizInfo[idx][bOwner], 32, "%s", RPNU(playerid));
                format(string, sizeof(string), ""COE"["CWE"%d, %s"COE"]\n"COE"Owner"CWE" %s\n"COE"Status"CWE" %s", idx, RBT(idx), BizInfo[idx][bOwner], RBS(idx));
                UpdateDynamic3DTextLabelText(BizInfo[idx][bText], COLOR_WHITE, string);
                SendClientMessage(playerid, COLOR_GREEN, " You have successfully bought a business.");
                SendClientMessage(playerid, COLOR_WHITE, " Type /bizhelp to view your business commands.");
                format(string, sizeof(string), "%s has bought business id %d.", RPN(playerid), idx);
                Log("logs/business.log", string);
                idx = MAX_BIZ;
                done = 1;
            }
            if(idx == MAX_BIZ-1 && !done)
            {
                SendClientMessage(playerid, COLOR_GREY, "This business is owned by someone else.");
            }
        }
        if(idx == MAX_BIZ-1 && !done)
        {
            SendClientMessage(playerid, COLOR_GREY, "You are not near a buyable business.");
        }
    }
    return 1;
} 
Find
Reply
Wizza
Big Clucker
***
Posts: 144
Threads: 20
Joined: Jul 2013
Reputation: 0
#2

02.12.2013, 20:00
What line?
Find
Reply
thomaswilliams
Big Clucker
***
Posts: 151
Threads: 44
Joined: Apr 2013
Reputation: 0
#3

02.12.2013, 20:01
somewhere here apparently:

PHP код:
   if(idx == MAX_BIZ-1 && !done)
            {
                SendClientMessage(playerid, COLOR_GREY, "This business is owned by someone else.");
            }
        }
        if(idx == MAX_BIZ-1 && !done)
        {
            SendClientMessage(playerid, COLOR_GREY, "You are not near a buyable business.");
        }
    } 
Find
Reply
Wizza
Big Clucker
***
Posts: 144
Threads: 20
Joined: Jul 2013
Reputation: 0
#4

02.12.2013, 20:05
Replace with this
pawn Код:
CMD:buybiz(playerid, params[])
{
    new string[128], done;
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command.");
    {
        for(new idx=1; idx<MAX_BIZ; idx++)
        if(IsPlayerInRangeOfPoint(playerid, 2, BizInfo[idx][bX], BizInfo[idx][bY], BizInfo[idx][bZ]))
        {
            if(!strcmp("The State", BizInfo[idx][bOwner], false))
            {
                if(PlayerInfo[playerid][pMoney] < BizInfo[idx][bPrice]) return SendClientMessage(playerid, COLOR_GREY, "You don't have enough money to buy this business.");
                GiveZaiatMoney(playerid, -BizInfo[idx][bPrice]);
                if(PlayerInfo[playerid][pBiz]) PlayerInfo[playerid][pVBiz] = idx;
                else PlayerInfo[playerid][pBiz] = idx;
                format(BizInfo[idx][bOwner], 32, "%s", RPNU(playerid));
                format(string, sizeof(string), ""COE"["CWE"%d, %s"COE"]\n"COE"Owner"CWE" %s\n"COE"Status"CWE" %s", idx, RBT(idx), BizInfo[idx][bOwner], RBS(idx));
                UpdateDynamic3DTextLabelText(BizInfo[idx][bText], COLOR_WHITE, string);
                SendClientMessage(playerid, COLOR_GREEN, " You have successfully bought a business.");
                SendClientMessage(playerid, COLOR_WHITE, " Type /bizhelp to view your business commands.");
                format(string, sizeof(string), "%s has bought business id %d.", RPN(playerid), idx);
                Log("logs/business.log", string);
                idx = MAX_BIZ;
                done = 1;
            }
            if(idx == MAX_BIZ-1 && !done)
            {
                SendClientMessage(playerid, COLOR_GREY, "This business is owned by someone else.");
            }
        }
        if(idx == MAX_BIZ-1 && !done)
        {
            SendClientMessage(playerid, COLOR_GREY, "You are not near a buyable business.");
        }
    }
    return 1;
}
Find
Reply
KesaSport
Little Clucker
**
Posts: 46
Threads: 17
Joined: May 2013
Reputation: 0
#5

02.12.2013, 20:07
Try:
pawn Код:
CMD:buybiz(playerid, params[]) 
{ 
    new string[128], done; 
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command."); 
    { 
        for(new idx=1; idx<MAX_BIZ; idx++)
        { 
            if(IsPlayerInRangeOfPoint(playerid, 2, BizInfo[idx][bX], BizInfo[idx][bY], BizInfo[idx][bZ])) 
            { 
                if(!strcmp("The State", BizInfo[idx][bOwner], false)) 
                { 
                    if(PlayerInfo[playerid][pMoney] < BizInfo[idx][bPrice]) return SendClientMessage(playerid, COLOR_GREY, "You don't have enough money to buy this business."); 
                    GiveZaiatMoney(playerid, -BizInfo[idx][bPrice]); 
                    if(PlayerInfo[playerid][pBiz]) PlayerInfo[playerid][pVBiz] = idx; 
                    else PlayerInfo[playerid][pBiz] = idx; 
                    format(BizInfo[idx][bOwner], 32, "%s", RPNU(playerid)); 
                    format(string, sizeof(string), ""COE"["CWE"%d, %s"COE"]\n"COE"Owner"CWE" %s\n"COE"Status"CWE" %s", idx, RBT(idx), BizInfo[idx][bOwner], RBS(idx)); 
                    UpdateDynamic3DTextLabelText(BizInfo[idx][bText], COLOR_WHITE, string); 
                    SendClientMessage(playerid, COLOR_GREEN, " You have successfully bought a business."); 
                    SendClientMessage(playerid, COLOR_WHITE, " Type /bizhelp to view your business commands."); 
                    format(string, sizeof(string), "%s has bought business id %d.", RPN(playerid), idx); 
                    Log("logs/business.log", string); 
                    idx = MAX_BIZ; 
                    done = 1; 
                } 
                if(idx == MAX_BIZ-1 && !done) 
                { 
                    SendClientMessage(playerid, COLOR_GREY, "This business is owned by someone else."); 
                }
            }
            if(idx == MAX_BIZ-1 && !done) 
            { 
                SendClientMessage(playerid, COLOR_GREY, "You are not near a buyable business."); 
            } 
        }
    } 
    return 1; 
}
Find
Reply
Wizza
Big Clucker
***
Posts: 144
Threads: 20
Joined: Jul 2013
Reputation: 0
#6

02.12.2013, 20:14
Quote:
Originally Posted by KesaSport
Посмотреть сообщение
Try:
pawn Код:
CMD:buybiz(playerid, params[]) 
{ 
    new string[128], done; 
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command."); 
    { 
        for(new idx=1; idx<MAX_BIZ; idx++)
        { 
            if(IsPlayerInRangeOfPoint(playerid, 2, BizInfo[idx][bX], BizInfo[idx][bY], BizInfo[idx][bZ])) 
            { 
                if(!strcmp("The State", BizInfo[idx][bOwner], false)) 
                { 
                    if(PlayerInfo[playerid][pMoney] < BizInfo[idx][bPrice]) return SendClientMessage(playerid, COLOR_GREY, "You don't have enough money to buy this business."); 
                    GiveZaiatMoney(playerid, -BizInfo[idx][bPrice]); 
                    if(PlayerInfo[playerid][pBiz]) PlayerInfo[playerid][pVBiz] = idx; 
                    else PlayerInfo[playerid][pBiz] = idx; 
                    format(BizInfo[idx][bOwner], 32, "%s", RPNU(playerid)); 
                    format(string, sizeof(string), ""COE"["CWE"%d, %s"COE"]\n"COE"Owner"CWE" %s\n"COE"Status"CWE" %s", idx, RBT(idx), BizInfo[idx][bOwner], RBS(idx)); 
                    UpdateDynamic3DTextLabelText(BizInfo[idx][bText], COLOR_WHITE, string); 
                    SendClientMessage(playerid, COLOR_GREEN, " You have successfully bought a business."); 
                    SendClientMessage(playerid, COLOR_WHITE, " Type /bizhelp to view your business commands."); 
                    format(string, sizeof(string), "%s has bought business id %d.", RPN(playerid), idx); 
                    Log("logs/business.log", string); 
                    idx = MAX_BIZ; 
                    done = 1; 
                } 
                if(idx == MAX_BIZ-1 && !done) 
                { 
                    SendClientMessage(playerid, COLOR_GREY, "This business is owned by someone else."); 
                }
            }
            if(idx == MAX_BIZ-1 && !done) 
            { 
                SendClientMessage(playerid, COLOR_GREY, "You are not near a buyable business."); 
            } 
        }
    } 
    return 1; 
}
Did you just copy my post lol?
Find
Reply
KesaSport
Little Clucker
**
Posts: 46
Threads: 17
Joined: May 2013
Reputation: 0
#7

02.12.2013, 20:16
Quote:
Originally Posted by Wizza
Посмотреть сообщение
Did you just copy my post lol?
Check again lol
Find
Reply
Threshold
High-roller
*****
Posts: 3,085
Threads: 37
Joined: Nov 2012
Reputation: 0
#8

03.12.2013, 00:10
This code should work, give it a quick try and respond with the results if possible.
pawn Код:
CMD:buybiz(playerid, params[])
{
    if(!IsPlayerLoggedIn(playerid)) return SendClientMessage(playerid, COLOR_GREY, "You need to login first before using any command.");
    for(new idx = 1; idx < MAX_BIZ; idx++)
    {
         if(IsPlayerInRangeOfPoint(playerid, 2, BizInfo[idx][bX], BizInfo[idx][bY], BizInfo[idx][bZ]))
         {
            if(strcmp("The State", BizInfo[idx][bOwner], false) == 0)
            {
                if(PlayerInfo[playerid][pMoney] < BizInfo[idx][bPrice]) return SendClientMessage(playerid, COLOR_GREY, "You don't have enough money to buy this business.");
                GiveZaiatMoney(playerid, -BizInfo[idx][bPrice]);
                if(PlayerInfo[playerid][pBiz]) PlayerInfo[playerid][pVBiz] = idx;
                else PlayerInfo[playerid][pBiz] = idx;
                format(BizInfo[idx][bOwner], 32, "%s", RPNU(playerid));
                new string[128];
                format(string, sizeof(string), ""COE"["CWE"%d, %s"COE"]\n"COE"Owner"CWE" %s\n"COE"Status"CWE" %s", idx, RBT(idx), BizInfo[idx][bOwner], RBS(idx));
                UpdateDynamic3DTextLabelText(BizInfo[idx][bText], COLOR_WHITE, string);
                SendClientMessage(playerid, COLOR_GREEN, " You have successfully bought a business.");
                SendClientMessage(playerid, COLOR_WHITE, " Type /bizhelp to view your business commands.");
                format(string, sizeof(string), "%s has bought business id %d.", RPN(playerid), idx);
                Log("logs/business.log", string);
                break;
            }
            if(idx == (MAX_BIZ - 1)) return SendClientMessage(playerid, COLOR_GREY, "This business is owned by someone else.");
        }
        if(idx == (MAX_BIZ - 1)) return SendClientMessage(playerid, COLOR_GREY, "You are not near a buyable business.");
    }
    return 1;
}
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


Users browsing this thread: 1 Guest(s)
  1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help