Правильно ли оформлен цикл загрузки бизнесов?

[Dimon_Fanat]

Добрый день, скажите правильно ли делаю for?

enum bInfo 
{ 
    bID, 
    bName[32], 
    bType, 
Float:bEnterX, 
Float:bEnterY, 
Float:bEnterZ, 
Float:bExitX, 
Float:bExitY, 
Float:bExitZ, 
    bInterior, 
} 
new BizInfo[MAX_BUSINESS][bInfo];  

        for(new i = 0; i < sizeof(BizInfo); i++) 
        { 
            if(IsPlayerInRangeOfPoint(playerid,0.8,BizInfo[i][bEnterX],BizInfo[i][bEnterY],BizInfo[i][bEnterZ])) 
            { 
                SetPlayerPos(playerid,BizInfo[i][bExitX],BizInfo[i][bExitY],BizInfo[i][bExitZ]); 
                SetPlayerInterior(playerid,BizInfo[i][bInterior]); 
                SetPlayerVirtualWorld(playerid,BizInfo[i][bID]); 
            } 
        }  

publics LoadBusiness() 
{ 
    new rows; 
    cache_get_row_count(rows); 
    for(new i = 0; i < rows; i++) 
    { 
        cache_get_value_name_int(i,"ID",BizInfo[i][bID]); 
        cache_get_value_name(i,"NameBiz",BizInfo[i][bName],32); 
        cache_get_value_name_int(i,"Type",BizInfo[i][bType]); 
        cache_get_value_name_float(i,"EnterX",BizInfo[i][bEnterX]); 
        cache_get_value_name_float(i,"EnterY",BizInfo[i][bEnterY]); 
        cache_get_value_name_float(i,"EnterZ",BizInfo[i][bEnterZ]); 
        cache_get_value_name_float(i,"ExitX",BizInfo[i][bExitX]); 
        cache_get_value_name_float(i,"ExitY",BizInfo[i][bExitY]); 
        cache_get_value_name_float(i,"ExitZ",BizInfo[i][bExitZ]); 
        cache_get_value_name_int(i,"Interior",BizInfo[i][bInterior]); 
         
        CreateDynamicMapIcon(BizInfo[i][bEnterX],BizInfo[i][bEnterY],BizInfo[i][bEnterZ],BizInfo[i][bType],0xFFFFFFAA,0,-1,-1,400.0); 
        CreateDynamicCP(BizInfo[i][bEnterX],BizInfo[i][bEnterY],BizInfo[i][bEnterZ],1.0,-1,-1,-1,3.0); 
    } 
    printf("Загружено [%i] бизнесов.",rows); 
}  

[DeimoS]

Да. Только нужно ещё либо в запрос добавить

"... LIMIT %d", ..., sizeof(BizInfo));

Либо в LoadBusiness

publics LoadBusiness() {     new rows;     cache_get_row_count(rows);      if(rows == 0)        return 0;      if(rows >= sizeof(BizInfo))        rows = sizeof(BizInfo);     for(new i = 0; i < rows; i++)

дабы в случае, если в таблице окажется слишком много строк, не словить выход за пределы массива. Можно ещё добавить сообщение в консоль с предупреждением о том, что нужно увеличить BizInfo

UPD: Вот так можно сделать ещё

publics LoadBusiness() {     new rows;     cache_get_row_count(rows);     if(!rows)    {        print("Загружено [0] бизнесов. Таблица с бизнесами пуста!");         return 0;    }    else if(rows >= sizeof(BizInfo))    {        printf("Внимание: в таблице с бизнесами слишком много строк. Увеличте массив BizInfo на %d, дабы исправить ошибку [%d|%d]", rows-sizeof(BizInfo), rows, sizeof(BizInfo));        rows = sizeof(BizInfo);    }    new Float:x,        Float:y,        Float:z;    for(new i = 0; i < rows; i++)     {         cache_get_value_name_int(i,"ID",BizInfo[i][bID]);         cache_get_value_name(i,"NameBiz",BizInfo[i][bName],32);         cache_get_value_name_int(i,"Type",BizInfo[i][bType]);         cache_get_value_name_float(i,"EnterX", x);         cache_get_value_name_float(i,"EnterY", y);         cache_get_value_name_float(i,"EnterZ", z);         cache_get_value_name_float(i,"ExitX",BizInfo[i][bExitX]);         cache_get_value_name_float(i,"ExitY",BizInfo[i][bExitY]);         cache_get_value_name_float(i,"ExitZ",BizInfo[i][bExitZ]);         cache_get_value_name_int(i,"Interior",BizInfo[i][bInterior]);          BizInfo[i][bEnterX] = x;        BizInfo[i][bEnterY] = y;        BizInfo[i][bEnterZ] = z;         CreateDynamicMapIcon(x, y, z, BizInfo[i][bType],0xFFFFFFAA,0,-1,-1,400.0);         CreateDynamicCP(x, y, z, 1.0, -1, -1, -1, 3.0);     }     printf("Загружено [%i] бизнесов.",rows); }

[Dimon_Fanat]

Да. Только нужно ещё либо в запрос добавить

"... LIMIT %d", ..., sizeof(BizInfo));

Либо в LoadBusiness

publics LoadBusiness() {     new rows;     cache_get_row_count(rows);      if(rows == 0)        return 0;      if(rows >= sizeof(BizInfo))        rows = sizeof(BizInfo);     for(new i = 0; i < rows; i++)

дабы в случае, если в таблице окажется слишком много строк, не словить выход за пределы массива. Можно ещё добавить сообщение в консоль с предупреждением о том, что нужно увеличить BizInfo

UPD: Вот так можно сделать ещё

publics LoadBusiness() {     new rows;     cache_get_row_count(rows);     if(!rows)    {        print("Загружено [0] бизнесов. Таблица с бизнесами пуста!");         return 0;    }    else if(rows >= sizeof(BizInfo))    {        printf("Внимание: в таблице с бизнесами слишком много строк. Увеличте массив BizInfo на %d, дабы исправить ошибку [%d|%d]", rows-sizeof(BizInfo), rows, sizeof(BizInfo));        rows = sizeof(BizInfo);    }    new Float:x,        Float:y,        Float:z;    for(new i = 0; i < rows; i++)     {         cache_get_value_name_int(i,"ID",BizInfo[i][bID]);         cache_get_value_name(i,"NameBiz",BizInfo[i][bName],32);         cache_get_value_name_int(i,"Type",BizInfo[i][bType]);         cache_get_value_name_float(i,"EnterX", x);         cache_get_value_name_float(i,"EnterY", y);         cache_get_value_name_float(i,"EnterZ", z);         cache_get_value_name_float(i,"ExitX",BizInfo[i][bExitX]);         cache_get_value_name_float(i,"ExitY",BizInfo[i][bExitY]);         cache_get_value_name_float(i,"ExitZ",BizInfo[i][bExitZ]);         cache_get_value_name_int(i,"Interior",BizInfo[i][bInterior]);          BizInfo[i][bEnterX] = x;        BizInfo[i][bEnterY] = y;        BizInfo[i][bEnterZ] = z;         CreateDynamicMapIcon(x, y, z, BizInfo[i][bType],0xFFFFFFAA,0,-1,-1,400.0);         CreateDynamicCP(x, y, z, 1.0, -1, -1, -1, 3.0);     }     printf("Загружено [%i] бизнесов.",rows); }

Понял, спасибо!
А еще такой вопрос, допустим у меня бизнес в базе данных начинается с 1, 0 бизнеса нет, но когда я ввожу команду /goto и тп к бизнесу 1, меня тп к бизнесу №2, а если введу №0 то тп к бизнесу 1.

CMD:gotobiz(playerid,params[])
{
    if(sscanf(params,"d",params[0]))return SendClientMessage(playerid,0x00BFFFAA,"[КОМАНДЫ] {FFFFFF}/gotobiz [id бизнеса].");
    if(params[0] < 0 || params[0] >= sizeof(BizInfo))return SendClientMessage(playerid,0x00BFFFAA, "Неправильный номер бизнеса");
    SetPlayerPos(playerid,BizInfo[params[0]][bEnterX],BizInfo[params[0]][bEnterY],BizInfo[params[0]][bEnterZ]);
    SetPlayerVirtualWorld(playerid,0);
    SetPlayerInterior(playerid,0);
    return 1;
}  

[Geebrox]

Можно так:

publics LoadBusiness() {     new rows;     cache_get_row_count(rows);     if(!rows)    {        print("Загружено [0] бизнесов. Таблица с бизнесами пуста!");         return 0;    }    else if(rows >= sizeof(BizInfo))    {        printf("Внимание: в таблице с бизнесами слишком много строк. Увеличте массив BizInfo на %d, дабы исправить ошибку [%d|%d]", rows-sizeof(BizInfo), rows, sizeof(BizInfo));        rows = sizeof(BizInfo);    }    new Float:x,        Float:y,        Float:z;    for(new i = 0, j = 1; i < rows; i++, j++)     {         cache_get_value_name_int(i,"ID",BizInfo[j][bID]);         cache_get_value_name(i,"NameBiz",BizInfo[j][bName],32);         cache_get_value_name_int(i,"Type",BizInfo[j][bType]);         cache_get_value_name_float(i,"EnterX", x);         cache_get_value_name_float(i,"EnterY", y);         cache_get_value_name_float(i,"EnterZ", z);         cache_get_value_name_float(i,"ExitX",BizInfo[j][bExitX]);         cache_get_value_name_float(i,"ExitY",BizInfo[j][bExitY]);         cache_get_value_name_float(i,"ExitZ",BizInfo[j][bExitZ]);         cache_get_value_name_int(i,"Interior",BizInfo[j][bInterior]);          BizInfo[j][bEnterX] = x;        BizInfo[j][bEnterY] = y;        BizInfo[j][bEnterZ] = z;         CreateDynamicMapIcon(x, y, z, BizInfo[j][bType],0xFFFFFFAA,0,-1,-1,400.0);         CreateDynamicCP(x, y, z, 1.0, -1, -1, -1, 3.0);     }     printf("Загружено [%i] бизнесов.",rows); }

[DeimoS]

Понял, спасибо!
А еще такой вопрос, допустим у меня бизнес в базе данных начинается с 1, 0 бизнеса нет, но когда я ввожу команду /goto и тп к бизнесу 1, меня тп к бизнесу №2, а если введу №0 то тп к бизнесу 1.

ID в базе данных никак не связан с ID на сервере. По крайней мере не должен быть связан.
ID в базе (поле с AUTO_INCREMENT) нужен сугубо для работы с базой. Хочешь связать ID бизнеса на сервере с базой - создавай дополнительное поле и храни ID в нём, а потом, при выгрузке данных, используй этот ID в качестве индекса массива.