Помогите оптимизировать код

[OKStyle]

Боги оптимизации

    case 5859:  
    {  
        if(!response) return ShowPlayerDialog(playerid, 5858, DIALOG_STYLE_LIST, "OFFMEMBERS", "- Весь состав\n- По рангу", "Далее", "Отмена"); 
        if(1 <= listitem <= 8) 
        { 
            if(PlayerInfo[playerid][pStranica] == 0) return 1; 
            if(PlayerInfo[playerid][pVsegostr] == listitem) PlayerInfo[playerid][pStranica] -= 8; 
        } 
        else if(listitem == 9) 
        { 
            if(PlayerInfo[playerid][pStranica] == 0) PlayerInfo[playerid][pStranica] += 8; 
            if(PlayerInfo[playerid][pStranica] > 0) 
            { 
                if(PlayerInfo[playerid][pVsegostr] == 9) PlayerInfo[playerid][pStranica] -= 8; 
                else if(PlayerInfo[playerid][pVsegostr] == 10) PlayerInfo[playerid][pStranica] += 8; 
            } 
        } 
        else if(listitem == 10) PlayerInfo[playerid][pStranica] -= 8; 
        if(listitem == 0) mysql_format(dbHandle, strings, 128, "SELECT * FROM `accounts` WHERE `Member` = '%d' LIMIT %d , 8", PlayerInfo[playerid][pLeader], PlayerInfo[playerid][pStranica]); 
        else 
        { 
            if(!PlayerInfo[playerid][pOrderRank]) mysql_format(dbHandle, strings, 128, "SELECT * FROM `accounts` WHERE `Member` = '%d' LIMIT %d , 8", PlayerInfo[playerid][pLeader], PlayerInfo[playerid][pStranica]); 
            else mysql_format(dbHandle, strings, 128, "SELECT * FROM `accounts` WHERE `Member` = '%d' ORDER BY `accounts`.`Rank` ASC LIMIT %d , 8", PlayerInfo[playerid][pLeader], PlayerInfo[playerid][pStranica]); 
        } 
        mysql_function_query(dbHandle, strings, true, "OnPlayerSelectOffmembers", "i", playerid); 
        ShowPlayerDialogUninvite(playerid, inputtext); 
        return 1; 
    }  

[Gressie]

Боги оптимизации

    case 5859:  
    {  
        if(!response) return ShowPlayerDialog(playerid, 5858, DIALOG_STYLE_LIST, "OFFMEMBERS", "- Весь состав\n- По рангу", "Далее", "Отмена"); 
        if(1 <= listitem <= 8) 
        { 
            if(PlayerInfo[playerid][pStranica] == 0) return 1; 
            if(PlayerInfo[playerid][pVsegostr] == listitem) PlayerInfo[playerid][pStranica] -= 8; 
        } 
        else if(listitem == 9) 
        { 
            if(PlayerInfo[playerid][pStranica] == 0) PlayerInfo[playerid][pStranica] += 8; 
            if(PlayerInfo[playerid][pStranica] > 0) 
            { 
                if(PlayerInfo[playerid][pVsegostr] == 9) PlayerInfo[playerid][pStranica] -= 8; 
                else if(PlayerInfo[playerid][pVsegostr] == 10) PlayerInfo[playerid][pStranica] += 8; 
            } 
        } 
        else if(listitem == 10) PlayerInfo[playerid][pStranica] -= 8; 
        if(listitem == 0) mysql_format(dbHandle, strings, 128, "SELECT * FROM `accounts` WHERE `Member` = '%d' LIMIT %d , 8", PlayerInfo[playerid][pLeader], PlayerInfo[playerid][pStranica]); 
        else 
        { 
            if(!PlayerInfo[playerid][pOrderRank]) mysql_format(dbHandle, strings, 128, "SELECT * FROM `accounts` WHERE `Member` = '%d' LIMIT %d , 8", PlayerInfo[playerid][pLeader], PlayerInfo[playerid][pStranica]); 
            else mysql_format(dbHandle, strings, 128, "SELECT * FROM `accounts` WHERE `Member` = '%d' ORDER BY `accounts`.`Rank` ASC LIMIT %d , 8", PlayerInfo[playerid][pLeader], PlayerInfo[playerid][pStranica]); 
        } 
        mysql_function_query(dbHandle, strings, true, "OnPlayerSelectOffmembers", "i", playerid); 
        ShowPlayerDialogUninvite(playerid, inputtext); 
        return 1; 
    }  

Не работает увольнение, как и во всех остальных вариантах, и появился лидер в списке (В стандарте моём этого не было).

[OKStyle]

А как увольнение относится к представленному коду? А лидер в списке и должен появляться, судя по коду. Просто из-за его кривизны у тебя он не показывался. Там где listitem == 0.

[Gressie]

А как увольнение относится к представленному коду? А лидер в списке и должен появляться, судя по коду. Просто из-за его кривизны у тебя он не показывался. Там где listitem == 0.

До этого в том стандарт коде, я щёлкаю по нику и диалог типа: Вы хотите уволить ... ?
Теперь его попросту нету, щелкаю по нику и ничего не происходит а хотя:

ShowPlayerDialogUninvite(playerid, inputtext);  

Отвечает за увольнение его.

[OKStyle]

if(PlayerInfo[playerid][pStranica] == 0) return 1;
if(PlayerInfo[playerid][pVsegostr] == listitem) PlayerInfo[playerid][pStranica] -= 8;  

Замени на:

if(PlayerInfo[playerid][pStranica] != 0)
{
if(PlayerInfo[playerid][pVsegostr] == listitem) PlayerInfo[playerid][pStranica] -= 8;
}  

[Gressie]

if(PlayerInfo[playerid][pStranica] == 0) return 1;
if(PlayerInfo[playerid][pVsegostr] == listitem) PlayerInfo[playerid][pStranica] -= 8;  

Замени на:

if(PlayerInfo[playerid][pStranica] != 0)
{
if(PlayerInfo[playerid][pVsegostr] == listitem) PlayerInfo[playerid][pStranica] -= 8;
}  

Ничего не происходит.

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

Никто не знает?

[DeimoS]

Эмм, удивляюсь тому, как вообще раньше ваш код работал... Столько возвратов...

case 5859:
{
    if(!response) return ShowPlayerDialog(playerid,5858,DIALOG_STYLE_LIST,"OFFMEMBERS","- Весь состав\n- По рангу","Далее","Отмена");
    switch(listitem)
    {
        case 1..8:
        {
            if(!PlayerInfo[playerid][pStranica]) return 1;
            if(1 <= PlayerInfo[playerid][pVsegostr] <= 8) PlayerInfo[playerid][pPlayerInfo[playerid][pStranica]] -= 8;
        }
        case 9:
        {
            if(!PlayerInfo[playerid][pStranica]) PlayerInfo[playerid][pPlayerInfo[playerid][pStranica]] += 8;
            else
            {
                if(PlayerInfo[playerid][pVsegostr] == 9) PlayerInfo[playerid][pPlayerInfo[playerid][pStranica]] -= 8;
                else if(PlayerInfo[playerid][pVsegostr] == 10) PlayerInfo[playerid][pPlayerInfo[playerid][pStranica]] += 8;
            }
        }
        case 10: PlayerInfo[playerid][pPlayerInfo[playerid][pStranica]] -= 8;
    }
    if(!PlayerInfo[playerid][pOrderRank]) mysql_format(dbHandle, strings, sizeof(string), "SELECT * FROM `accounts` WHERE `Member` = '%d' LIMIT %d , 8", PlayerInfo[playerid][pLeader], PlayerInfo[playerid][pPlayerInfo[playerid][pStranica]]);
    else mysql_format(dbHandle, strings, 128, "SELECT * FROM `accounts` WHERE `Member` = '%d' ORDER BY `accounts`.`Rank` ASC LIMIT %d , 8", PlayerInfo[playerid][pLeader], PlayerInfo[playerid][pPlayerInfo[playerid][pStranica]]);
    mysql_function_query(dbHandle, strings, true, "OnPlayerSelectOffmembers", "i", playerid);
    ShowPlayerDialogUninvite(playerid, inputtext);
    return 1;
}  

[J-Ex]

Я не вкуриваю, откуда тут inputtext, если это dialog_style_list?

ShowPlayerDialogUninvite(playerid, inputtext);  

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

[DeimoS]

Я не вкуриваю, откуда тут inputtext, если это dialog_style_list?

ShowPlayerDialogUninvite(playerid, inputtext);  

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

Inputtext будет хранить содержание строки, которую выбрал игрок. То бишь, если будет такой диалог

1) Один
2) Два
3) Три  

И игрок нажмёт на второй пункт, подобны код:

if(listitem == 1) printf("%s",inputtext);  

отобразит в консоль "2) Два"

[J-Ex]

Это то ясно, я чёт туплю прост, логику кода не пойму =/

OnPlayerSelectOffmembers  

для полной картины не хватает :(