cin в while

[Rock145]

Доброго дня суток. Я начинающий программист на с++ и начал учить его не так давно. Недавно начал писать программку, но сразу же наткнулся на проблему. Мне надо сделать проверку числа при вводе. Нужно чтобы оно было целым. Все бы ничего, но код зависает в бесконечном цикле. Вроде бы все правильно... Помогите кто чем может))))

int _tmain(int argc, _TCHAR* argv[])
{	
	int a,b,c;
	bool success=false;

	cout<<("plz, enter integer \n");
	while (!success)
	{
		cin>>c;
		if (cin.good())
		{
			success=true;
			if (summsqr(a,b,c))
				cout<<(a)<<endl<<(b)<<endl;
			else
				cout<<("didn't find this kind of integers \n");

			system("pause");
		} else
		{
			cout<<("this value is wrong. plz, enter integer value\n");
		}
	}

	return 0;
}

P.S. конкретней зависает в while!

[bedman]

переменную c лучше сделать тип char, а потом пробовать парсить в инт

[Stilet]

надо сделать проверку числа при вводе.

А может лучше делать проверку после ввода?

[_Bers]

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

Доброго дня суток. Я начинающий программист на с++ и начал учить его не так давно. Недавно начал писать программку, но сразу же наткнулся на проблему. Мне надо сделать проверку числа при вводе. Нужно чтобы оно было целым. Все бы ничего, но код зависает в бесконечном цикле. Вроде бы все правильно... Помогите кто чем может))))

int _tmain(int argc, _TCHAR* argv[])
{	
	int a,b,c;
	bool success=false;

	cout<<("plz, enter integer \n");
	while (!success)
	{
		cin>>c;
		if (cin.good())
		{
			success=true;
			if (summsqr(a,b,c))
				cout<<(a)<<endl<<(b)<<endl;
			else
				cout<<("didn't find this kind of integers \n");

			system("pause");
		} else
		{
			cout<<("this value is wrong. plz, enter integer value\n");
                        cin.clear();
                        cin.rdbuf()->pubseekpos(0);
		}
	}

	return 0;
}

P.S. конкретней зависает в while!

Когда стрим ломается, у него выставляется флаг о внутренней ошибки. Этот флаг нужно почистить. При этом, несмотря на поломку, он уже зажевал какие то байты в свой внутренний буфер. Нужно "почистить буфер".
В данном случае, я просто перемотал позицию внутри этого буфера на начало. Новая запись затрет старую.

[_Bers]

Можно ещё вот так вот сделать:

    int c=0; bool success=false;
    cout<<("plz, enter integer \n");
    while (!success)
    {
        cin>>c;
        if (cin.good())
        {
            std::cout<<"c = "<<c<<endl; 
            success=true; 
            system("pause");
        }
        else
        {
            cout<<("this value is wrong. plz, enter integer value\n");
            cin.clear();    //очищаем флаги состояния механизма.
            fflush(stdin);  //ощищаем буфер от старых некорретных данных.
        }
    }

[kineziz]

Как то вы буфер очищаете по умному я так не видел еще. Как на счет cin.sync(), я всегда так очищал его или в этом если недостатки

[pproger]

2_Bers

            fflush(stdin);  //ощищаем буфер от старых некорретных данных.

нельзя fflush применять к потокам ввода.

2kineziz

Как то вы буфер очищаете по умному я так не видел еще

умного в этом ничего нет криво и непортабельно.

вон правильно говорят. читай в строку, проверяй, что это нужная тебе последовательность символов. все анализаторы входных данных в консольных приложениях так и поступают. никто не пишет scanf("%d", &i) или cin >> i, ибо эти вещи чреваты ошибками. это можно писать только в тех случаях, когда ты уверен, что будет введено число (например, читаешь из структурированного файла). но даже тут нужно проверять на ошибки. вот в таких случаях и нужны все эти методы good() bad() и т.д. в твоем же - просто читай строку.

[_Bers]

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

2_Bers

            fflush(stdin);  //ощищаем буфер от старых некорретных данных.

нельзя fflush применять к потокам ввода.

Почему?

И кстати, накой парсить строчку, если всегда можно проверить флаг, и зачистить буфер стрима? По-моему, парсить строчку в этом случае - мартышкин труд.

[_Bers]

гцц/мингв/студия прожевали, и ничо так. В чем именно заключается не портабельность?

[pproger]

гцц/мингв/студия прожевали, и ничо так. В чем именно заключается не портабельность?

если код скомпилился и даже работает в отдельных случаях, это не значит, что он правильный (в данном случае портабельный).
криво - потому что ты использовал c-style stream в с++. непортабельно - вызов fflush на поток ввода - неопределенное поведение

man fflush
...
The standards do not specify the behavior for input streams.

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

И кстати, накой парсить строчку, если всегда можно проверить флаг, и зачистить буфер стрима? По-моему, парсить строчку в этом случае - мартышкин труд.

по вышеназванной причине.