Оптимизация?

[Daniel_Cortez]

DeimoS, Это только 2 ранга, а у меня их на 7 выйдет, если даже сделать на 7 рангов будет лучше чем вообще без switch?

Выложите полный код команды, или хотя бы тот, в котором определяются требования к ID организации и рангу. Из ваших слов вообще ничего не понятно.

[vovandolg]

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

    if(strcmp(cmd, "/giverank", true) == 0)
    {
        new memb = PlayerInfo[playerid][pMember];
        new rank = PlayerInfo[playerid][pRank];

        if(memb == 1 && rank >= 14 || memb == 3 && rank >= 14 || memb == 10 && rank >= 14 || memb == 19 && rank >= 14
        || memb == 21 && rank >= 14 || memb == 2 && rank >= 9 || memb == 26 && rank >= 9 || memb == 4 && rank >= 9
        || memb == 5 && rank >= 9 || memb == 6 && rank >= 9 || memb == 11 && rank >= 9 || memb == 12 && rank >= 9
        || memb == 13 && rank >= 9 || memb == 14 && rank >= 9 || memb == 15 && rank >= 9 || memb == 17 && rank >= 9
        || memb == 18 && rank >= 9 || memb == 22 && rank >= 9 || memb == 24 && rank >= 9 || memb == 25 && rank >= 9
        || memb == 9 && rank >= 13 || memb == 16 && rank >= 13 || memb == 20 && rank >= 13 || memb == 7 && rank >= 5
        || memb == 27 && rank >= 5 || memb == 8 && rank >= 5 || memb == 28 && rank >= 5 || memb == 29 && rank >= 6
        || memb == 23 && rank >= 11 || memb == 30 && rank >= 6)
        {
            tmp = strcharsplit(cmdtext, idx);
             if(!strlen(tmp))
                return SendClientMessage(playerid, COLOR_WHITE, "Наберите: /giverank [id] [ðàíã]");
            new para1 = ReturnUser(tmp);
            tmp = strcharsplit(cmdtext, idx);
            new level = strval(tmp);
            switch(memb)
            {
            case 8: if(level > 4 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 4!");
            case 7,27,28: if(level > 5 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 5!");
            case 29,30: if(level > 6 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 6!");
            case 2,4..6,11..15,17,18,22,24..26: if(level > 9 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 9!");
            case 23: if(level > 11 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 11!");
            case 16: if(level > 12 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 12!");
            case 1,3,10,19,21: if(level > 14 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 14!");
            }
            if(!IsPlayerConnected(para1) || para1 == INVALID_PLAYER_ID || para1 == playerid) return true;
            if (PlayerInfo[para1][pLeader] >= 1) return SendClientMessage(playerid, COLOR_GREY, "Он лидер!");
            if(memb != PlayerInfo[para1][pMember]) return SendClientMessage(playerid, COLOR_GREY, "Он не в вашей фраке!");
            if(memb > 0 && rank <= level) return SendClientMessage(playerid,COLOR_GREY,"Îøèáêà");
            if(PlayerInfo[playerid][pLeader] <= 0 && rank <=  PlayerInfo[para1][pRank]) return SendClientMessage(playerid, COLOR_GREY, "Ошибка");
            format(stringer, sizeof(stringer), "Вас повысил/понизил %s до %d ранга",level, PlayerInfo[playerid][pNames]);
            SendClientMessage(para1, 0x6BB3FFAA, stringer);
            format(stringer, sizeof(stringer), "Вы повысили %s до %d ранга.", PlayerInfo[para1][pNames],level);
            SendClientMessage(playerid, 0x6BB3FFAA, stringer);
            PlayerInfo[para1][pRank] = level;
            return true;
        }
        else return SendClientMessage(playerid,COLOR_GREY,"Вам не доступна команда");
    }  

Вот допустим)

[NewGreen]

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

    if(strcmp(cmd, "/giverank", true) == 0)
    {
        new memb = PlayerInfo[playerid][pMember];
        new rank = PlayerInfo[playerid][pRank];

        if(memb == 1 && rank >= 14 || memb == 3 && rank >= 14 || memb == 10 && rank >= 14 || memb == 19 && rank >= 14
        || memb == 21 && rank >= 14 || memb == 2 && rank >= 9 || memb == 26 && rank >= 9 || memb == 4 && rank >= 9
        || memb == 5 && rank >= 9 || memb == 6 && rank >= 9 || memb == 11 && rank >= 9 || memb == 12 && rank >= 9
        || memb == 13 && rank >= 9 || memb == 14 && rank >= 9 || memb == 15 && rank >= 9 || memb == 17 && rank >= 9
        || memb == 18 && rank >= 9 || memb == 22 && rank >= 9 || memb == 24 && rank >= 9 || memb == 25 && rank >= 9
        || memb == 9 && rank >= 13 || memb == 16 && rank >= 13 || memb == 20 && rank >= 13 || memb == 7 && rank >= 5
        || memb == 27 && rank >= 5 || memb == 8 && rank >= 5 || memb == 28 && rank >= 5 || memb == 29 && rank >= 6
        || memb == 23 && rank >= 11 || memb == 30 && rank >= 6)
        {
            tmp = strcharsplit(cmdtext, idx);
             if(!strlen(tmp))
                return SendClientMessage(playerid, COLOR_WHITE, "Наберите: /giverank [id] [ðàíã]");
            new para1 = ReturnUser(tmp);
            tmp = strcharsplit(cmdtext, idx);
            new level = strval(tmp);
            switch(memb)
            {
            case 8: if(level > 4 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 4!");
            case 7,27,28: if(level > 5 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 5!");
            case 29,30: if(level > 6 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 6!");
            case 2,4..6,11..15,17,18,22,24..26: if(level > 9 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 9!");
            case 23: if(level > 11 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 11!");
            case 16: if(level > 12 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 12!");
            case 1,3,10,19,21: if(level > 14 || level < 1) return SendClientMessage(playerid, COLOR_GREY, "С 1 по 14!");
            }
            if(!IsPlayerConnected(para1) || para1 == INVALID_PLAYER_ID || para1 == playerid) return true;
            if (PlayerInfo[para1][pLeader] >= 1) return SendClientMessage(playerid, COLOR_GREY, "Он лидер!");
            if(memb != PlayerInfo[para1][pMember]) return SendClientMessage(playerid, COLOR_GREY, "Он не в вашей фраке!");
            if(memb > 0 && rank <= level) return SendClientMessage(playerid,COLOR_GREY,"Îøèáêà");
            if(PlayerInfo[playerid][pLeader] <= 0 && rank <=  PlayerInfo[para1][pRank]) return SendClientMessage(playerid, COLOR_GREY, "Ошибка");
            format(stringer, sizeof(stringer), "Вас повысил/понизил %s до %d ранга",level, PlayerInfo[playerid][pNames]);
            SendClientMessage(para1, 0x6BB3FFAA, stringer);
            format(stringer, sizeof(stringer), "Вы повысили %s до %d ранга.", PlayerInfo[para1][pNames],level);
            SendClientMessage(playerid, 0x6BB3FFAA, stringer);
            PlayerInfo[para1][pRank] = level;
            return true;
        }
        else return SendClientMessage(playerid,COLOR_GREY,"Вам не доступна команда");
    }  

Вот допустим)

if(strcmp(cmd, "/giverank", true) == 0)
    {
        new memb = PlayerInfo[playerid][pMember];
        new rank = PlayerInfo[playerid][pRank];

        if(memb > 0 && rank > 0)
        {
// ... продолжение кода  

p.s. переменные memb и rank лишние, можно напрямую обращаться к массиву PlayerInfo

[vovandolg]

А не чего что я овер9000 строк превратил в пару строк?
так для себя скажем) чтобы не листать не искать и проще было

[VVWVV]

if(strcmp(cmd, "/giverank", true) == 0)
    {
        new memb = PlayerInfo[playerid][pMember];
        new rank = PlayerInfo[playerid][pRank];

        if(memb > 0 && rank > 0)
        {
// ... продолжение кода  

p.s. переменные memb и rank лишние, можно напрямую обращаться к массиву PlayerInfo

Для того, чтобы двадцать раз не обращаться к массиву, сделан буфер (две переменные), который хранит в себе одно значение. Убедится в этом вы можете, посмотрев листинг ассемблера.

[NewGreen]

Для того, чтобы двадцать раз не обращаться к массиву, сделан буфер (две переменные), который хранит в себе одно значение. Убедится в этом вы можете, посмотрев листинг ассемблера.

Раздаю вредные советы