Eingabe überprüfen (Integer)

evo-style

Dies wird aber nicht in die while-Anweisung geschrieben oder?

char c = _getch();
if(c == '1' ¦¦ c == '2' ¦¦ c == 3)
    std::cout << "Korrekte Eingabe";

daddy_felix

JIPOOP schrieb:

char c = _getch();
if(c == '1' ¦¦ c == '2' ¦¦ c == 3)
    std::cout << "Korrekte Eingabe";

falsch

evo-style

Wie muss dann dieses std::cin.clear()eingesetzt werden? Einfach in der Schalufe funktioniert es nicht.
Die Lösung mit nur 1,2 oder 3 zulassen find ich nicht wirklich gut, weill wenn ich eine Eingabe von 0-100 hätte könnte ich ja auch nicht jede Za^hl einzlen "ausschliessen".

Seh im Moment wirklich nicht durch, hoffe es kann mir jemand helfen.

Vielen Dank schon im voraus.

Nathan

Du machst folgendes:

int input = -1; // Oder irgendwas anders, "ungültiges"

while (!cin >> input) // Solange das Lesen von ints nicht klappt
{
    cerr << "Fehlerhafte Eingabe!" << endl;
    cin.clear();
}
// Input beschränkenm auf [min, max)
if (input < min)
    input = min;
else if (input >= max)
    input = max;
//input verwenden

evo-style

So, oder versteh ich was komplett falsch?

// Test.cpp: Hauptprojektdatei.

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <limits>

using namespace std;

void main()
{
   int input=-1;
   int min=0;
   int max=3;

	do	//In dieser Do-While-Schelife wird geprüft, ob die Eingabe gültig ist. Wenn nicht wird der Text wieder ausgegeben!
	{

		cout << "Welche Kanaele sollen gemessen werden?" << endl;
		cout << "Wahelen Sie 1 fuer Kanal 1" << endl;
		cout << "Wahelen Sie 2 fuer Kanal 2" << endl;
		cout << "Wahelen Sie 3 fuer Kanal 1+2" << endl;
		cout << "Wahelen Sie nun den entsprechenden Kanal : " << endl;

		cin >> input;

	}

	while (!cin >> input); // Solange das Lesen von ints nicht klappt 
{ 
    cerr << "Fehlerhafte Eingabe!" << endl; 
    cin.clear(); 
} 

	if (input < min) 
    input = min; 
	else if (input >= max) 
    input = max; 

	char ch=getch ();
}

evo-style schrieb:

So, oder versteh ich was komplett falsch?

Was ist jetzt genau das Problem an deinem Quellcode?

An alle anderen: Ich frage mich gerade... Wie stelle ich sicher, dass der Benutzer wirklich, wirklich nur eine Ganzzahl eingeben kann? while (!cin >> input)( klappt ja immer, wenn am Anfang eine Zahl steht... Mein schneller Ansatz wäre das hier? Aber das geht bestimmt noch besser?

#include <iterator>
#include <limits>
#include <string>
#include <sstream>
#include <cctype>
#include <iostream>
using namespace std;

int is_integer(const string& str)
{
	if( str.length()==0 )
		return numeric_limits<int>::min();

	const unsigned minus = str[0]=='-' ? 1 : 0;

	for(string::const_iterator it=str.begin()+minus; it!=str.end(); ++it)
	{
		if( !isdigit(*it) )
			return numeric_limits<int>::min();
	}

	istringstream iss(str);
	int integer;
	iss>>integer;
	return integer;
}

int main()
{
	int n;
	for(n=numeric_limits<int>::min(); n==numeric_limits<int>::min(); )
	{
		string str;
		getline(cin,str);
		n = is_integer(str);
	}

	cout << "Zahl: " << n;
}

SeppJ

out schrieb:

An alle anderen: Ich frage mich gerade... Wie stelle ich sicher, dass der Benutzer wirklich, wirklich nur eine Ganzzahl eingeben kann? while (!cin >> input)( klappt ja immer, wenn am Anfang eine Zahl steht... Mein schneller Ansatz wäre das hier? Aber das geht bestimmt noch besser?

Mein Tipp: Ist das wirklich wichtig? Das ist doch nicht die typische Anwendung der Streamoperatoren oder wie ein typisches Konsolenprogramm genutzt wird. Du baust mit viel Aufwand eine (Text-)GUI nach. Wenn man so etwas möchte, dann nehme man doch gleich ein fertiges GUI-Framework.
Ich halte das jedenfalls immer so, dass davon ausgegangen wird, dass der Benutzer weiß, was er tut. Wenn eine Eingabe fehlerhaft war, dann war sie eben fehlerhaft. Das Programm arbeitet dann (je nach Wichtigkeit der Eingabe) so weiter als ob die Eingabe das Streamende war oder es bricht ab.

P.S.: Natürlich kann man das mit viel Aufwand alles hinfummeln, aber das mag ich gerade nicht vormachen.

Nathan

evo-style schrieb:

So, oder versteh ich was komplett falsch?

// Test.cpp: Hauptprojektdatei.

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <limits>

using namespace std;

 

void main()
{
   int input=-1;
   int min=0;
   int max=3;

   

	do	//In dieser Do-While-Schelife wird geprüft, ob die Eingabe gültig ist. Wenn nicht wird der Text wieder ausgegeben!
	{
		
		cout << "Welche Kanaele sollen gemessen werden?" << endl;
		cout << "Wahelen Sie 1 fuer Kanal 1" << endl;
		cout << "Wahelen Sie 2 fuer Kanal 2" << endl;
		cout << "Wahelen Sie 3 fuer Kanal 1+2" << endl;
		cout << "Wahelen Sie nun den entsprechenden Kanal : " << endl;
		
		cin >> input;
	
		
	}

	while (!cin >> input); // Solange das Lesen von ints nicht klappt 
{ 
    cerr << "Fehlerhafte Eingabe!" << endl; 
    cin.clear(); 
} 

	if (input < min) 
    input = min; 
	else if (input >= max) 
    input = max; 

	char ch=getch ();
}

Lösche das cin.clear() und das cerr und füge am Anfang der do-while-Schleife cin.clear() ein.

@out:
Bei cin >> integer wird doch abgebrochen, sobald das erste nicht Zahl-zeichen ist (bei richtiger Benutzung bei Enter, bei falscher bei irgendeinem anderen char). Also wo ist dein Problem?

Nathan schrieb:

@out:
Bei cin >> integer wird doch abgebrochen, sobald das erste nicht Zahl-zeichen ist (bei richtiger Benutzung bei Enter, bei falscher bei irgendeinem anderen char). Also wo ist dein Problem?

int main()
{
	int input;
	while (!(cin >> input)) // Eingabe: 5.5
	{
		cerr << "Fehlerhafte Eingabe!" << endl;
		cin.clear();
	}

	double d;
	cin >> d; // 0.5
}

Wie handelst du das hier?

Nathan

Ich lösche vorher den Buffer mit cin.ignore()?

Nathan schrieb:

Ich lösche vorher den Buffer mit cin.ignore()?

Ja schon. Ich wollte damit nur sagen, dass eine eigentlich fehlerhafte Eingabe als gültig angesehen wird.