[Konstantinos]

I've never spawned many vehicles and it was something I never knew about valid vehicles 1-1999. I thought it was up to 2000.

But it can still be used as:

pawn Код:

new bool:VehiclesByVCommand[MAX_VEHICLES - 1];

// In your /v command use:
VehiclesByVCommand[vehicleid - 1] = true;

and the worst part of the above is:

pawn Код:

for (new vid; vid < MAX_VEHICLES; vid++)

Your server has 4 vehicles only (an example) and you loop 2000 times. It seems bad, isn't it? The best solution would be an iterator (foreach) for the created vehicles and it will loop through ONLY the valid (those which exist) vehicles.

I'm not saying that the code is bad, this is how it would be used by the most users but optimizations (even small ones) are always good - atleast for me.