Проверка на координаты входа

[Saibot]

Делаю вход в бизнес через KEY_WALK в OnPlayerKeyStateChange и задался таким вопросом.
Если игрок будет нажимать на L.Alt в любом месте, ему в OnPlayerKeyStateChange, будет циклом прогонять все координаты входа в бизнес, дабы проверить находится ли он возле бизнеса.
Пример:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys){    if(newkeys & KEY_WALK)    {        for(new b = 1; b <= gNumberBusiness; b++)        {            if(IsPlayerInRangeOfPoint(playerid, 2.0, BusinessInfo[b][bEnterX], BusinessInfo[b][bEnterY], BusinessInfo[b][bEnterZ]))            {                //Вход                return 1;            }        }        return 1;    }}

Будет ли лучше, если создать динамическую зону и проверять в ней?

public OnPlayerEnterDynamicArea(playerid, areaid){    for(new b = 1; b <= gNumberBusiness; b++)    {        if(areaid == ar_BusinessEnter[b])        {            gPlayerBusinessID[playerid] = b;            return 1;        }    }    return 1;} public OnPlayerLeaveDynamicArea(playerid, areaid){    for(new b = 1; b <= gNumberBusiness; b++)    {        if(areaid == ar_BusinessEnter[b])        {            gPlayerBusinessID[playerid] = 0;            return 1;        }    }    return 1;} public OnPlayerKeyStateChange(playerid, newkeys, oldkeys){    if(newkeys & KEY_WALK)    {        if(gPlayerBusinessID[playerid] != 0)        {            SetPlayerPos(playerid,                BusinessInfo[gPlayerBusinessID[playerid]][bEnterX],                BusinessInfo[gPlayerBusinessID[playerid]][bEnterY],                BusinessInfo[gPlayerBusinessID[playerid]][bEnterZ]            );            return 1;        }        return 1;    }    return 1;}

[punkochel]

Лично я повсеместно использую динамические зоны, и советовал бы именно их, ибо проверка нахождения в зоне происходит быстрее чем проверка на нахождение в радиусе по координатам.
Единственное что я бы сделал, то не стал бы создавать массив (gPlayerBusinessID[MAX_PLAYERS]) для этой реализации:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys){    if(newkeys & KEY_WALK)    {        for(new b = 1; b <= gNumberBusiness; b++)        {            if(IsPlayerInDynamicArea(playerid, areaid))            {                //Вход                return 1;            }        }        return 1;    }}

[Saibot]

Лично я повсеместно использую динамические зоны, и советовал бы именно их, ибо проверка нахождения в зоне происходит быстрее чем проверка на нахождение в радиусе по координатам.
Единственное что я бы сделал, то не стал бы создавать массив (gPlayerBusinessID[MAX_PLAYERS]) для этой реализации:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys){    if(newkeys & KEY_WALK)    {        for(new b = 1; b <= gNumberBusiness; b++)        {            if(IsPlayerInDynamicArea(playerid, areaid))            {                //Вход                return 1;            }        }        return 1;    }}

Но проблема в том, что цикл остается в OnPlayerKeyStateChange и при нажатие L.Alt (в любом месте), он будет каждый раз проверять все зоны.

Мне нужно узнать, большая ли разница между первым и воторым вариантом, который я написал.

[whale]

Дабы априори избежать циклов, можно, при создании динамической зоны, записывать некоторую дополнительную информацию о ней в отдельный массив данных (в данном случае - закреплена ли зона за каким-либо бизнесом и, если да - то является ли эта зона входом, либо выходом).

Примерный вариант (не тестировал):

/* 
    Definitions 
*/ 
 
#define INVALID_BUSINESS_ID             (-1) 
#define AREA_BUSINESS_STATUS_ENTRANCE   (0) 
#define AREA_BUSINESS_STATUS_EXIT       (1) 
 
/* 
    Global variables 
*/ 
 
enum E_DYNAMIC_AREA_DATA 
{ 
    area_bussiness_id, 
    area_bussiness_status 
} 
 
new 
    gDynamicAreaExtraData[/*кол-во зон*/][E_DYNAMIC_AREA_DATA]; // заполнять информацию при создании зон 
 
new 
    gPlayerCurrentDynamicAreaId[MAX_PLAYERS] = {INVALID_DYNAMIC_AREA_ID, ...}; 
 
 
/* 
    Publics 
*/ 
 
public OnPlayerEnterDynamicArea(playerid, areaid) 
{ 
    gPlayerCurrentDynamicAreaId[playerid] = areaid; 
    return 1; 
} 
 
public OnPlayerLeaveDynamicArea(playerid, areaid) 
{ 
    gPlayerCurrentDynamicAreaId[playerid] = INVALID_DYNAMIC_AREA_ID; 
    return 1; 
} 
 
public OnPlayerKeyStateChange(playerid, newkeys, oldkeys) 
{ 
    if(newkeys & KEY_WALK) 
    { 
        new 
            player_dynamic_area_id = gPlayerCurrentDynamicAreaId[playerid]; 
 
        if(player_dynamic_area_id != INVALID_DYNAMIC_AREA_ID) 
        { 
            if(gDynamicAreaExtraData[player_dynamic_area_id][area_bussiness_id] != INVALID_BUSINESS_ID) 
            { 
                new 
                    dynamic_area_business_status = gDynamicAreaExtraData[player_dynamic_area_id][area_bussiness_status]; 
 
                // ID бизнеса указан в структуре динамических зон, так что получать координаты входа/выхода не сосавит труда 
 
                if(dynamic_area_business_status == AREA_BUSINESS_STATUS_ENTRANCE) 
                { 
                    // вход 
 
                    return 1; 
                } 
                else if(dynamic_area_business_status == AREA_BUSINESS_STATUS_EXIT) 
                { 
                    // elise if дабы избежать возможных ошибок в коде (присвоение значения отличного от 0 или 1) 
                    // выход 
 
                    return 1; 
                } 
            } 
        } 
    } 
    return 1; 
}  

[Saibot]

public OnPlayerEnterDynamicArea(playerid, areaid){    gPlayerCurrentDynamicAreaId[playerid] = areaid;    return 1;} public OnPlayerLeaveDynamicArea(playerid, areaid){    gPlayerCurrentDynamicAreaId[playerid] = INVALID_DYNAMIC_AREA_ID;    return 1;}

А разве оно запишет в gPlayerCurrentDynamicAreaId нужный ID зоны в которой находится игрок?

[whale]

public OnPlayerEnterDynamicArea(playerid, areaid){    gPlayerCurrentDynamicAreaId[playerid] = areaid;    return 1;} public OnPlayerLeaveDynamicArea(playerid, areaid){    gPlayerCurrentDynamicAreaId[playerid] = INVALID_DYNAMIC_AREA_ID;    return 1;}

А разве оно запишет в gPlayerCurrentDynamicAreaId нужный ID зоны в которой находится игрок?

Запишется ID последней динамической зоны, в которую вошёл игрок.

[Pro_Coder]

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

Я предпочитаю использовать то, что выполняется один раз

[execution]

На самом деле, было-бы неплохо сделать сделать вход через пикапы с помощью нажатия клавиши и всё

[Saibot]

На самом деле, было-бы неплохо сделать сделать вход через пикапы с помощью нажатия клавиши и всё

В первом варианте. сделано через пикап по нажатию на Alt.
Или, как ты имел введу?

[execution]

В первом варианты показано нажатие на АЛЬТ по координатам входа