Why doesnt this work? - Printable Version

Why doesnt this work? - 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)
+---- Forum: Help Archive (https://sampforum.blast.hk/forumdisplay.php?fid=89)
+---- Thread: Why doesnt this work? (/showthread.php?tid=186132)

Why doesnt this work? - BigAl - 27.10.2010

public OnPlayerStateChange(playerid, newstate, oldstate)
{
new vehicleid = GetPlayerVehicleID(playerid);

if(newstate == PLAYER_STATE_DRIVER)
{
AddVehicleComponent(vehicleid, 1010);
NOS = SetTimerEx("NOSGiver", 1000, 1, "i", vehicleid);
}
else if(oldstate == PLAYER_STATE_DRIVER)
{
KillTimer(NOS);
}
return 1;
}

forward NOSGiver(vehicleid);
public NOSGiver(vehicleid)
{
AddVehicleComponent(vehicleid, 1010);
return 1;
}

it doesnt give me errors but nos doesnt work in game!

Re: Why doesnt this work? - pater - 27.10.2010

pawn Код:

NOS = SetTimerEx("NOSGiver", 1000, 1, "i", playerid);

You had vehicleid for timerex, which had to be playerid

also change NOS into NOS[MAX_PLAYERS]
than use

pawn Код:

NOS[playerid] = SetTimerEx("NOSGiver", 1000, 1, "i", playerid);
// and
KillTimer(NOS[playerid]);

otherwise you will disable the timer for all players when one player exits the vehicle..

good luck.