[Cypress]

Quote:

Originally Posted by [uL]Pottus Well that will work but you don't want to keep calling the same function every iteration it should actually look like this. pawn Код: public OnPlayerEnterVehicle(playerid, vehicleid, ispassenger) {     new veh = GetPlayerVehicleID(playerid);     for(new i; i < sizeof(CGLSVehicles); i++)     {         if(veh== CGLSVehicles[i]) // LINE 656         {             //Do stuff, they're in one of your vehicle!             break; //Stop the loop, we don't need it anymore.         }     }     return 1; } I'd also make a dynamic car loader instead of sequentially incrementing the array during vehicle creation.

By the way, it's:

pawn Код:

public OnPlayerEnterVehicle(playerid, vehicleid, ispassenger)
{
    for(new i; i < sizeof(CGLSVehicles); i++)
    {
        if(vehicleid == CGLSVehicles[i]) // LINE 656
        {
            //Do stuff, they're in one of your vehicle!
            break; //Stop the loop, we don't need it anymore.
        }
    }
    return 1;
}

And, there is no difference as this function will be called along side again and again either if it's out the loop.