Перебор машин (по типу foreach Player)

Доброго времени суток. На Pro-Pawn наткнулся на темку с перебором машин, и в теме нашел 2 кода:

1. Код с 3 страницы этой же темы

  Открыть/закрыть

PHP код:

---

#include <foreach> // by Y_Less 

new Iterator:valid_vehicles<MAX_VEHICLES>; 

#define forveh(%0) foreach (new %0 : valid_vehicles) 

stock Forveh_CreateVehicle(modelid, Float:x, Float:y, Float:z, Float:angle, color1, color2, respawn_delay) 
{ 
    new vehid = CreateVehicle(modelid, x, y, z,angle, color1, color2, respawn_delay); 
    Iter_Add(valid_vehicles, vehid); 
    return vehid; 
} 

#if defined _ALS_CreateVehicle 
    #undef CreateVehicle 
#else 
    #define _ALS_CreateVehicle 
#endif 
#define CreateVehicle Forveh_CreateVehicle 

stock Forveh_AddStaticVehicle(modelid, Float:spawn_x, Float:spawn_y, Float:spawn_z, Float:angle, color1, color2) 
{ 
    new vehid = AddStaticVehicle(modelid, spawn_x, spawn_y, spawn_z, angle, color1, color2); 
    Iter_Add(valid_vehicles, vehid); 
    return vehid; 
} 

#if defined _ALS_AddStaticVehicle 
    #undef AddStaticVehicle 
#else 
    #define _ALS_AddStaticVehicle 
#endif 
#define AddStaticVehicle Forveh_AddStaticVehicle 

stock Forveh_AddStaticVehicleEx(modelid, Float:spawn_x, Float:spawn_y, Float:spawn_z, Float:angle, color1, color2, respawn_delay)
{ 
    new vehid = AddStaticVehicleEx(modelid, spawn_x, spawn_y, spawn_z, angle, color1, color2,respawn_delay); 
    Iter_Add(valid_vehicles, vehid); 
    return vehid; 
} 

#if defined _ALS_AddStaticVehicleEx 
    #undef AddStaticVehicleEx 
#else 
    #define _ALS_AddStaticVehicleEx 
#endif 
#define AddStaticVehicleEx Forveh_AddStaticVehicleEx 

stock Forveh_DestroyVehicle(vehicleid) 
{ 
    if(Iter_Contains(valid_vehicles, vehicleid)) 
        Iter_Remove(valid_vehicles, vehicleid); 
    return DestroyVehicle(vehicleid); 
} 
#if defined _ALS_DestroyVehicle 
    #undef DestroyVehicle 
#else 
    #define _ALS_DestroyVehicle 
#endif 
#define DestroyVehicle Forveh_DestroyVehicle 


---

2. Код автора темы:

  Открыть/закрыть

PHP код:

---

new forveh_vehicles[MAX_VEHICLES];
new forveh_count = 0;

#if !defined IsValidVehicle
     native IsValidVehicle(vehicleid);
#endif

#define forveh(%0) for(new ouf, %0=forveh_vehicles[ouf]; ouf<forveh_count; %0=forveh_vehicles[++ouf])

stock Forveh_CreateVehicle(modelid, Float:x, Float:y, Float:z, Float:angle, color1, color2, respawn_delay)
{
    new vehid = CreateVehicle(modelid, x, y, z,angle, color1, color2, respawn_delay);
    forveh_vehicles[forveh_count++]=vehid;
    return vehid;
}
#if defined _ALS_CreateVehicle
     #undef CreateVehicle
#else
     #define _ALS_CreateVehicle
#endif
#define CreateVehicle Forveh_CreateVehicle

stock Forveh_AddStaticVehicle(modelid, Float:spawn_x, Float:spawn_y, Float:spawn_z, Float:angle, color1, color2)
{
    new vehid = AddStaticVehicle(modelid, spawn_x, spawn_y, spawn_z, angle, color1, color2);
    forveh_vehicles[forveh_count++]=vehid;
    return vehid;
}
#if defined _ALS_AddStaticVehicle
     #undef AddStaticVehicle
#else
     #define _ALS_AddStaticVehicle
#endif
#define AddStaticVehicle Forveh_AddStaticVehicle

stock Forveh_AddStaticVehicleEx(modelid, Float:spawn_x, Float:spawn_y, Float:spawn_z, Float:angle, color1, color2, respawn_delay)
{
    new vehid = AddStaticVehicleEx(modelid, spawn_x, spawn_y, spawn_z, angle, color1, color2,respawn_delay);
    forveh_vehicles[forveh_count++]=vehid;
    return vehid;
}
#if defined _ALS_AddStaticVehicleEx
    #undef AddStaticVehicleEx
#else
    #define _ALS_AddStaticVehicleEx
#endif
#define AddStaticVehicleEx Forveh_AddStaticVehicleEx

stock Forveh_DestroyVehicle(vehicleid)
{
    if(!IsValidVehicle(vehicleid)) return 0;
    for(new i=0;i<forveh_count;i++)
    {
        if(forveh_vehicles[i]==vehicleid)
        {
            forveh_vehicles[i]=forveh_vehicles[--forveh_count];
            break;
        }
    }
    return DestroyVehicle(vehicleid);
}
#if defined _ALS_DestroyVehicle
    #undef DestroyVehicle
#else
    #define _ALS_DestroyVehicle
#endif
#define DestroyVehicle Forveh_DestroyVehicle 


---

Вопрос состоит в следующем, стоит ли использовать перебор этих машин, вместо стандартного цикла, и если да, то какой код лучше, первый или второй?

Стоит.
Если используешь foreach, то первый

Могу предложить этот foreach: https://github.com/Open-GTO/foreach
Там уже встроена итерация по транспорту. Такой же, как и в первом варианте, только без колхоза с #define forveh(%0) foreach (new %0 : valid_vehicles) и с поддержкой FS.

Цитата:

---

Сообщение от ziggi

Могу предложить этот foreach: https://github.com/Open-GTO/foreach
Там уже встроена итерация по транспорту. Такой же, как и в первом варианте, только без колхоза с #define forveh(%0) foreach (new %0 : valid_vehicles) и с поддержкой FS.

---

Спасибо, взял эту версию foreach.. Даже смотрю foreach для актеров есть, правда не совсем понятно зачем)