Проверка 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 Максимальный 542]

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

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

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

Цитата

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

 

[kala4ik Хороший 1]

1 час назад, MuhammadPawn сказал:

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

 

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

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

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

 

 

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