[Помогите!] Внутри

[Enginerr]

Вобщем кину часть кода:

int a;
a = 24001;
cout<<"Введите пароль: \n";

cin>>a;

if (a == 24001)
{

cout<<"Правильный пароль\n\n";


}

else
{
cout<<"Неправильный пароль !\n;
}


Если ввести буквы или к примеру очень большое число то программа пишет что "правильно" и начинается бесконечное повторение этого слова. В общем сбой.

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

[артём612]

Цитата: Сообщение от Enginerr Вобщем кину часть кода: int a; a = 24001; cout<<"Введите пароль: \n"; cin>>a; if (a == 24001) { cout<<"Правильный пароль\n\n"; } else { cout<<"Неправильный пароль !\n; } Если ввести буквы или к примеру очень большое число то программа пишет что "правильно" и начинается бесконечное повторение этого слова. В общем сбой. Как избежать этого и сделать так чтобы кроме указанного пароля программа не реагировала ни на какой другой пароль, будь то числовое или символьное. Возможно очень легкий и глупый вопрос, скорее так и есть...

Вы допустили ошибку в коде поэтому при считывании данных не своего типа пароль правильный
Для этого вам надо просто убрать строку a = 24001;
Ошибка в том что вы инициализируете переменную числом равным паролю
И когда вводится в консоле данные не её типа они просто не записываются
Переменная не меняется и равна паролю....

Если же вы хотите пароль из символов

int a[10] = {0};
cout<<"Введите пароль: \n";

cin>>&a;

if (a == 'jn24001jn')
{
cout<<"Правильный пароль\n\n";
}else {
cout<<"Неправильный пароль !\n;
}

[twice]

Цитата: Сообщение от артём612 int a[10] = {0}; cout<<"Введите пароль: \n"; cin>>&a; if (a == 'jn24001jn') { cout<<"Правильный пароль\n\n"; }else { cout<<"Неправильный пароль !\n; } ___

Точнее:

std::string pas;
cin >> pas;
if (!pas.compare("your_real_password"))
{
   std::cout << "Correct!"
}
else
{
   std::cout << "Wrong password !"
}

артём612, твой код содержит 3 ошибки.
1. Фиксированный малый объем буфера под пароль дает возможность к атаке через buffer-overflow
2. В cin нужно передать просто "a", а не адрес а. Или же &a[0]
3. Сравнение строк через "==" недопустимо и не работает так, как хотелось бы. Нужно использовать функции сравнения строк.

[Enginerr]

Всё заработало =)
Спасибо Артёму за разъяснение, а twice за улучшенный код.

Добавлено через 1 час 18 минут
Ещё вопрос, вот допустим мини код:

switch (a)

case1:
cout<<"Один \n";
break;

case 2:
cout<<"Два \n";
break;

default:
cout<<"Ошибка \n";
break;


Как тут защититься от переполнения буфера? Потому что у меня есть меню в программе и там аналогичное с паролем, если набрать длинные цифра или буквы то идёт сбой.

[twice]

Цитата: Сообщение от Enginerr Всё заработало =) Спасибо Артёму за разъяснение, а twice за улучшенный код. Добавлено через 1 час 18 минут Ещё вопрос, вот допустим мини код: switch (a) case1: cout<<"Один \n"; break; case 2: cout<<"Два \n"; break; default: cout<<"Ошибка \n"; break; Как тут защититься от переполнения буфера? Потому что у меня есть меню в программе и там аналогичное с паролем, если набрать длинные цифра или буквы то идёт сбой.

Тут все зависит от того, что за переменная а.
Если это строка, то данная конструкция не рабочая, надо использовать

const std::string case1 = "first";
const std::string case2 = "second";
const std::string case3 = "third";
...
std::string a;
...

if (!a.compare(case1)) //compare вернет 0 в случае совпадения строк
{
   //что-то делаем
}
else if (!a.compare(case2))
{
   //...
}
else if ...

Если а - это интегральный тип (целочисленный), то здесь единственное решение брать максимально большой тип __int64 (он же long long). Но с целочисленным можно использовать switch-case.