Abfangen von falschen cin Eingaben?

[freaky]

Heinzelmännchen! schrieb:

wofür das ignore?

Wofür ignore? Probiers halt aus und schau was passiert. Tipp : Stichwort Endlosschleife...

[unskilled]

FreakY<3Cpp schrieb:

Heinzelmännchen! schrieb:

wofür das ignore?

Wofür ignore? Probiers halt aus und schau was passiert. Tipp : Stichwort Endlosschleife...

.. oder lies draveres post noch mal...

[blade1982]

So hier mal meine Lösung:
(double spannung; double temperatur, int prozess)

void CFaktoren::bestimmeFaktoren() {
   bool valid;
   do {
   cout << "Bitte geben sie die drei folgenden Parameter ein:" <<endl;
   cout << "Spannung (Werte zwischen 1.08V und 1.32V): ";
   if (cin >> spannung){}
   else {
      cin.clear();
      cin.ignore();
      spannung = -100;
   }
   cout << "Temperatur (Werte zwischen -25C und 125C) : ";
   if (cin >> temperatur){}
   else {
      cin.clear();
      cin.ignore();
      temperatur = -100;
   }
   cout << "Prozess (Werte koennen nur 1=slow 2=typical 3=fast sein) : ";
   if( cin >> prozess){}
   else {
      cin.clear();
      cin.ignore();
      prozess = 100;
   }
   if( spannung > 1.32 || 
       spannung < 1.08 || 
       temperatur > 125 ||
       temperatur < -25 ||
       prozess > 3 || 
       prozess < 1 ){
       valid = false;
       cout << endl;
       cout << "Achtung sie haben Paramter eingegeben die ausserhalb der Spezifikationen liegen" <<endl;
       cout << "Bitte geben sie die Werte erneut ein" <<endl;
       cout << endl;
   }
   else
      valid = true;   
   }
   while( valid == false ); 
   berechneSpannungsFaktor();
   berechneTemperaturFaktor();
   berechneProzessFaktor();
   ausgabeFaktoren();
   system("pause");
}

Es mag bestimmt nicht der schnellste oder optimalste Code sein, aber es sollte nun sein Zweck erfüllen.
Danke an alle für die guten Tips und Vorschläge.

[freaky]

Dein Code geht nicht. Gebe ich bei der ersten Eingabe "erge" ein, überspringt er ALLE Eingaben und ich bekomme alle Ausgaben in die Konsole. Erst am Ende sagt er, das die Eingabe falsch war. Schau dir mein Code an, den ich auf der ersten Seite gepostet habe:

FreakY<3Cpp schrieb:

#include <iostream>

int main()
{
    int eingabe;
    while(!(std::cin >> eingabe))
    {
        std::cout << "Falsche Eingabe!" << std::endl;
        std::cin.clear();
        std::cin.ignore(std::numeric_limits<int>::max(),'\n');
    }
    std::cout << eingabe;
}

Benutze ihn auch so wie er da steht, denn er ist richtig.

[blade1982]

Hmm, ich habs wohl nicht richtig getestet....
1 zu 1 kann ich das eh nicht übernehmen, aber kannst du mir bitte die Zeile 10 erläutern? Die Parameter von Ignore erklären sich mir nicht so recht.
MFG Blade

[Dravere]

Die Zeile 10 sollte eigentlich so aussehen:

std::cin.ignore(std::cin.rdbuf()->in_avail());

Hier werden einfach so viele Zeichen ignoriert ( ignore ), wie es noch im Puffer ( rdbuf holt den Puffer) des Streams hat ( in_avail gibt die Menge zurück).

Da es aber anscheinend Implementationen von der Standardbibliothek gibt, welche dies nicht korrekt unterstützen, gibt es da einen Workaround und das ist die Lösung von FreakY<3Cpp.
Dort ignoriert man die maximale mögliche Anzahl an Zeichen oder bis zu und mit einem Linefeed. Die Konsoleneingaben werden grundsätzlich alle mit einem Linefeed abgeschlossen, da der Benutzer <Enter> gedrückt hat, zum bestätigen der Eingabe.
Ganz korrekt wäre hier eigentlich:

std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

std::numeric_limits ist eine Hilfsklasse, bzw. eine sogenannte Traitsklasse, um Informationen über den numerischen Typ zu erhalten:
http://www.cplusplus.com/reference/std/limits/numeric_limits/

War das verständlich genug?

Grüssli

[freaky]

http://www.c-plusplus.net/forum/viewtopic-var-t-is-243173-and-highlight-is-ignore.html
Da habe ich es schonmal gepostet und doch, du kannst ihn 1 zu 1 übernehmen.

void CFaktoren::bestimmeFaktoren() 
{
    bool valid;
    do 
    {
        cout << "Bitte geben sie die drei folgenden Parameter ein:" <<endl;
        cout << "Spannung (Werte zwischen 1.08V und 1.32V): ";
        while(!(cin >> spannung))
        {
            cout << "Falsche Eingabe!" << endl;
            cin.clear();
            cin.ignore(numeric_limits<int>::max(),'\n');
        }
        cout << "Temperatur (Werte zwischen -25C und 125C) : ";
        while(!(cin >> temperatur))
        {
            cout << "Falsche Eingabe!" << endl;
            cin.clear();
            cin.ignore(numeric_limits<int>::max(),'\n');
        }
        cout << "Prozess (Werte koennen nur 1=slow 2=typical 3=fast sein) : ";
        while(!(cin >> prozess))
        {
            cout << "Falsche Eingabe!" << endl;
            cin.clear();
            cin.ignore(numeric_limits<int>::max(),'\n');
        }
        if(spannung > 1.32 || spannung < 1.08 ||
           temperatur > 125 || temperatur < -25 ||
           prozess > 3 || prozess < 1 )
        {
           valid = false;
           cout << endl;
           cout << "Achtung sie haben Paramter eingegeben die ausserhalb der Spezifikationen liegen" <<endl;
           cout << "Bitte geben sie die Werte erneut ein" <<endl;
           cout << endl;
        }
        else
            valid = true;  
    }
    while( valid == false );
    berechneSpannungsFaktor();
    berechneTemperaturFaktor();
    berechneProzessFaktor();
    ausgabeFaktoren();
    system("pause");
}

[blade1982]

Hier mal in mein Programm eingeführt:
Das (numeric_limits<int>::max(),'\n') führt zu einen kompilierer Fehler "bezeichner numeric_limits nicht gefunden"

void CFaktoren::bestimmeFaktoren() {
   bool valid;
   do {
   cout << "Bitte geben sie die drei folgenden Parameter ein:" <<endl;
   cout << "Spannung (Werte zwischen 1.08V und 1.32V): ";
   while(! (cin >> spannung)){
      cin.clear();
      cin.ignore(cin.rdbuf()->in_avail());	
   }
   cout << "Temperatur (Werte zwischen -25C und 125C) : ";
      while(! (cin >> temperatur)){
      cin.clear();
      cin.ignore(cin.rdbuf()->in_avail());	
   }
   cout << "Prozess (Werte koennen nur 1=slow 2=typical 3=fast sein) : ";
   while(! (cin >> prozess)){
      cin.clear();
      cin.ignore(cin.rdbuf()->in_avail());
   }
   if( spannung > 1.32 || 
       spannung < 1.08 || 
       temperatur > 125 ||
       temperatur < -25 ||
       prozess > 3 || 
       prozess < 1 ){
       valid = false;
       cout << endl;
       cout << "Achtung sie haben Paramter eingegeben die ausserhalb der Spezifikationen liegen" <<endl;
       cout << "Bitte geben sie die Werte erneut ein" <<endl;
       cout << endl;
   }
   else
      valid = true;   
   }
   while( valid == false ); 
   berechneSpannungsFaktor();
   berechneTemperaturFaktor();
   berechneProzessFaktor();
   ausgabeFaktoren();
   system("pause");
}

Leider ist das Programm so nicht lauffähig, kompiliert wird es zwar aber es macht nicht mehr das was es sollte.
MFG Blade

[freaky]

Dann schreib mal std:: vor numeric_limits. Und was genau macht es nicht was es aber machen sollte?

[blade1982]

So ist sind nun schon 9 kompilierer Fehler:

error C2039: 'numeric_limits': Ist kein Element von 'std'
error C2065: 'numeric_limits': nichtdeklarierter Bezeichner
error C2062: 'int'-Typ unerwartet
Und die fehler jeweils für spannung, temperatur und prozess.
Der namespace ist im übrigen mit using namespace std; schon abgedeckt.

void CFaktoren::bestimmeFaktoren() {
   bool valid;
   do {
   cout << "Bitte geben sie die drei folgenden Parameter ein:" <<endl;
   cout << "Spannung (Werte zwischen 1.08V und 1.32V): ";
   while(! (cin >> spannung)){
      cin.clear();
      cin.ignore(std::numeric_limits<int>::max(),'\n');	
   }
   cout << "Temperatur (Werte zwischen -25C und 125C) : ";
      while(! (cin >> temperatur)){
      cin.clear();
      cin.ignore(std::numeric_limits<int>::max(),'\n');	
   }
   cout << "Prozess (Werte koennen nur 1=slow 2=typical 3=fast sein) : ";
   while(! (cin >> prozess)){
      cin.clear();
      cin.ignore(std::numeric_limits<int>::max(),'\n');
   }
   if( spannung > 1.32 || 
       spannung < 1.08 || 
       temperatur > 125 ||
       temperatur < -25 ||
       prozess > 3 || 
       prozess < 1 ){
       valid = false;
       cout << endl;
       cout << "Achtung sie haben Paramter eingegeben die ausserhalb der Spezifikationen liegen" <<endl;
       cout << "Bitte geben sie die Werte erneut ein" <<endl;
       cout << endl;
   }
   else
      valid = true;   
   }
   while( valid == false ); 
   berechneSpannungsFaktor();
   berechneTemperaturFaktor();
   berechneProzessFaktor();
   ausgabeFaktoren();
   system("pause");
}

[Dravere]

#include <limits>

Grüssli

[blade1982]

Jop, jetzt gehts. Danke
Hier nochmal das vollständige Programm:

void CFaktoren::bestimmeFaktoren() {
   bool valid;
   do {
   cout << "Bitte geben sie die drei folgenden Parameter ein:" <<endl;
   cout << "Spannung (Werte zwischen 1.08V und 1.32V): ";
   while(! (cin >> spannung)){
      cin.clear();
      cin.ignore(numeric_limits<int>::max(),'\n');
      cout <<"Eingabe falsch, bitte erneut eingeben: ";
   }
   cout << "Temperatur (Werte zwischen -25C und 125C) : ";
      while(! (cin >> temperatur)){
      cin.clear();
      cin.ignore(numeric_limits<int>::max(),'\n');	
      cout <<"Eingabe falsch, bitte erneut eingeben: ";
   }
   cout << "Prozess (Werte koennen nur 1=slow 2=typical 3=fast sein) : ";
   while(! (cin >> prozess)){
      cin.clear();
      cin.ignore(numeric_limits<int>::max(),'\n');
      cout <<"Eingabe falsch, bitte erneut eingeben: ";
   }
   if( spannung > 1.32 || 
       spannung < 1.08 || 
       temperatur > 125 ||
       temperatur < -25 ||
       prozess > 3 || 
       prozess < 1 ){
       valid = false;
       cout << endl;
       cout << "Achtung sie haben Paramter eingegeben die ausserhalb der Spezifikationen liegen" <<endl;
       cout << "Bitte geben sie die Werte erneut ein" <<endl;
       cout << endl;
   }
   else
      valid = true;   
   }
   while( valid == false ); 
   berechneSpannungsFaktor();
   berechneTemperaturFaktor();
   berechneProzessFaktor();
   ausgabeFaktoren();
   system("pause");
}