[Вопрос] Статус Email

Статус Email

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

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

PHP код:

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,"Выбрать","Назад"); }

Как вариант:

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 можно еще тернарный оператор применить.

Можно так:

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, "Выбрать", "Назад");
}

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

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

Цитата:

Сообщение от DeimoS

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]);

Цитата:
Сообщение от pawnoholic
Цвет должен форматироваться как HEX:
format(dialog,sizeof(dialog), "[1] Email: {%x}%s\n[2] Сменить пароль\n[3] Настройка кода безопасности", email_mes_color[dialog[0]], dialog[1]);
Эмм, в данном случае он записывается как строка, а не как число, поэтому там должен быть именно "%s". Для твоего варианта массив должен выглядеть так:

PHP код:

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

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

Спасибо.