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

Threaded Mode
Job Problem
Theepiccoder
Little Clucker
**
Posts: 48
Threads: 6
Joined: Dec 2015
Reputation: 0
#1

03.07.2016, 14:21
When i start a job (go through checkpoints for money) and quit while it is active, then when i reconnect and i start the job again, i go through the first marker and get payed (I'm supposed to go through like 8, not 1). Then again i start the job, it still gives the money on the first checkpoint.

Defining the checkpoint checker (If a player goes through the first checkpoint, pokupipostu[playerid] = 2;)
Код:
new pokupipostu[MAX_PLAYERS];
When the player connects, pokupipostu[playerid] = 0;
Код:
public OnPlayerConnect(playerid)
{
	pokupipostu[playerid] = 0;
        return 1;
}
When the player disconnects, pokupipostu[playerid] = 0;
Код:
public OnPlayerDisconnect(playerid)
{
        pokupipostu[playerid] = 0;
        return 1;
}
If a player is in range of a checkpoint, pokupipostu[playerid] = some number;
Код:
if(IsPlayerInRangeOfPoint(playerid, 3.0, -2358.3467,803.3227,38.3803) && pokupipostu[playerid] == 1 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 2;
	    SetPlayerCheckpoint(playerid, -2224.3804,804.1550,49.2969, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2224.3804,804.1550,49.2969) && pokupipostu[playerid] == 2 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 3;
	    SetPlayerCheckpoint(playerid, -2151.5386,786.5508,69.4141, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2151.5386,786.5508,69.4141) && pokupipostu[playerid] == 3 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 4;
	    SetPlayerCheckpoint(playerid, -2151.6677,644.7795,52.3672, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2151.6677,644.7795,52.3672) && pokupipostu[playerid] == 4 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 5;
	    SetPlayerCheckpoint(playerid, -2185.7598,573.2388,35.0156, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2185.7598,573.2388,35.0156) && pokupipostu[playerid] == 5 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 6;
	    SetPlayerCheckpoint(playerid, -2246.9546,620.5477,45.3104, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2246.9546,620.5477,45.3104) && pokupipostu[playerid] == 6 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 7;
	    SetPlayerCheckpoint(playerid, -2321.4087,674.2640,42.0596, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2321.4087,674.2640,42.0596) && pokupipostu[playerid] == 7 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 8;
	    SetPlayerCheckpoint(playerid, -2468.9990,798.5485,35.1797, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2468.9990,798.5485,35.1797) && pokupipostu[playerid] == 8 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
        pokupipostu[playerid] = 0;
	    SendClientMessage(playerid, 0xFFFFFFFF, "{FF0000}[IN:RP] {FFFFFF}Uspesno ste pokupili postu, pa ste dobili $100 i 1 EXP.");
		PlayerInfo[playerid][pExp] += 1;
		GivePlayerMoney(playerid, 100);
		DisablePlayerCheckpoint(playerid);
	}
        return 1;
}
Command to start the job (first checks if the player has a job as 'POSTAR' in my language, then does the command).
Код:
CMD:pokupipostu(playerid, params[])
{
    if(PlayerInfo[playerid][pPosao] == 1)
	{
		SendClientMessage(playerid, 0xFFFFFFFF, "{FF0000}[IN:RP] {FFFFFF}Uspesno ste poceli da pokupljate postu.");
		SetPlayerCheckpoint(playerid, -2358.3467, 803.3227, 38.3803, 3.0);
		pokupipostu[playerid] = 1;
	}
	else
	{
	    SendClientMessage(playerid, 0xFFFFFFFF, "{FF0000}ERROR: {FFFFFF}Morate biti zaposleni kao POSTAR!");
	}
	return 1;
}
Find
Reply
Dayrion
High-roller
*****
Posts: 1,915
Threads: 64
Joined: Jan 2016
Reputation: 0
#2

03.07.2016, 14:28
You should process like this.
There a way more easy to do.
Add a variable in your like ActiveJob[playerid] = 1; when you do the command.
Use
PHP код:
public OnPlayerEnterCheckpoint(playerid)
if(Activejob[playerid] == 1)
{
...
Activejob[playerid] = 2;
... 
}
if(Activejob[playerid] == 2)
... 
You get it?
Find
Reply
Theepiccoder
Little Clucker
**
Posts: 48
Threads: 6
Joined: Dec 2015
Reputation: 0
#3

03.07.2016, 20:49
Quote:
Originally Posted by Dayrion
Посмотреть сообщение
You should process like this.
There a way more easy to do.
Add a variable in your like ActiveJob[playerid] = 1; when you do the command.
Use
PHP код:
public OnPlayerEnterCheckpoint(playerid)
if(Activejob[playerid] == 1)
{
...
Activejob[playerid] = 2;
... 
}
if(Activejob[playerid] == 2)
... 
You get it?
Umm, i think you misunderstood me. There are multiple jobs which require the player to be in a specific vehicle. Also, i already have Activejob defined. It's raznesipostu1[playerid], raznesipostu2[playerid] and pokupipostu[playerid].

Example in the code:
Код:
if(IsPlayerInRangeOfPoint(playerid, 3.0, -2541.7466, 896.7819, 64.9844) && raznesipostu1[playerid] == 7 && IsPlayerInVehicle(playerid, pv1) || IsPlayerInVehicle(playerid, pv2) || IsPlayerInVehicle(playerid, pv3) || IsPlayerInVehicle(playerid, pv4) || IsPlayerInVehicle(playerid, pv5))
	{
	    raznesipostu1[playerid] = 8;
	    SetPlayerCheckpoint(playerid, -2467.7964,797.8687,35.1719, 3.0);
	}
Find
Reply
Dayrion
High-roller
*****
Posts: 1,915
Threads: 64
Joined: Jan 2016
Reputation: 0
#4

03.07.2016, 21:58
You misunderstood me.
I talking about using public OnPlayerEnterCheckpoint(playerid) instead of IsPlayerInRangeOfPoint
This is more efficient.
Find
Reply
Theepiccoder
Little Clucker
**
Posts: 48
Threads: 6
Joined: Dec 2015
Reputation: 0
#5

04.07.2016, 13:44
What did you think was i using

Код:
public OnPlayerEnterCheckpoint(playerid)
{
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2358.3467,803.3227,38.3803) && pokupipostu[playerid] == 1 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 2;
	    SetPlayerCheckpoint(playerid, -2224.3804,804.1550,49.2969, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2224.3804,804.1550,49.2969) && pokupipostu[playerid] == 2 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 3;
	    SetPlayerCheckpoint(playerid, -2151.5386,786.5508,69.4141, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2151.5386,786.5508,69.4141) && pokupipostu[playerid] == 3 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 4;
	    SetPlayerCheckpoint(playerid, -2151.6677,644.7795,52.3672, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2151.6677,644.7795,52.3672) && pokupipostu[playerid] == 4 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 5;
	    SetPlayerCheckpoint(playerid, -2185.7598,573.2388,35.0156, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2185.7598,573.2388,35.0156) && pokupipostu[playerid] == 5 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 6;
	    SetPlayerCheckpoint(playerid, -2246.9546,620.5477,45.3104, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2246.9546,620.5477,45.3104) && pokupipostu[playerid] == 6 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 7;
	    SetPlayerCheckpoint(playerid, -2321.4087,674.2640,42.0596, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2321.4087,674.2640,42.0596) && pokupipostu[playerid] == 7 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
	    pokupipostu[playerid] = 8;
	    SetPlayerCheckpoint(playerid, -2468.9990,798.5485,35.1797, 3.0);
	}
	if(IsPlayerInRangeOfPoint(playerid, 3.0, -2468.9990,798.5485,35.1797) && pokupipostu[playerid] == 8 && IsPlayerInVehicle(playerid, pv6) || IsPlayerInVehicle(playerid, pv7))
	{
        pokupipostu[playerid] = 0;
	    SendClientMessage(playerid, 0xFFFFFFFF, "{FF0000}[IN:RP] {FFFFFF}Uspesno ste pokupili postu, pa ste dobili $100 i 1 EXP.");
		PlayerInfo[playerid][pExp] += 1;
		GivePlayerMoney(playerid, 100);
		DisablePlayerCheckpoint(playerid);
	}
	return 1;
}
Find
Reply
Dayrion
High-roller
*****
Posts: 1,915
Threads: 64
Joined: Jan 2016
Reputation: 0
#6

04.07.2016, 14:42
So why are you using IsPlayerInRangeOfPoint(playerid, 3.0, -2358.3467,803.3227,38.3803) ?
There is no sence. Remove it, you don't need it.
Find
Reply
Theepiccoder
Little Clucker
**
Posts: 48
Threads: 6
Joined: Dec 2015
Reputation: 0
#7

05.07.2016, 11:26
When a player enters a checkpoint, it checks if he entered the specific checkpoint, checks if raznesipostu[playerid] == 1 and checks if you're in the right vehicle.

This is pointless, i need help with fixing the problem. Not help with optimizing the code. Now, can you help me or not?

raznesipostu[playerid] function is like ActiveJob[playerid], just a different name.
Find
Reply
Mencent
High-roller
*****
Posts: 1,311
Threads: 11
Joined: Mar 2015
Reputation: 0
#8

05.07.2016, 11:41
Hello!

Try this:
PHP код:
public OnPlayerEnterCheckpoint(playerid)
{
    DisablePlayerCheckpoint(playerid);
    if(IsPlayerInVehicle(playerid,pv6) || IsPlayerInVehicle(playerid,pv7))
    {
        switch(pokupipostu[playerid])
        {
            case 1:{
                if(!IsPlayerInRangeOfPoint(playerid, 3.0, -2358.3467,803.3227,38.3803))return 1;
                pokupipostu[playerid] = 2;
                SetPlayerCheckpoint(playerid, -2224.3804,804.1550,49.2969, 3.0);
                return 1;
            }
            case 2:{
                if(!IsPlayerInRangeOfPoint(playerid, 3.0, -2224.3804,804.1550,49.2969))return 1;
                pokupipostu[playerid] = 3;
                SetPlayerCheckpoint(playerid, -2151.5386,786.5508,69.4141, 3.0);
                return 1;
            }
            case 3:{
                if(!IsPlayerInRangeOfPoint(playerid, 3.0, -2151.5386,786.5508,69.4141))return 1;
                pokupipostu[playerid] = 4;
                SetPlayerCheckpoint(playerid, -2151.6677,644.7795,52.3672, 3.0);
                return 1;
            }
            case 4:{
                if(!IsPlayerInRangeOfPoint(playerid, 3.0, -2151.6677,644.7795,52.3672))return 1;
                pokupipostu[playerid] = 5;
                SetPlayerCheckpoint(playerid, -2185.7598,573.2388,35.0156, 3.0);
                return 1;
            }
            case 5:{
                if(!IsPlayerInRangeOfPoint(playerid, 3.0, -2185.7598,573.2388,35.0156))return 1;
                pokupipostu[playerid] = 6;
                SetPlayerCheckpoint(playerid, -2246.9546,620.5477,45.3104, 3.0);
                return 1;
            }
            case 6:{
                if(!IsPlayerInRangeOfPoint(playerid, 3.0, -2246.9546,620.5477,45.3104))return 1;
                pokupipostu[playerid] = 7;
                SetPlayerCheckpoint(playerid, -2321.4087,674.2640,42.0596, 3.0);
                return 1;
            }
            case 7:{
                if(!IsPlayerInRangeOfPoint(playerid, 3.0, -2321.4087,674.2640,42.0596))return 1;
                pokupipostu[playerid] = 8;
                SetPlayerCheckpoint(playerid, -2468.9990,798.5485,35.1797, 3.0);
                return 1;
            }
            case 8:{
                if(!IsPlayerInRangeOfPoint(playerid, 3.0, -2468.9990,798.5485,35.1797))return 1;
                pokupipostu[playerid] = 0;
                SendClientMessage(playerid, 0xFFFFFFFF, "{FF0000}[IN:RP] {FFFFFF}Uspesno ste pokupili postu, pa ste dobili $100 i 1 EXP.");
                PlayerInfo[playerid][pExp] += 1;
                GivePlayerMoney(playerid, 100);
                DisablePlayerCheckpoint(playerid);
                return 1;
            }
        }
    }
    return 1;
} 
I changed your if clause to a switch clause.
Find
Reply
Theepiccoder
Little Clucker
**
Posts: 48
Threads: 6
Joined: Dec 2015
Reputation: 0
#9

05.07.2016, 13:16
Код:
if(!IsPlayerInRangeOfPoint(playerid, 3.0, -2151.5386,786.5508,69.4141))return 1;
C:\Users\notshowing\Desktop\notshowing\gamemodes\n otshowing.pwn(476) : error 079: inconsistent return types (array & non-array)
Find
Reply
Mencent
High-roller
*****
Posts: 1,311
Threads: 11
Joined: Mar 2015
Reputation: 0
#10

05.07.2016, 13:25
I can compile this perfectly. Can you show us the lines around 476 (476+10 and 476-10)
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