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

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

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

http://rgho.st/6YZLqXN2Q/thumb.png

PHP код:

---

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


---

PHP код:

---

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);
} 


---

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

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

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

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

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

Цитата:

---

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

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

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

---

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

Цитата:

---

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

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

---

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

Цитата:

---

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

---