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

Написал я диалог авторизации(впервые в жизни :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, так как вы сравниваете один символ с буквой из алфавита и тут же с цифрой. Самый первый вариант автора самый рациональный.

Цитата:

---

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

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

---

Можно использовать одну переменную static len = strlen(input); и подставлять вместо вызова тойже функи в разных частях кода)

Цитата:

---

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

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

---

Есть такая таблица ASCII в которой некоторым символам сопоставлены числовые коды. Именно на основе этой таблицы выполняется проверка ниже:

PHP код:

---

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


---

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

PHP код:

---

switch(inputtext[i])  
        {  
            case '0'..'9', 'a'..'z', 'A'..'Z': continue;  
            default: return ShowPlayerDialog(playerid, 1, DIALOG_STYLE_INPUT, "Авторизация", "{FFFFFF}Введённый пароль содержит запрещённые символы!\nВведите пароль от аккаунта для входа на сервер:", "Вход", "Выход");  
        } 


---

Если case переписать используя if он будет выглядеть примерно так:

PHP код:

---

case '0'..'9':
for(new j = 0; j < 10; j++)
    if(inputtext[i] == i) return true;
'a'..'z':
for(new j = 97; j < 123; j++) // коды в таблице ASCII 97 - a, 122 - z
    if(inputtext[i] == i) return true;
'A'..'Z:
for(new j = 65; j < 91; j++)  // коды в таблице ASCII 65 - A, 90 - Z
    if(inputtext[i] == i) return true; 


---

Итого теоретически по 62 проверки на каждый символ массива.
В том варианте который предложил я выполняется проверка на диапазон, находится ли символ в диапазоне от до, всего 3 проверки.

Цитата:

---

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

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

---

 Спойлер: картинка

https://pp.vk.me/c608720/v608720642/...P15dwdRyZ0.jpg

- - - Добавлено - - -

Цитата:

---

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

Есть такая таблица ASCII в которой некоторым символам сопоставлены числовые коды. Именно на основе этой таблицы выполняется проверка ниже:

PHP код:

---

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


---

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

PHP код:

---

switch(inputtext[i])  
        {  
            case '0'..'9', 'a'..'z', 'A'..'Z': continue;  
            default: return ShowPlayerDialog(playerid, 1, DIALOG_STYLE_INPUT, "Авторизация", "{FFFFFF}Введённый пароль содержит запрещённые символы!\nВведите пароль от аккаунта для входа на сервер:", "Вход", "Выход");  
        } 


---

Если case переписать используя if он будет выглядеть примерно так:

PHP код:

---

case '0'..'9':
for(new j = 0; j < 10; j++)
    if(inputtext[i] == i) return true;
'a'..'z':
for(new j = 97; j < 123; j++) // коды в таблице ASCII 97 - a, 122 - z
    if(inputtext[i] == i) return true;
'A'..'Z:
for(new j = 65; j < 91; j++)  // коды в таблице ASCII 65 - A, 90 - Z
    if(inputtext[i] == i) return true; 


---

Итого теоретически по 62 проверки на каждый символ массива.
В том варианте который предложил я выполняется проверка на диапазон, находится ли символ в диапазоне от до, всего 3 проверки.

---

Да ты новый DeimoS. Так же все объясняешь и расписываешь :good:

Цитата:

---

{FFFFFF}Добро пожаловать на сервер - 34
Ваш ник зарегистрирован - 23
Логин: {1DBF3A}%s{FFFFFF} - 47
{FF0000}Неверный пароль! Осталось попыток: %d - 44

---

new string[148];

Цитата:

---

Сообщение от gangzone.ini

new string[148];

---

Раз уж пошло на это дело, то 150; Так как 34+23+48+44+1=150 :dirol:
И не хочу повторяться, этот текст пробный :)

Решено
(закрывай тему)