Переменная

[DeimoS]

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

[PawnoNoob]

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

В общем, при нажатии на определённый текстдрав менять скин по порядку из таблицы. Ну и если скин больше 5, то вернуть к первому скину.

[DeimoS]

Для чего каждый раз выгружать данные из skins?

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

Разве эта информация не будет использоваться всеми игроками и довольно часто?
Не лучше ли один раз выгрузить её при старте сервера и потом использовать? От этого было бы гораздо больше пользы, чем от бессмысленной паковки строки в sscanf.
Хотя не совсем понятно для чего её хранить в таблице вообще, если она далеко не индивидуальна и изменяться особо не будет. Да и сам код построен так, что проще было бы хранить её прямо в моде.


Теперь конкретно по твоему вопросу.
Так а ничего, что ты переменную, которая должна хранить количество нажатий, хранишь в стэке? Она у тебя каждый раз ровна -1, потому и дальше 0 не уходит. Хотя даже если бы и уходила, то после 10 нажатий у тебя просто случался бы выход за пределы массива.

Пробуй так (хотя я бы убрал постоянное насилование sscanf)

stock SSkin(playerid, gend, nat, age)
{
    static 
        id[5],
        idx,
        specifier_buff[10+10+1];


    if(isnull(specifier_buff))
        format(specifier_buff, sizeof(specifier_buff), "p<,>a<i>[%d]", sizeof(id));    

    if((idx = GetPVarInt(playerid, "SkinArrayIndex")) >= sizeof(id))
        idx = 0;

    for(new i = 0; i < sizeof(skins); i++)
    {
        if(reg_gend{i} == gend && reg_nat{i} == nat && (age >= reg_minage{i} && age <= reg_maxage{i}))
        {
            sscanf(skins[i], specifier_buff, id);

            format(s_string, sizeof(s_string), "%i", idx);
            SendClientMessage(playerid, -1, s_string);

            SetPlayerSkin(playerid, id[idx]);
            SetPVarInt(playerid, "SkinArrayIndex", idx+1); 
            break;
        }
    }
    return 1;
}  

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

[PawnoNoob]

Для чего каждый раз выгружать данные из skins?

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

Разве эта информация не будет использоваться всеми игроками и довольно часто?
Не лучше ли один раз выгрузить её при старте сервера и потом использовать? От этого было бы гораздо больше пользы, чем от бессмысленной паковки строки в sscanf.
Хотя не совсем понятно для чего её хранить в таблице вообще, если она далеко не индивидуальна и изменяться особо не будет. Да и сам код построен так, что проще было бы хранить её прямо в моде.


Теперь конкретно по твоему вопросу.
Так а ничего, что ты переменную, которая должна хранить количество нажатий, хранишь в стэке? Она у тебя каждый раз ровна -1, потому и дальше 0 не уходит. Хотя даже если бы и уходила, то после 10 нажатий у тебя просто случался бы выход за пределы массива.

Пробуй так (хотя я бы убрал постоянное насилование sscanf)

stock SSkin(playerid, gend, nat, age)
{
    new id[5],
        idx = GetPVarInt(playerid, "SkinArrayIndex")+1;
    if(idx > sizeof(skins))
        idx = 0;
    for(new i = 0; i < 10; i++)
    {
        if(reg_gend{i} == gend && reg_nat{i} == nat && (age >= reg_minage{i} && age <= reg_maxage{i}))
        {
            sscanf(skins[i], !"p<,>iiiii", id[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, id[skin[playerid]]);
            SetPVarInt(playerid, "SkinArrayIndex", idx); 
            break;
        }
    }
    return 1;
}  

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

crashdetect выдал ошибки тут:

SetPlayerSkin(playerid, id[idx]);  

Вот лог:

[debug] AMX backtrace:
[debug] #0 000228c0 in SSkin(playerid=0, gen=1, nat=1, age=18) at D:\mode\gamemodes\inc\dev.inc:73
[debug] #1 00022a90 in public cmd_test (playerid=0, ... <1 argument>) at D:\mode\gamemodes\inc\dev.inc:83
[debug] #2 native DC_CMD () from dc_cmd.dll
[debug] #3 000001a0 in public OnPlayerCommandText (playerid=0, cmdtext[]=@003ad02c "/test") at D:\mode\pawno\include\dc_cmd.inc:67

[DeimoS]

Несколько строк выше дай. Там причина ошибки написана

[PawnoNoob]

Несколько строк выше дай. Там причина ошибки написана

[13:21:40] [debug] Run time error 4: "Array index out of bounds"
[13:21:40] [debug]  Attempted to read/write array element at index 5 in array of size 5

new id[6] поможет?

[DeimoS]

stock SSkin(playerid, gend, nat, age)
{
    static 
        id[5],
        idx,
        specifier_buff[10+10+1];


    if(isnull(specifier_buff))
        format(specifier_buff, sizeof(specifier_buff), "p<,>a<i>[%d]", sizeof(id));    

    if((idx = GetPVarInt(playerid, "SkinArrayIndex")) >= sizeof(id))
        idx = 0;

    for(new i = 0; i < sizeof(skins); i++)
    {
        if(reg_gend{i} == gend && reg_nat{i} == nat && (age >= reg_minage{i} && age <= reg_maxage{i}))
        {
            sscanf(skins[i], specifier_buff, id);

            format(s_string, sizeof(s_string), "%i", idx);
            SendClientMessage(playerid, -1, s_string);

            SetPlayerSkin(playerid, id[idx]);
            SetPVarInt(playerid, "SkinArrayIndex", idx+1); 
            break;
        }
    }
    return 1;
}  

[PawnoNoob]

stock SSkin(playerid, gend, nat, age)
{
    static 
        id[5],
        idx,
        specifier_buff[10+10+1];


    if(isnull(specifier_buff))
        format(specifier_buff, sizeof(specifier_buff), "p<,>a<i>[%d]", sizeof(id));    

    if((idx = GetPVarInt(playerid, "SkinArrayIndex")) >= sizeof(id))
        idx = 0;

    for(new i = 0; i < sizeof(skins); i++)
    {
        if(reg_gend{i} == gend && reg_nat{i} == nat && (age >= reg_minage{i} && age <= reg_maxage{i}))
        {
            sscanf(skins[i], specifier_buff, id);

            format(s_string, sizeof(s_string), "%i", idx);
            SendClientMessage(playerid, -1, s_string);

            SetPlayerSkin(playerid, id[idx]);
            SetPVarInt(playerid, "SkinArrayIndex", idx+1); 
            break;
        }
    }
    return 1;
}  

Этот вариант работает. Спасибо

- - - Добавлено - - -

Кстати, куда лучше перенести sscanf, чтобы его не насиловать? В загрузку?

[ziggi]

Кстати, куда лучше перенести sscanf, чтобы его не насиловать? В загрузку?

Да, в LoadSkins.

Вместо твоего:

cache_get_value_name(i, !"sid", skins[i], 26);  

что-то вроде этого:

#define MAX_SKINS_COUNT 5 
new skins[10][MAX_SKINS_COUNT]; 
 
// LoadSkins 
cache_get_value_name(i, !"sid", temp); 
sscanf(temp, !"p<,>a<i>[" #MAX_SKINS_COUNT "]", skins[i][0]);  

[PawnoNoob]

Да, в LoadSkins.

Вместо твоего:

cache_get_value_name(i, !"sid", skins[i], 26);  

что-то вроде этого:

#define MAX_SKINS_COUNT 5
new skins[10][MAX_SKINS_COUNT];

// LoadSkins
cache_get_value_name(i, !"sid", temp);
sscanf(temp, !"p<,>a<i>[" #MAX_SKINS_COUNT "]", skins[i][0]);  

А каким образом тогда переделывать функцию "SSkin"?