Оптимизация небольшого кода

Всем привет.
Каким образом возможно оптимизировать следующий код:

Код:

---

    if(strcmp(cmd, "/leaders3", true) == 0)
    {
        string[0] = 0;
        for(new i=0;i<MAX_PLAYERS;i++)
        {
            new fname[1024];
            if(!IsPlayerConnected(i) || IsPlayerNPC(i)) continue;
            if(PlayerInfo[i][pLeader] <= 0) continue;
            {
            if(PlayerInfo[i][pLeader] == 1) { fname = "FBI"; }
            else if(PlayerInfo[i][pLeader] == 2) { fname = "ФБР"; }
            else if(PlayerInfo[i][pLeader] == 3) { fname = "Национальная Гвардия"; }
            else if(PlayerInfo[i][pLeader] == 4) { fname = "Министерство Здравоохранение"; }
            else if(PlayerInfo[i][pLeader] == 5) { fname = "La Cosa Nostra"; }
            else if(PlayerInfo[i][pLeader] == 6) { fname = "Yakuza"; }
            else if(PlayerInfo[i][pLeader] == 7) { fname = "Правительство Штата"; }
            else if(PlayerInfo[i][pLeader] == 8) { fname = "Хитманы"; }
            else if(PlayerInfo[i][pLeader] == 9) { fname = "Репортеры"; }
            else if(PlayerInfo[i][pLeader] == 10) { fname = "Такси"; }
            else if(PlayerInfo[i][pLeader] == 11) { fname = "Инструкторы"; }
            else if(PlayerInfo[i][pLeader] == 13) { fname = "Grove"; }
            else if(PlayerInfo[i][pLeader] == 14) { fname = "El Coronos"; }
            else if(PlayerInfo[i][pLeader] == 15) { fname = "LS Vagos"; }
            else if(PlayerInfo[i][pLeader] == 16) { fname = "The Ballas"; }
            else if(PlayerInfo[i][pLeader] == 17) { fname = "Flaming Cobra"; }
            else if(PlayerInfo[i][pLeader] == 18) { fname = "Street Dogs"; }
            else if(PlayerInfo[i][pLeader] == 19) { fname = "SF Rifa"; }
            else if(PlayerInfo[i][pLeader] == 20) { fname = "Русская Мафия"; }
            }
            new num = PlayerInfo[i][pPnumber];
            GetPlayerName(i, sendername, sizeof(sendername));
            if(i == 0) format(string,sizeof(string),"Организация\tНикнейм\tНомер\n%s%s\t%s\t[%d]\n",string,fname,sendername,num);
            else format(string,sizeof(string),"%s%s\t%s\t[%d]\n",string,fname,sendername,num);
        }
        ShowPlayerDialog(playerid,13, DIALOG_STYLE_TABLIST_HEADERS, "{999999}Лидеры онлайн:", string, "OK", "");
        return 1;
    }

---

Что это вообще:

Код HTML:

---

[IMG]http://f3.s.qip.ru/WMlLzbn8.png[/IMG]

---

Если обнаружите ошибки - прошу указать на них.
Заранее благодарю за ответ! :blush2:

Из самого значимого - перевести условия if/else if на switch, а также исправить ошибку:

PHP код:

---

if(PlayerInfo[i][pLeader] <= 0) continue;
{ 


---

- можно создать массив и туда по порядочку записать названия фракций
- наверняка у вас есть глобальный массив с именами игроков, например,

PHP код:

---

PlayerInfo[playerid][pName] 


---

используйте его, тогда вам не надо будет в цикле получать имена игроков

- используйте foreach.inc
- так же посчитывайте размеры массивов. В массиве fname точно не 1024 символа

Не лучше название фракций взять в массив ?

PHP код:

---

#define MAX_ORGANIZATION (5)

static const organization_name[MAX_ORGANIZATION][] =
    {"Fraction_One", "Fraction_Two", "Fraction_Three", "Fraction_Four", "Fraction_Five"}; 


---

PHP код:

---

format(string, sizeof(string), "Вы лидер фракции \"%s\"", organization_name[playerid-1]);
SendClientMessage(playerid, -1, string); 


---

Цитата:

---

Сообщение от gangzone.ini

Не лучше название фракций взять в массив ?

PHP код:

---

#define MAX_ORGANIZATION (5)

static const organization_name[MAX_ORGANIZATION][] =
    {"Fraction_One", "Fraction_Two", "Fraction_Three", "Fraction_Four", "Fraction_Five"}; 


---

PHP код:

---

format(string, sizeof(string), "Вы лидер фракции \"%s\"", organization_name[playerid-1]);
SendClientMessage(playerid, -1, string); 


---

---

Неверный пример использования. Зачем от playerid отнимать единицу?

Как-то так :)

PHP код:

---

if(strcmp(cmd, "/leaders3", true) == 0)
{
    static const fraction_name_array[19][29] = 
    {
        "FBI",
        "ФБР",
        "Национальная Гвардия",
        "Министерство Здравоохранение",
        "La Cosa Nostra",
        "Yakuza",
        "Правительство Штата",
        "Хитманы",
        "Репортеры",
        "Такси",
        "Инструкторы",
        "Grove",
        "El Coronos",
        "LS Vagos",
        "The Ballas",
        "Flaming Cobra",
        "Street Dogs",
        "SF Rifa",
        "Русская Мафия"
    };
    // new string[размер];
    foreach(new i:Player)
    {
        format(
            string, sizeof(string),
            ((i == 0)?("Организация\tНикнейм\tНомер\n%s%s\t%s\t[%d]\n"):("%s%s\t%s\t[%d]\n")),
            string, fraction_name_array[PlayerInfo[i][pLeader]-1], PlayerInfo[i][pName], PlayerInfo[i][pPnumber]
        );

    }
    ShowPlayerDialog(playerid, 13, DIALOG_STYLE_TABLIST_HEADERS, "{999999}Лидеры онлайн:", string, "OK", "");
    return 1;
} 


---

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

Цитата:

---

Сообщение от Anton Styazhkin

Как-то так :)

PHP код:

---

if(strcmp(cmd, "/leaders3", true) == 0)
{
    static const fraction_name_array[19][29] = 
    {
        "FBI",
        "ФБР",
        "Национальная Гвардия",
        "Министерство Здравоохранение",
        "La Cosa Nostra",
        "Yakuza",
        "Правительство Штата",
        "Хитманы",
        "Репортеры",
        "Такси",
        "Инструкторы",
        "Grove",
        "El Coronos",
        "LS Vagos",
        "The Ballas",
        "Flaming Cobra",
        "Street Dogs",
        "SF Rifa",
        "Русская Мафия"
    };
    // new string[размер];
    foreach(new i:Player)
    {
        format(
            string, sizeof(string),
            ((i == 0)?("Организация\tНикнейм\tНомер\n%s%s\t%s\t[%d]\n"):("%s%s\t%s\t[%d]\n")),
            string, fraction_name_array[PlayerInfo[i][pLeader]-1], PlayerInfo[i][pName], PlayerInfo[i][pPnumber]
        );

    }
    ShowPlayerDialog(playerid, 13, DIALOG_STYLE_TABLIST_HEADERS, "{999999}Лидеры онлайн:", string, "OK", "");
} 


---

---

Огромное спасибо!
Всем удачи. :boast:

Цитата:

---

Сообщение от Anton Styazhkin

Как-то так :)

PHP код:

---

if(strcmp(cmd, "/leaders3", true) == 0)
{
    static const fraction_name_array[19][29] = 
    {
        "FBI",
        "ФБР",
        "Национальная Гвардия",
        "Министерство Здравоохранение",
        "La Cosa Nostra",
        "Yakuza",
        "Правительство Штата",
        "Хитманы",
        "Репортеры",
        "Такси",
        "Инструкторы",
        "Grove",
        "El Coronos",
        "LS Vagos",
        "The Ballas",
        "Flaming Cobra",
        "Street Dogs",
        "SF Rifa",
        "Русская Мафия"
    };
    // new string[размер];
    foreach(new i:Player)
    {
        format(
            string, sizeof(string),
            ((i == 0)?("Организация\tНикнейм\tНомер\n%s%s\t%s\t[%d]\n"):("%s%s\t%s\t[%d]\n")),
            string, fraction_name_array[PlayerInfo[i][pLeader]-1], PlayerInfo[i][pName], PlayerInfo[i][pPnumber]
        );

    }
    ShowPlayerDialog(playerid, 13, DIALOG_STYLE_TABLIST_HEADERS, "{999999}Лидеры онлайн:", string, "OK", "");
} 


---

---

return забыл.