Wie man char-Eingaben in cin zurückweist und minimale und maximale int-Werte definiert?

Question

Ich habe ein Programm, das die Fähigkeit hat, Benutzereingaben zu verwerfen, wenn ein Zeichen anstelle eines int eingegeben wird, und das funktioniert fast perfekt - alles eingegeben, das keine Nummer ist, wird zurückgewiesen.

Allerdings müssen alle diese cin s jeden Wert zwischen einem Minimum und einem Maximum akzeptieren, aber ich kann es nicht zum Funktionieren bringen. Der Code unten zeigt meine bisherigen Bemühungen, aber es gibt einen kleinen Fehler. Wenn ein Char eingegeben wird, gefolgt von einem int, der außerhalb des Bereichs ist, und ein anderer char eingegeben wird (Ich möchte rigoros testen - ich meine, wer weiß, was passieren könnte, wenn ein tatsächlicher Endbenutzer über die Problem) das Programm wirft den Endwert von hypothekenTerm heraus als 0.

Könnte mir jemand sagen, wo ich falsch liege und mir irgendwelche Hinweise geben, um mir zu helfen, es zu beheben? Vielen Dank im Voraus an alle, die mir helfen können, mein Problem zu lösen!

int mortgageTerm; 
string line; 
cout << "Mortgage term (1 - 40 years) : "; 
while (!(cin >> mortgageTerm)) 
{ 
    cout << "That's not a valid choice! Try again : "; 
    cin.clear(); 
    cin.ignore(numeric_limits<streamsize>::max(), '\n'); 
} 
while (getline(cin, line)) 
{ 
    stringstream linestream; 
    if (!linestream >> mortgageTerm) 
    { 
     cout << "Input was not a number! Try again : "; 
     cin >> mortgageTerm; 
     continue; 
    } 
    if ((mortgageTerm <= 0 || mortgageTerm > 40)) 
    { 
     cout << "Input out of range. Try again : "; 
     cin >> mortgageTerm; 
     continue; 
    } 
    char errorTest; 
    if (linestream >> errorTest) 
    { 
     cout << "Invalid input. Try again : "; 
     cin >> mortgageTerm; 
     continue; 
    } 
    break; 
} 
cout << mortgageTerm; 

Answer 1

Sie sind fast da. Ihr erstes Problem ist Ihre erste While-Schleife wird überhaupt nicht benötigt. Dann müssen wir nur die zweite Schleife optimieren, um sicherzustellen, dass der gesamte gelesene Eingang in dem Wert verwendet wurde, den Sie erhalten. Wir können es auch vereinfachen, indem eine einzige Fehler Anweisung, gibt Ihnen diese Änderungen zu machen

int mortgageTerm; 
string line; 
cout << "Mortgage term (1 - 40 years) : "; 
while (getline(cin, line)) // consume all input given 
{ 
    stringstream linestream(line); // you have to construct the stream from the string here 
    linestream >> mortgageTerm; // try and read the data 
    if (!linestream.eof() || mortgageTerm <= 0 || mortgageTerm > 40) 
    { 
     // either there is input left in linestream or the value is not in range 
     cout << "Invalid input. Try again : "; 
    } 
} 

Answer 2

überprüfen Nur für das Minimum und Maximum im gleichen Zustand, in dem Sie überprüfen, ob es in ein int umgewandelt werden kann, || verwenden, in einem Zustand der Ausdrücke geprüft links nach rechts um, so der erste hat seine Arbeit bereits getan, wenn Sie den zweiten bewerten und MortgageTerm den Wert haben wird.

Bearbeitet um Kommentare zu adressieren.

int mortgageTerm; 
cout << "Mortgage term (1 - 40 years) : "; 

while (!(cin >> mortgageTerm) || 
     mortageTerm < 1  || 
     mortgageTerm > 40  ) 
{ 
    cout << "That's not a valid choice! Try again : "; 
    cin.clear(); 
    cin.ignore(std::numeric_limits<streamsize>::max(), '\n'); 
} 

// If you are concerned about extra input after the number and want to clear the input stream 
// cin.ignore(std::numeric_limits<streamsize>::max(), '\n');