Переменная

[PawnoNoob]

Да, это глупый вопрос.

Приветствую. Снова столкнулся с проблемой, найти решение которой я, к сожалению не смог.

stock SSkin(playerid) 
{ 
    new id[6], skin[MAX_PLAYERS] = -1; 
    for(new i = 0; i < 10; i++) 
    { 
        sscanf(skins[i], !"p<,>iiiii", id[0], id[1], id[2], id[3], id[4]); 
        skin[playerid]++; 
 
        format(s_string, sizeof(s_string), "%i", skin[playerid]); 
        SendClientMessage(playerid, -1, s_string); 
 
        SetPlayerSkin(playerid, id[sskin[playerid]]); 
    } 
    return true; 
}  

Всё время выводит 0 (прибавляет 1, а дальше никак).

[DeimoS]

А в консоль что пишет?
И что это за странная функция? Каково её предназначение?

[PawnoNoob]

В консоль выводить не пробовал, только в чат.

Функция предназначена для выбора скина при регистрации. Сами скины из базы данных загружаются (есть несколько причин, по которым я не поместил их в массив). Если честно, то я не уверен в правильности реализации, но система изменяет внешность.

[DeimoS]

Сколько я не пытался, но суть кода понять так и не смог. Переменным следует давать более говорящие имена, а не просто какой-то набор символов.

Для начала сделай так:

stock SSkin(playerid) 
{ 
    new id[6], skin[MAX_PLAYERS] = -1; 
    for(new i = 0; i < 10; i++) 
    { 
        SendClientMessage(playerid, -1, skins[i]); 
        sscanf(skins[i], !"p<,>iiiii", id[0], id[1], id[2], id[3], id[4]); 
        skin[playerid]++; 

        format(s_string, sizeof(s_string), "%i", skin[playerid]); 
        SendClientMessage(playerid, -1, s_string); 

        SetPlayerSkin(playerid, id[sskin[playerid]]); 
    } 
    return true; 
}  

И скинь скрин того, что в чат отобразит

[PawnoNoob]

Сколько я не пытался, но суть кода понять так и не смог. Переменным следует давать более говорящие имена, а не просто какой-то набор символов.

Для начала сделай так:

stock SSkin(playerid) 
{ 
    new id[6], skin[MAX_PLAYERS] = -1; 
    for(new i = 0; i < 10; i++) 
    { 
        SendClientMessage(playerid, -1, skins[i]); 
        sscanf(skins[i], !"p<,>iiiii", id[0], id[1], id[2], id[3], id[4]); 
        skin[playerid]++; 

        format(s_string, sizeof(s_string), "%i", skin[playerid]); 
        SendClientMessage(playerid, -1, s_string); 

        SetPlayerSkin(playerid, id[sskin[playerid]]); 
    } 
    return true; 
}  

И скинь скрин того, что в чат отобразит

Про название переменных я знаю. Это просто тестирование системы, поэтому название соответствующее. В будущем, когда буду внедрять это в код (не просто инклуд для разработки, где тестируются все системы перед внедрением), то буду давать нормальные имена переменным.

Вот то, что вывело в чат:

[ziggi]

Про название переменных я знаю. Это просто тестирование системы, поэтому название соответствующее. В будущем, когда буду внедрять это в код (не просто инклуд для разработки, где тестируются все системы перед внедрением), то буду давать нормальные имена переменным.

Вот то, что вывело в чат:

Скорее всего sskin[playerid] имеет значение, выходящие за рамки массива id:

SetPlayerSkin(playerid, id[sskin[playerid]]);  

поэтому выполнение функции прерывается. Установи crashdetect, чтобы такое сразу бросалось в глаза.

А по поводу функции - она вообще ничего путного не делает. Массив skin не используется, SetPlayerSkin зачем-то вызывается 10 раз (то есть это предполагается).
В общем, тебе нужно сначала понять, что ты вообще хочешь сделать.

[PawnoNoob]

Скорее всего sskin[playerid] имеет значение, выходящие за рамки массива id:

SetPlayerSkin(playerid, id[sskin[playerid]]);  

поэтому выполнение функции прерывается. Установи crashdetect, чтобы такое сразу бросалось в глаза.

А по поводу функции - она вообще ничего путного не делает. Массив skin не используется, SetPlayerSkin зачем-то вызывается 10 раз (то есть это предполагается).
В общем, тебе нужно сначала понять, что ты вообще хочешь сделать.

Дело в том, что crashdetect не выводит ошибок вообще. Я понял, что хочу сделать, но в плане реализации, если честно, не очень... SetPlayerSkin вызывается всего один раз, как и сообщения в чат (для теста), так как добавлено ещё одно условие на проверку возраста и национальности персонажа.

[ziggi]

Дело в том, что crashdetect не выводит ошибок вообще. Я понял, что хочу сделать, но в плане реализации, если честно, не очень... SetPlayerSkin вызывается всего один раз, как и сообщения в чат (для теста), так как добавлено ещё одно условие на проверку возраста и национальности персонажа.

У тебя SetPlayerSkin находится в цикле на 10 итераций и никаких проверок я не вижу. В общем, объясни что ты хочешь сделать, а "Функция предназначена для выбора скина при регистрации." - не объяснение.

[Long-]

У тебя SetPlayerSkin находится в цикле на 10 итераций и никаких проверок я не вижу. В общем, объясни что ты хочешь сделать, а "Функция предназначена для выбора скина при регистрации." - не объяснение.

1 раз - значит функция у него действительно прерывается, и дальше действия не проходят, выход за пределы массива.
Автор , а мод скомпилировал с ключом -d3?

[PawnoNoob]

У тебя SetPlayerSkin находится в цикле на 10 итераций и никаких проверок я не вижу. В общем, объясни что ты хочешь сделать, а "Функция предназначена для выбора скина при регистрации." - не объяснение.

Я написал, что добавил условие.

В принципе, функция сама по себе простая, но что-то у меня всё не очень хорошо с её реализацией. В общем: идентификаторы внешности хранятся в базе данных:

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

mysql_tquery(MySQL, !"SELECT * FROM `skins_info`", !"LoadSkins", "");  

И, соответственно, функция:

function LoadSkins()
{
    new rows;
    cache_get_row_count(rows);
    if(rows)
    {
        for(new i = 0, buff; i < rows; i++)
        {
            cache_get_value_name_int(i, !"gend", buff);
            reg_gend{i} = buff;
            cache_get_value_name_int(i, !"nat", buff);
            reg_nat{i} = buff;
            cache_get_value_name_int(i, !"minage", buff);
            reg_minage{i} = buff;
            cache_get_value_name_int(i, !"maxage", buff);
            reg_maxage{i} = buff;
            cache_get_value_name(i, !"sid", skins[i], 26);
        }
        print(!"Загружено.");
    }
    else print(!"Не загружено.");
    return true;
}  

Из сообщения #5 видно, что всё загружается (выводит в чат).
Теперь сама функция выбора (дополненная):

stock SSkin(playerid, gend, nat, age)
{
    new id[6], skin[MAX_PLAYERS] = -1;
    skin[playerid]++;
    for(new i = 0; i < 10; i++)
    {
        if(reg_gend{i} == gend && reg_nat{i} == nat && (age >= reg_minage{i} && age <= reg_maxage{i}))
        {
            SendClientMessage(playerid, -1, skins[i]);

            sscanf(skins[i], !"p<,>iiiii", tempid[0], id[1], id[2], id[3], id[4]);

            format(s_string, sizeof(s_string), "%i", sskin[playerid]);
            SendClientMessage(playerid, -1, s_string);

            SetPlayerSkin(playerid, tempid[skin[playerid]]);
        }
    }
    return true;
}  

Ну и команда для тестирования:

cmd:test(playerid)
{
    SSkin(playerid, /*пол*/, /*национальность*/, /*возраст*/);
    return true;
}