Правильно ли использую функцию p<,> ?

[Dimon_Fanat]

Решил создать настройки заместителя, но увидел на форум что лучше сокращать это в базе данных. И тут появился вопрос правильно ли сделал, или где-то допустил ошибку.?

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

enum FnInfo 
{ 
    fID, 
    fName[32], 
    fLeader[MAX_PLAYER_NAME], 
    fDSettings[4], 
}; 
new FracInfo[MAX_FRACTION][FnInfo];  

publics LoadFractions() 
{ 
    new rows; 
    new string[10]; 
    cache_get_row_count(rows); 
    for(new i = 0; i < rows; i++) 
    { 
        cache_get_value_name_int(i,"ID",FracInfo[i][fID]); 
        cache_get_value_name(i,"NameFrac",FracInfo[i][fName],32); 
        cache_get_value_name(i,"Leader",FracInfo[i][fLeader],MAX_PLAYER_NAME); 
        cache_get_value_name(i,"DSettings",string,8); 
        sscanf(string, "p<,>a<i>[4]",FracInfo[i][fDSettings]); 
         
    } 
    printf("Загружено [%i] фракций.",rows); 
}  

[ziggi]

Это не лучше, такая таблица не соответствует первой нормальной форме.

Что касается кода - если это работает, то ты сделал правильно, а если нет, то не правильно. В общем, если не работает, то попробуй это:

sscanf(string, "p<,>a<i>[4]",FracInfo[i][fDSettings][0]);

[DeimoS]

Дополню ответ выше: если ты собираешься расширять количество опций постоянно, то лучше создай ещё одну таблицу и уже в ней под каждую опцию выдели свой столбец, связав данные по ID фракции.
А если не собираешься дополнять, то, соответственно, создай столбцы прямо в текущей таблице.

Это и вразрез с нормализацией идёт, и ты лишние действия будешь каждый раз выполнять при загрузке/записи (например, нужно получить значение только одной опции, но нужно будет выгружать и обрабатывать все сразу).

[Dimon_Fanat]

Дополню ответ выше: если ты собираешься расширять количество опций постоянно, то лучше создай ещё одну таблицу и уже в ней под каждую опцию выдели свой столбец, связав данные по ID фракции.
А если не собираешься дополнять, то, соответственно, создай столбцы прямо в текущей таблице.

Это и вразрез с нормализацией идёт, и ты лишние действия будешь каждый раз выполнять при загрузке/записи (например, нужно получить значение только одной опции, но нужно будет выгружать и обрабатывать все сразу).

Ну у меня у фракции всего 5 настроек будет. А чтобы не делать в базе данных, Dsetting1, Dsetting2, и т.д. я сделал в таком виде. От сюда и вопрос так лучше или же нет.

[DeimoS]

Ну у меня у фракции всего 5 настроек будет. А чтобы не делать в базе данных, Dsetting1, Dsetting2, и т.д. я сделал в таком виде. От сюда и вопрос так лучше или же нет.

Выше уже ответ есть

Это и вразрез с нормализацией идёт, и ты лишние действия будешь каждый раз выполнять при загрузке/записи (например, нужно получить значение только одной опции, но нужно будет выгружать и обрабатывать все сразу).