foreach от ziggi

**123** · 21.04.2020, 03:41

Использую форк foreach от ziggi. В нем есть интегратор машин, так вот, получаю следующие ошибки:

PHP код:


[17:09:30] Run time error 4: "Array index out of bounds" 
[17:09:30]  Attempted to read/write array element at index 2001 in array of size 2000 
[17:09:30] AMX backtrace: 
[17:09:30] #0 004ade54 in main (params[]=@00000028 "", 9092328) at C:\GRP\gamemodes\grp.pwn:34171 
[20:43:11] Run time error 4: "Array index out of bounds" 
[20:43:11]  Attempted to read/write array element at index 2001 in array of size 2000 
[20:43:11] AMX backtrace: 
[20:43:11] #0 004ade54 in main (params[]=@00000014 "", 9092328) at C:\GRP\gamemodes\grp.pwn:34171

Привожу ниже код команды и функции, которые используются:

PHP код:


CMD:spveh(playerid, params[]) 
{ 
    if(AdminInfo[playerid][aLevel] < 2 || !AdminInfo[playerid][aLogin]) 
        return true; 
 
    extract params -> new Float:rad = 200.0; else 
        return SendClientMessage(playerid, -1, !"Введите: /spveh [радиус]"); 
 
    if(rad < 1.0 || rad > 200.0) 
        return SendClientMessage(playerid, COLOR_GREY, !"Радиус не может быть меньше 1.00 выше 200.0 (области видимости)."); 
 
    new 
        sp_car; 
 
    foreach (new vehicleid : Vehicle) { 
 
        if(IsPlayerInRangeOfPoint(playerid, rad, VehicleInfo[vehicleid][vX], VehicleInfo[vehicleid][vY], VehicleInfo[vehicleid][vZ]) && !IsVehicleOccupied(vehicleid)) // ошибки 
        { 
            SetVehicleToRespawn(vehicleid); 
            sp_car ++; 
        } 
    } 
 
    format(stringer1, sizeof(stringer1), "Администратор %s (%d) зареспавнил весь транспорт в радиусе %f м (%d машин)", PlayerInfo[playerid][pNames], playerid, rad, sp_car); 
    return SendAdminMessage(COLOR_LIGHTRED,stringer1); 
} 
 
stock IsVehicleOccupied(vehicleid) 
{ 
    foreach(new i : Player) 
        if(IsPlayerInVehicle(i, vehicleid)) return 1; 
 
    return 0; 
}

Как видно, ошибки нет. До этого использовался оригинальный foreach и собственный интегратор машин, таких проблем не было. Ошибка у ziggi? Пишу сюда, потому что его репозиторий мертв.

**Daniel_Cortez** · 21.04.2020, 16:37

Я так понимаю, это происходит только когда на сервере заспавнены все 1999 машин? (2000 нельзя из-за криворукости разработчиков SA-MP)
И если код внутри цикла foreach закомментировать, ошибки всё так же останутся?

**123** · 22.04.2020, 05:00

Нет. Только что проверил, создал все 1999 машин и испытывал это. Безрезультатно. Но тем не менее изо дня в день я наблюдаю эти ошибки в логах. Да и на моем сервере не может быть такого количества транспорта, даже в часы пик.

**Daniel_Cortez** · 22.04.2020, 09:56

Немного пальцем в небо, но всё же, попробуйте в инклуде foreach.inc найти такую строку:

new Iterator:Vehicle<MAX_VEHICLES>;

и заменить "MAX_VEHICLES" на "MAX_VEHICLES + 1". И, возможно, то же самое понадобится проделать с итератором PlayerVehiclesStream, если он тоже используется.
Я точно помню, что сообщал ziggi об этой проблеме 2 года назад. Как видно, воз и ныне там...

UPD: Порылся в архиве ЛС; я тогда сообщил ziggi не совсем об этой, но о похожей проблеме: ID транспорта начинаются не с 0, а с 1, и поэтому при попытке добавления в итератор последний возможный ID (2000) может быть отвергнут, и предложил увеличить размер итератора Vehicle на 1, на что он и ответил, что да, смысл в этом есть (не уверен, насколько корректно будет публиковать личную переписку, поэтому вот такой вот пересказ). Но, как оказалось, больше 1999 машин создать нельзя и получить транспорт с ID 2000 невозможно - наверняка поэтому он и оставил размеры итераторов без изменений. Возможно, теперь есть новая причина их увеличить.

**Kovshevoy** · 22.04.2020, 11:02

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

Немного пальцем в небо, но всё же, попробуйте в инклуде foreach.inc найти такую строку:

new Iterator:Vehicle<MAX_VEHICLES>;

и заменить "MAX_VEHICLES" на "MAX_VEHICLES + 1". И, возможно, то же самое понадобится проделать с итератором PlayerVehiclesStream, если он тоже используется.
Я точно помню, что сообщал ziggi об этой проблеме 2 года назад. Как видно, воз и ныне там...

UPD: Порылся в архиве ЛС; я тогда сообщил ziggi не совсем об этой, но о похожей проблеме: ID транспорта начинаются не с 0, а с 1, и поэтому при попытке добавления в итератор последний возможный ID (2000) может быть отвергнут, и предложил увеличить размер итератора Vehicle на 1, на что он и ответил, что да, смысл в этом есть (не уверен, насколько корректно будет публиковать личную переписку, поэтому вот такой вот пересказ). Но, как оказалось, больше 1999 машин создать нельзя и получить транспорт с ID 2000 невозможно - наверняка поэтому он и оставил размеры итераторов без изменений. Возможно, теперь есть новая причина их увеличить.

Весьма интересно
У меня кстати тоже, при использовании foreach от ziggi такая фигня с машинами. Правда я думал, что это ошибка из-за своего кода.

**Daniel_Cortez** · 23.04.2020, 16:46

Вопрос к ТС: после предложенного парой постов выше исправления проблема решена?

**123** · 23.04.2020, 17:04

Только загрузил предложенное Вами исправления, буду наблюдать. О результатах я отпишусь.

**Shaolinka** · 23.04.2020, 23:26

Не использую форк от Зигги, однако использую следующее:

PHP код:


// раздефайним константу MAX_VEHICLES, установив ей значение 1999, потому что идёт от 1 до 1999
#if defined MAX_VEHICLES
#undef MAX_VEHICLES
#define MAX_VEHICLES 1999
#else 
#define MAX_VEHICLES 1999
#endif

// создадим итератор для всего транспорта на сервере, как-раз тут нам надо было бы отнимать единицу, однако проще установить новое значение константе
new Iterator: Vehicle<MAX_VEHICLES>;

// рекомендую перехватить нативные функции для работы с транспортом, а именно: (CreateVehicle, AddStaticVehicleEx, DestroyVehicle).
stock CreateVehicleEx(vehicletype, Float: x, Float: y, Float: z, Float: rotation, color1, color2, respawn_delay, addsiren)
{
      new veh_id = CreateVehicle(vehicletype, Float: x, Float: y, Float: z, Float: rotation, color1, color2, respawn_delay, addsiren);
      Iter_Add(Vehicle, veh_id);

      return veh_id;
}
#if defined _ALS_CreateVehicle
     #undef CreateVehicle
#else
     #define _ALS_CreateVehicle
#endif
#define CreateVehicle CreateVehicleEx

stock ac_AddStaticVehicleEx(modelid,  Float: spawn_x, Float: spawn_y, Float: spawn_z, Float:z_angle, color1, color2, respawn_delay, addsiren)
{
      new veh_id = AddStaticVehicleEx(vehicletype, spawn_x, spawn_y, spawn_z, z_angle, color1, color2, respawn_delay, addsiren);
      Iter_Add(Vehicle, veh_id);

      return veh_id;
}
#if defined _ALS_AddStaticVehicleEx
     #undef AddStaticVehicleEx
#else
     #define _ALS_AddStaticVehicleEx
#endif
#define AddStaticVehicleEx ac_AddStaticVehicleEx

stock DestroyVehicleEx(vehicleid)
{
      if(!Iter_Contains(Vehicle, vehicleid))
         return ;

      return Iter_Remove(Vehicle, vehicleid);
}
#if defined _ALS_DestroyVehicle
     #undef DestroyVehicle
#else
     #define _ALS_DestroyVehicle
#endif
#define DestroyVehicle DestroyVehicleEx

// ну и мой вариант команды
CMD:spveh(playerid, params[])
{
    if(AdminInfo[playerid][aLevel] < 2 || !AdminInfo[playerid][aLogin])
        return ;

    extract params -> new Float: rad = 200.0; else
          return SendClientMessage(playerid, -1, "Введите: /spveh [радиус]");

    if(!(1.0 <= rad <= 200.0))
          return SendClientMessage(playerid, COLOR_GREY, "Радиус не может быть меньше 1.00 выше 200.0 (области видимости).");

    new count = MAX_VEHICLES;

    foreach (new vehicleid : Vehicle) 
    {
        if(IsPlayerInRangeOfPoint(playerid, rad, VehicleInfo[vehicleid][vX], VehicleInfo[vehicleid][vY], VehicleInfo[vehicleid][vZ])
            continue;
        
        if(IsVehicleOccupied(vehicleid))
            continue;

       SetVehicleToRespawn(vehicleid);

       count --;
    }

    new const fmt_str[] = "Администратор %s (%d) зареспавнил весь транспорт в радиусе %f м (%d машин)";
    
    new result_str[(((sizeof fmt_str) + ((- 2 + MAX_PLAYER_NAME) + (- 2 + 3) + (- 2 + 5) + (- 2 + 4)) + 1))];

    format
    (
           result_str, sizeof result_str,
           fmt_str,
           PlayerInfo[playerid][pNames], 
           playerid, 
           rad, 
           (MAX_VEHICLES - count)
    );
           
    return SendAdminMessage(COLOR_LIGHTRED, result_str);
}

stock IsVehicleOccupied(vehicleid)
{
    foreach(new playerid : Player)
    {
        if(!IsPlayerInVehicle(i, vehicleid))
             continue;

        return 1;
     }

     return ;
}

Собсна как-то так

**Nexius_Tailer** · 24.04.2020, 02:09

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

Не использую форк от Зигги, однако использую следующее:

PHP код:


// раздефайним константу MAX_VEHICLES, установив ей значение 1999, потому что идёт от 1 до 1999
#if defined MAX_VEHICLES
#undef MAX_VEHICLES
#define MAX_VEHICLES 1999
#else 
#define MAX_VEHICLES 1999
#endif

// создадим итератор для всего транспорта на сервере, как-раз тут нам надо было бы отнимать единицу, однако проще установить новое значение константе
new Iterator: Vehicle<MAX_VEHICLES>;

// рекомендую перехватить нативные функции для работы с транспортом, а именно: (CreateVehicle, AddStaticVehicleEx, DestroyVehicle).
stock CreateVehicleEx(vehicletype, Float: x, Float: y, Float: z, Float: rotation, color1, color2, respawn_delay, addsiren)
{
      new veh_id = CreateVehicle(vehicletype, Float: x, Float: y, Float: z, Float: rotation, color1, color2, respawn_delay, addsiren);
      Iter_Add(Vehicle, veh_id);

      return veh_id;
}
#if defined _ALS_CreateVehicle
     #undef CreateVehicle
#else
     #define _ALS_CreateVehicle
#endif
#define CreateVehicle CreateVehicleEx

stock ac_AddStaticVehicleEx(modelid,  Float: spawn_x, Float: spawn_y, Float: spawn_z, Float:z_angle, color1, color2, respawn_delay, addsiren)
{
      new veh_id = AddStaticVehicleEx(vehicletype, spawn_x, spawn_y, spawn_z, z_angle, color1, color2, respawn_delay, addsiren);
      Iter_Add(Vehicle, veh_id);

      return veh_id;
}
#if defined _ALS_AddStaticVehicleEx
     #undef AddStaticVehicleEx
#else
     #define _ALS_AddStaticVehicleEx
#endif
#define AddStaticVehicleEx ac_AddStaticVehicleEx

stock DestroyVehicleEx(vehicleid)
{
      if(!Iter_Contains(Vehicle, vehicleid))
         return ;

      return Iter_Remove(Vehicle, vehicleid);
}
#if defined _ALS_DestroyVehicle
     #undef DestroyVehicle
#else
     #define _ALS_DestroyVehicle
#endif
#define DestroyVehicle DestroyVehicleEx

// ну и мой вариант команды
CMD:spveh(playerid, params[])
{
    if(AdminInfo[playerid][aLevel] < 2 || !AdminInfo[playerid][aLogin])
        return ;

    extract params -> new Float: rad = 200.0; else
          return SendClientMessage(playerid, -1, "Введите: /spveh [радиус]");

    if(!(1.0 <= rad <= 200.0))
          return SendClientMessage(playerid, COLOR_GREY, "Радиус не может быть меньше 1.00 выше 200.0 (области видимости).");

    new count = MAX_VEHICLES;

    foreach (new vehicleid : Vehicle) 
    {
        if(IsPlayerInRangeOfPoint(playerid, rad, VehicleInfo[vehicleid][vX], VehicleInfo[vehicleid][vY], VehicleInfo[vehicleid][vZ])
            continue;
        
        if(IsVehicleOccupied(vehicleid))
            continue;

       SetVehicleToRespawn(vehicleid);

       count --;
    }

    new const fmt_str[] = "Администратор %s (%d) зареспавнил весь транспорт в радиусе %f м (%d машин)";
    
    new result_str[(((sizeof fmt_str) + ((- 2 + MAX_PLAYER_NAME) + (- 2 + 3) + (- 2 + 5) + (- 2 + 4)) + 1))];

    format
    (
           result_str, sizeof result_str,
           fmt_str,
           PlayerInfo[playerid][pNames], 
           playerid, 
           rad, 
           (MAX_VEHICLES - count)
    );
           
    return SendAdminMessage(COLOR_LIGHTRED, result_str);
}

stock IsVehicleOccupied(vehicleid)
{
    foreach(new playerid : Player)
    {
        if(!IsPlayerInVehicle(i, vehicleid))
             continue;

        return 1;
     }

     return ;
}

Собсна как-то так

Редефайн MAX_VEHICLES можно сделать до подключения foreach не переизобретая итераторы транспорта, эффект во всяком случае тот же должен быть.

**123** · 24.04.2020, 06:07

PHP код:


[15:40:32] Run time error 4: "Array index out of bounds"
[15:40:32]  Attempted to read/write array element at index 2002 in array of size 2002
[15:40:32] AMX backtrace:
[15:40:32] #0 004bf650 in main (params[]=@00000019 !"", 9113472) at C:\GRP\gamemodes\grp.pwn:34453
[15:40:46] Run time error 4: "Array index out of bounds"
[15:40:46]  Attempted to read/write array element at index 2002 in array of size 2002
[15:40:46] AMX backtrace:
[15:40:46] #0 004bf650 in main (params[]=@00000019 !"", 9113472) at C:\GRP\gamemodes\grp.pwn:34453
[15:40:52] Run time error 4: "Array index out of bounds"
[15:40:52]  Attempted to read/write array element at index 2002 in array of size 2002

Собственно увеличение на один не помогает.

PHP код:


foreach(new x : Vehicle) // ошибка

Тема: foreach от ziggi

Опции темы

Поиск по теме

foreach от ziggi

Информация о теме

Пользователи, просматривающие эту тему

Ваши права