Возможно ли оптимизировать данный код?

Написал я диалог авторизации(впервые в жизни :3) и мне код показался громоздким, вопрос, "оптимизировать возможно и что именно тут надо переписывать?"

PHP код:

---

case 2: 
{ 
    if(!strlen(inputtext)) return ShowPlayerDialog(playerid, 2, DIALOG_STYLE_INPUT,"Авторизация","{FFFFFF}Вы не можете продолжить авторизацию не введя пароль!\nВведите пароль от аккаунта для входа на сервер:", "Вход", "Выход"); 
    for(new i = strlen(inputtext)-1; i != -1; i--) 
    { 
        switch(inputtext[i]) 
        { 
            case '0'..'9', 'a'..'z', 'A'..'Z': continue; 
            default: return ShowPlayerDialog(playerid, 1, DIALOG_STYLE_INPUT, "Авторизация", "{FFFFFF}Введённый пароль содержит запрещённые символы!\nВведите пароль от аккаунта для входа на сервер:", "Вход", "Выход"); 
        } 
    } 
    if(!strcmp(pInfo[playerid][pPassword], inputtext)) 
    { 
        new query_string[52+MAX_PLAYER_NAME]; 
        format(query_string, sizeof(query_string), "SELECT * FROM `gamers` WHERE `name` = '%s'", pInfo[playerid][pName]); 
        mysql_function_query(mysql_variable, query_string, true, "OnPlayerLogin","i", playerid); 
    } 
    else 
    {
        SetPVarInt(playerid, "WrongPass", GetPVarInt(playerid, "WrongPass") +1);
        if(GetPVarInt(playerid, "WrongPass") == 4) return ShowPlayerDialog(playerid,1001,DIALOG_STYLE_MSGBOX,"{E6B122} Лимит попыток авторизации","{FFFFFF} Вы ввели неправильный пароль 3 раза подряд.","Закрыть",""), Kick(playerid);//KickFix(playerid, COLOR_WHITE, "{FF6347}При неправильном вводе пароля Вы будете забанены");
        new logtires = 4 - GetPVarInt(playerid, "WrongPass");
        new string[500];
        format(string,sizeof(string),"{FFFFFF}Добро пожаловать на сервер\nВаш ник зарегистрирован\n\nЛогин: {1DBF3A}%s{FFFFFF}\n{FF0000}Неверный пароль! Осталось попыток: %d", pInfo[playerid][pName], logtires);
        ShowPlayerDialog(playerid,2,DIALOG_STYLE_INPUT,"Авторизация",string, "Войти","Отмена");
        return true;
    }
    return 1; 
} 


---

В формате стринга явно не задействовано 500 ячеек.

Цитата:

---

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

В формате стринга явно не задействовано 500 ячеек.

---

Мне просто лень отсчитывать, так как текст пробный.

Все молчат и это значт что? Да то, что все у меня хорошо :dirol:

PHP код:

---

    if(inputtext[0] == '\0') return ShowPlayerDialog(playerid, 2, DIALOG_STYLE_INPUT,"Авторизация","{FFFFFF}Вы не можете продолжить авторизацию не введя пароль!\nВведите пароль от аккаунта для входа на сервер:", "Вход", "Выход");  
    for(new i = strlen(inputtext)-1; i != -1; i--)  
    {  
        if(inputtext[i] >= 0 && inputtext[i] <= 9 || inputtext[i] >= 'a' && inputtext[i] <= 'Z') continue;
        else return ShowPlayerDialog(playerid, 1, DIALOG_STYLE_INPUT, "Авторизация", "{FFFFFF}Введённый пароль содержит запрещённые                                         символы!\nВведите пароль от аккаунта для входа на сервер:", "Вход", "Выход");  
    } 


---

Цитата:

---

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

PHP код:

---

    if(inputtext[0] == '\0') return ShowPlayerDialog(playerid, 2, DIALOG_STYLE_INPUT,"Авторизация","{FFFFFF}Вы не можете продолжить авторизацию не введя пароль!\nВведите пароль от аккаунта для входа на сервер:", "Вход", "Выход");  
    for(new i = strlen(inputtext)-1; i != -1; i--)  
    {  
        if(inputtext[i] >= 0 && inputtext[i] <= 9 || inputtext[i] >= 'a' && inputtext[i] <= 'Z') continue;
        else return ShowPlayerDialog(playerid, 1, DIALOG_STYLE_INPUT, "Авторизация", "{FFFFFF}Введённый пароль содержит запрещённые                                         символы!\nВведите пароль от аккаунта для входа на сервер:", "Вход", "Выход");  
    } 


---

---

Ну практически хорошо написал я. Все равно впервые пишу мод :)

---

А это значит:

PHP код:

---

'a' && inputtext[i] <= 'Z' 


---

типа a..z, A..Z ? Просто я даже не думал, что можно начать от маленькой до заглавной буквы алфавита. :blush2:

Цитата:

---

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

Ну практически хорошо написал я. Все равно впервые пишу мод :)

---

А это значит:

PHP код:

---

'a' && inputtext[i] <= 'Z' 


---

типа a..z, A..Z ? Просто я даже не думал, что можно начать от маленькой до заглавной буквы алфавита. :blush2:

---

Запрещенный приём))
Я немного подзабыл позицию этих букв в таблице кодов, и то что между ними есть еще другие символы. Поэтому лучше все же использовать так:

PHP код:

---

if(inputtext[i] >= 0 && inputtext[i] <= 9 || inputtext[i] >= 'a' &&  inputtext[i] <= 'z ' || inputtext[i] >= 'A' && inputtext[i] <= 'Z') continue; 


---

Компилятор проклинает этот код :wizard:

PHP код:

---

if(inputtext[i] >= 0 && inputtext[i] <= 9 || inputtext[i] >= 'a' &&  inputtext[i] <= 'z ' || inputtext[i] >= 'A' && inputtext[i] <= 'Z') continue; 


---

Код HTML:

---

new.pwn(235) : error 027: invalid character constant
new.pwn(235) : warning 215: expression has no effect
new.pwn(235) : error 001: expected token: ";", but found ")"
new.pwn(235) : error 029: invalid expression, assumed zero
new.pwn(235) : fatal error 107: too many error messages on one line

---

Цитата:

---

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

Компилятор проклинает этот код :wizard:

PHP код:

---

if(inputtext[i] >= 0 && inputtext[i] <= 9 || inputtext[i] >= 'a' &&  inputtext[i] <= 'z ' || inputtext[i] >= 'A' && inputtext[i] <= 'Z') continue; 


---

Код HTML:

---

new.pwn(235) : error 027: invalid character constant
new.pwn(235) : warning 215: expression has no effect
new.pwn(235) : error 001: expected token: ";", but found ")"
new.pwn(235) : error 029: invalid expression, assumed zero
new.pwn(235) : fatal error 107: too many error messages on one line

---

---

Вот что значит долго не писать код)) забыл расставить скобки, извиняюсь))

PHP код:

---

if((inputtext[i] >= 0 && inputtext[i] <= 9) || (inputtext[i] >= 'a' &&  inputtext[i] <= 'z') || (inputtext[i] >= 'A' && inputtext[i] <= 'Z')) continue; 


---

Дело не в скобках. Ошибки выдаёт на цифры 0 и 9, так как вы сравниваете один символ с буквой из алфавита и тут же с цифрой. Самый первый вариант автора самый рациональный.