Статус Email

[Dimon_Fanat]

Всем добрый день, помогите пожалуйста, как можно сделать вот этот код более нормальным?*

Суть кода хочется сделать если Еmail не указан то пишет Красным не указан, если не подтвержденый, то желтым пишет Email, ну и если подтвержденный то зеленым.*

stock AccountSecurity(playerid) 
{ 
    new dialog[-2+144],string[64]; 
    if(Player[playerid][pEmCheck] == 0)string = "{FF0000}Не указан"; 
    else if(Player[playerid][pEmCheck] == 1)string = "{FFFF00}%s",Player[playerid][pEmail]; 
    else if(Player[playerid][pEmCheck] == 2)string = "{00FF00}%s",Player[playerid][pEmail]; 
    format(dialog,sizeof(dialog), 
    "[1] Email: %s\n[2] Сменить пароль\n\ 
    [3] Настройка кода безопасности",string); 
    ShowPlayerDialog(playerid,DLG_KPK_ACCOUNT_SECURITY,2,"{33CCFF}КПК -> Аккаунт -> Безопасность",dialog,"Выбрать","Назад"); 
}  

[pawnoholic]

Как вариант:

stock AccountSecurity(playerid) {     new string[144] = "{FF0000}Не указан";     if (Player[playerid][pEmCheck] == 1)        format(string, sizeof(string), "{FFFF00}%s", Player[playerid][pEmail]);     if (Player[playerid][pEmCheck] == 2)        format(string, sizeof(string), "{00FF00}%s", Player[playerid][pEmail]);     format(string, sizeof(string), "[1] Email: %s\n[2] Сменить пароль\n[3] Настройка кода безопасности", string);      ShowPlayerDialog(playerid, DLG_KPK_ACCOUNT_SECURITY, DIALOG_STYLE_LIST, "{33CCFF}КПК -> Аккаунт -> Безопасность", string, "Выбрать", "Назад"); }

За место if можно еще тернарный оператор применить.

[DeimoS]

Можно так:

stock AccountSecurity(playerid) {     new dialog[144] = "_Не указано";      static email_mes_color[] =     {        0xFF0000,        0xFFFF00,        0x00FF00    };     if((dialog[0] = Player[playerid][pEmCheck]))    {        dialog[1] = 0;        strcat(dialog[1], Player[playerid][pEmail]);    }    format(dialog,sizeof(dialog), "[1] Email: {%x}%s\n[2] Сменить пароль\n[3] Настройка кода безопасности", email_mes_color[dialog[0]], dialog[1]);     ShowPlayerDialog(playerid, DLG_KPK_ACCOUNT_SECURITY, DIALOG_STYLE_LIST, "{33CCFF}КПК -> Аккаунт -> Безопасность", dialog, "Выбрать", "Назад");}

На компилируемость не проверял

 Старые варианты с цветом в виде строки

stock AccountSecurity(playerid) {     new dialog[144];      static email_mes_color[][] =     {        "FF0000",        "FFFF00",        "00FF00"    };     if(!(dialog[0] = Player[playerid][pEmCheck]))    {        strcat(dialog[1], "Не указано");    }    else    {        strcat(dialog[1], Player[playerid][pEmail]);    }    format(dialog,sizeof(dialog), "[1] Email: {%s}%s\n[2] Сменить пароль\n[3] Настройка кода безопасности", email_mes_color[dialog[0]], dialog[1]);     ShowPlayerDialog(playerid, DLG_KPK_ACCOUNT_SECURITY, DIALOG_STYLE_LIST, "{33CCFF}КПК -> Аккаунт -> Безопасность", dialog, "Выбрать", "Назад");}

На компилируемость не проверял

UPD: Или даже так

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

stock AccountSecurity(playerid) {     new dialog[144] = "_Не указано";      static email_mes_color[][] =     {        "FF0000",        "FFFF00",        "00FF00"    };     if((dialog[0] = Player[playerid][pEmCheck]))    {        dialog[1] = 0;        strcat(dialog[1], Player[playerid][pEmail]);    }    format(dialog,sizeof(dialog), "[1] Email: {%s}%s\n[2] Сменить пароль\n[3] Настройка кода безопасности", email_mes_color[dialog[0]], dialog[1]);     ShowPlayerDialog(playerid, DLG_KPK_ACCOUNT_SECURITY, DIALOG_STYLE_LIST, "{33CCFF}КПК -> Аккаунт -> Безопасность", dialog, "Выбрать", "Назад");}

[pawnoholic]

format(dialog,sizeof(dialog), "[1] Email: {%s}%s\n[2] Сменить пароль\n[3] Настройка кода безопасности", email_mes_color[dialog[0]], dialog[1]);

Встраиваемые цвета нужно форматировать как HEX:

format(dialog,sizeof(dialog), "[1] Email: {%x}%s\n[2] Сменить пароль\n[3] Настройка кода безопасности", email_mes_color[dialog[0]], dialog[1]);

[DeimoS]

Цвет должен форматироваться как HEX:

format(dialog,sizeof(dialog), "[1] Email: {%x}%s\n[2] Сменить пароль\n[3] Настройка кода безопасности", email_mes_color[dialog[0]], dialog[1]);

Эмм, в данном случае он записывается как строка, а не как число, поэтому там должен быть именно "%s". Для твоего варианта массив должен выглядеть так:

static email_mes_color[] =
{
    0xFF0000,
    0xFFFF00,
    0x00FF00
};  

P.S. Пока копировал код массива, заметил недостающую точку с запятой. Сейчас перепишу в вариант с "%x" заодно

[Dimon_Fanat]

Спасибо.