Проверка Email.

[kala4ik Хороший 1]

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

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

 

stock AccountSecurity(playerid)
{
	new dialog[-2+64],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,"Выбрать","Назад");
}

 

Отредактировано 1 мая, 2018 пользователем kala4ik

[MuhammadPawn Отличный 248]

это мало ещё надо менять внекоторых местах код что-бы ты хотел как ты описал 

[Cawfee Максимальный 558]

@MuhammadPawn тут уже все реализовано, автору лишь нужна оптимизация кода.

[MuhammadPawn Отличный 248]

@Elrmrnt-Kritik а теперь внимательно читай 

  Цитата

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

Читать далее...  

 

[kala4ik Хороший 1]

  В 5/1/2018 в 16:16, MuhammadPawn сказал:

@Elrmrnt-Kritik а теперь внимательно читай 

 

Читать далее...  

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

[DEST Максимальный 1 032]

  В 5/1/2018 в 10:52, kala4ik сказал:

хочется сделать

Читать далее...  

хочется сделать - это когда этого еще нет, а нужно. Тут код готовый есть. Вы просто сначала указываете одно, а потом другое. Тут уже кто какие строки прочитал

Если по теме, то тут можно разве немного причесать код: 

  Код (открыть спойлер)

stock AccountSecurity(playerid)
{
    new dialog[69+8+32];

    switch(Player[playerid][pEmCheck])
    {
        case 0:
          dialog = "{FF0000}Не указан";

        case 1:
          format(dialog, sizeof dialog, "{FFFF00}%s", Player[playerid][pEmail]);

        case 2:
          format(dialog, sizeof dialog, "{00FF00}%s", Player[playerid][pEmail]);
    }

    format(dialog, sizeof dialog, "\
          [1] Email: %s\n\
          [2] Сменить пароль\n\
          [3] Настройки кода безопасности", dialog);
    ShowPlayerDialog(playerid,DLG_KPK_ACCOUNT_SECURITY,2,"{33CCFF}КПК -> Аккаунт -> Безопасность",dialog,"Выбрать","Назад");
}

 

 

можно, конечно, и через тернарный оператор, но это для истинных извращенцев