[Overhaul]

Quote:

Originally Posted by Luis- Also, https://sampforum.blast.hk/showthread.php?tid=570635

^ That, and if I may be honest, you haven't explained much either.

Just for the sake of convenience, you should rename the IsEngineOn variable to something like: 'EngineStatus' and declare it as a bool:

PHP код:

new bool:EngineStatus[MAX_PLAYERS]; 


By declaring it as a boolean, you will no longer have to use the '==' in if statements that use this variable.

PHP код:

if(EngineStatus[playerid]) 


which is equal to

PHP код:

if(EngineStatus[playerid] == 1) 


and

PHP код:

if(!EngineStatus[playerid]) 


which is equal to

PHP код:

if(EngineStatus[playerid] == 0) 


That is just for the sake of convenience though.

You declare all the variables that will be put to use in the function's body. Okay, but why? I know why but I am 100 percent sure that other people that don't know will question that. In that case I prefer to go to the wiki and figure it out myself or seek a better tutorial. I am talking about the variables used for the SetVehicleParamsEx function.