Короткие вопросы [Вопрос:Ответ]

[ziggi]

Вопрос о флагах/типах.

Каким образом можно построить систему так, чтобы можно было устанавливать множество типов.

Рассмотрим на системе фракций.

new FactionTypePolice = DefineFactionType("Правоохранительные органы"); // Возвращает 1
new FactionTypeArmy = DefineFactionType("Вооруженные силы"); // Возвращает 2
new FactionTypeEmercom = DefineFactionType("Служба спасения"); // Возвращает 3

CreateFaction("Полиция", FactionTypePolice);
CreateFaction("ФБР", FactionTypePolice | FactionTypeArmy); // Тип получается - 3, т.е. служба спасения, а нужно два типа.  

Интересует способ именно с битами.

Чтобы возвращало именно 1, 2, 3 - нельзя, можно только чтобы возвращало 1, 2, 4, 8 и т.д., то с битовым сдвигом.

new gFactionTypes = 1;new gFactionCurrent = 0; stock DefineFactionType(name[]){    new id = gFactionTypes << gFactionCurrent;    gFactionCurrent++;    return id;} stock CreateFaction(name[], types){    if (types & FactionTypeArmy) {     }    if (types & FactionTypePolice) {     }}

[pawnoholic]

Какая то необъяснимая для меня проблема с перечислением.

Я не могу обнулить строку, которая инициализируется в enum.

enum E_PLAYER_DATA{    E_PLAYER_NAME[24 + 1],    E_PLAYER_PASSWORD[16 + 1],    E_PLAYER_EMAIL[144 + 1],}new PlayerData[MAX_PLAYERS][E_PLAYER_DATA]; stock SetPlayerData(playerid, E_PLAYER_DATA:data, const source[]){    PlayerData[playerid][data][0] = '\0'; // Ловлю fatal error     return strcat(PlayerData[playerid][data], source);}

А инициализация напрямую, проходит без ошибок.

PlayerData[playerid][E_PLAYER_NAME][0] = '\0';

[Daniel_Cortez]

Это происходит на последней версии компилятора от Zeex?

[pawnoholic]

Это происходит на последней версии компилятора от Zeex?

Я использую sampctl, так что скорее всего на предыдущей версии.

Пришлось пока к такому варианту прийти:

stock SetPlayerDataString(playerid, E_PLAYER_DATA:data, const source[]){    switch (data)    {        case E_PLAYER_NAME :            strcpy(PlayerData[playerid][E_PLAYER_NAME], source, MAX_PLAYER_NAME);         case E_PLAYER_PASSWORD :            strcpy(PlayerData[playerid][E_PLAYER_PASSWORD], source, MAX_PLAYER_PASSWORD);    }    return 0;}

В моем случае, это даже наверное лучший вариант.

[DeimoS]

Эмм, так потому что ты пытаешься обратиться к трехмерному массиву, когда на деле у тебя двумерный? Ттретья мера существует только когда ты обращается к члену перечисления. И существует она не как третья мера, а как своеобразный индекс для перечисления (к какой ячейке второй меры нужно обратиться, чтоб получить нужные данные) и не более.
Вот так всё будет работать

PlayerData[playerid][data] = '\0';  

Хотя strcat всё равно не будет работать нормально, ибо не сможет нормально получить размер двумерного массива (sizeof будет выдавать 0).

Вот так будет правильнее

enum E_PLAYER_DATA
{
    E_PLAYER_NAME[24 + 1],
    E_PLAYER_PASSWORD[16 + 1],
    E_PLAYER_EMAIL[144 + 1],
}
new PlayerData[MAX_PLAYERS][E_PLAYER_DATA];

stock SetPlayerData(playerid, E_PLAYER_DATA:data, const source[], const size = sizeof(source))
{
    PlayerData[playerid][data] = '\0'; //если в "data" передать "E_PLAYER_PASSWORD", то, соответственно, мы обратимся к ячейке "25+1", так как перед членом перечисления "E_PLAYER_PASSWORD" находится "E_PLAYER_NAME", занимающий первые "24+1" ячеек. Соответственно, мы обнулим именно 26-ую ячейку, которая является началом "E_PLAYER_PASSWORD"

    return strcat(PlayerData[playerid][data], source, size);
}  

[Daniel_Cortez]

Вот так будет правильнее

enum E_PLAYER_DATA
{
    E_PLAYER_NAME[24 + 1],
    E_PLAYER_PASSWORD[16 + 1],
    E_PLAYER_EMAIL[144 + 1],
}
new PlayerData[MAX_PLAYERS][E_PLAYER_DATA];

stock SetPlayerData(playerid, E_PLAYER_DATA:data, const source[], const size = sizeof(source))
{
    PlayerData[playerid][data] = '\0'; //если в "data" передать "E_PLAYER_PASSWORD", то, соответственно, мы обратимся к ячейке "25+1", так как перед членом перечисления "E_PLAYER_PASSWORD" находится "E_PLAYER_NAME", занимающий первые "24+1" ячеек. Соответственно, мы обнулим именно 26-ую ячейку, которая является началом "E_PLAYER_PASSWORD"

    return strcat(PlayerData[playerid][data], source, size);
}  

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

[DeimoS]

Угадай, что случится, если размер массива-источника окажется больше приёмника.

Об этом стоит думать ещё тогда, когда в голову приходит мысль написать подобную "универсальную" функцию. И решается это проверкой размера текста ещё до вызова SetPlayerData (что должно быть в любом случае, если не хотите на выходе получить проект по типу SA-MP, который целиком состоит из костылей и багов).
Хотя это в любом случае является извращением, как по мне.

[pawnoholic]

Так все таки что по моему вопросу?

Хотя это в принципе уже не важно, меня устраивает и такой вариант:

stock SetPlayerDataString(playerid, E_PLAYER_DATA:data, const source[]){    switch (data)    {        case E_PLAYER_NAME :            strcpy(PlayerData[playerid][E_PLAYER_NAME], source, MAX_PLAYER_NAME);         case E_PLAYER_PASSWORD :            strcpy(PlayerData[playerid][E_PLAYER_PASSWORD], source, MAX_PLAYER_PASSWORD);    }    return 0;}

Эмм, так потому что ты пытаешься обратиться к трехмерному массиву, когда на деле у тебя двумерный? Ттретья мера существует только когда ты обращается к члену перечисления. И существует она не как третья мера, а как своеобразный индекс для перечисления (к какой ячейке второй меры нужно обратиться, чтоб получить нужные данные) и не более.

Ну вот теперь все ясно, спасибо.

[Daniel_Cortez]

Хотя это в принципе уже не важно, меня устраивает и такой вариант:

stock SetPlayerDataString(playerid, E_PLAYER_DATA:data, const source[]){    switch (data)    {        case E_PLAYER_NAME :            strcpy(PlayerData[playerid][E_PLAYER_NAME], source, MAX_PLAYER_NAME);         case E_PLAYER_PASSWORD :            strcpy(PlayerData[playerid][E_PLAYER_PASSWORD], source, MAX_PLAYER_PASSWORD);    }    return 0;}

Хотя это в любом случае является извращением, как по мне.

DeimoS, полностью согласен, хотя я бы назвал это не просто извращением, а конкретно мазохизмом. Если уж так не охота возиться с данными напрямую, проще сделать отдельные функции (например, Player_SetName(), Player_SetPassword(), Player_SetEMail()). Зачем пытаться запихнуть всё в одну функцию, если всё равно придётся в switch прописывать отдельный код для каждого варианта?

[pawnoholic]

DeimoS, полностью согласен, хотя я бы назвал это не просто извращением, а конкретно мазохизмом. Если уж так не охота возиться с данными напрямую, проще сделать отдельные функции (например, Player_SetName(), Player_SetPassword(), Player_SetEMail()). Зачем пытаться запихнуть всё в одну функцию, если всё равно придётся в switch прописывать отдельный код для каждого варианта?

SetPlayerData(String, Int, Float) - это что то вроде API, для доступа из вне для static PlayerData[MAX_PLAYERS][E_PLAYER_DATA];