[FilterScript] Work Bus Driver
#5

I was thinking this was copied from somewhere at first but I couldn't find it so I'm pretty sure it's his script here is what you need to change.

This
Код:
new Text:Subtitles[MAX_PLAYERS_EX];
Belongs here
pawn Код:
enum PLAYER_INFO{
    Jobs
}
And it should be PlayerText in fact you don't even destroy the textdraws so eventually all the textdraws slots in your server would be filled!

Incorrect order here you don't need the vehicleid if they didn't even press the key!
pawn Код:
new vehicleid = GetPlayerVehicleID(playerid);
    if(PRESSED( KEY_YES )){
pawn Код:
if(PRESSED( KEY_YES )) {   
new vehicleid = GetPlayerVehicleID(playerid);
Of course the player is connected you don't need to check this
pawn Код:
if(IsPlayerConnected(playerid)){
Why oh why do you need this non-sense?
pawn Код:
if(IsPlayerInSphere(playerid,361.7982,-1527.4463,33.0697,5) ||
            IsPlayerInSphere(playerid,535.0513,-1262.2404,16.3687,5) ||
            IsPlayerInSphere(playerid,626.7946,-1636.8909,16.0026,5) ||
Make an enum and array to store all that data then loop through it you'll be able to shave off about 100 lines of code or so doing that furthermore you could then use a variable to keep track of the next checkpoint so you wouldn't need to keep doing a bunch of useless checks and you could use dynamic areas then you wouldn't even need a timer.
Reply


Messages In This Thread
Work Bus Driver - by Ludek - 11.03.2014, 11:45
Re : Work Bus Driver - by Abdeldu06 - 11.03.2014, 12:36
Re : Work Bus Driver - by Clad - 11.03.2014, 13:05
Re: Work Bus Driver - by Arjanz - 11.03.2014, 13:08
Re: Work Bus Driver - by Pottus - 11.03.2014, 13:08
Respuesta: Work Bus Driver - by OTACON - 11.03.2014, 13:32

Forum Jump:


Users browsing this thread: 1 Guest(s)