Fragen:
1. Warum kein bollean sondern eine resourcen fressende Var die die niemals einen sohohen wert erreicht
2. Warum immer ein neuer Funktionsaufruf in den Variablen?
Eine Variable erstellen das das ergebnis drin speichern und so aufrufen.
3. Warum keine Arrays?
BSP:
new veh = GetPlayerVehicleID(playerid);
light1[veh] = CreateObject(19154,-0.0089111328, 0.273193359, 1.4037838, 0.0, 0.0, 96.0);
Außerdem ist die stelle mit OnVehicleSpawn überhaupt nicht durchdacht.
Von was willst du denn die VehicleID auslesen? von der Spielerid 65565? Die ist niemals connected und wird immer failen.
In ObVehicleSpawn gibt es überhaut keinen playerid Paramter.
Statt der sinnlosen Funktion GetPlayerVehicleID einfach vehicleid einsetzen, vehicleid ist da ja schon gegeben