C++ Kiểm tra số nguyên.

Question

Mới đối với C++. Có vấn đề lặp chính xác trong khi xử lý lỗi. Tôi đang cố gắng kiểm tra xem liệu đầu vào của người dùng có phải là số nguyên hay không và là số dương.

do{ 
    cout << "Please enter an integer."; 
    cin >> n; 

    if (cin.good()) 
    { 
     if (n < 0) {cout << "Negative.";} 
     else {cout << "Positive.";} 
    } 
    else 
    { 
     cout << "Not an integer."; 
     cin.clear(); 
     cin.ignore(); 
    } 
}while (!cin.good() || n < 0); 

cout << "\ndone."; 

Khi không nhập số nguyên, ngắt vòng lặp. Tôi cảm thấy như tôi hiểu sai việc sử dụng vốn có của cin.clear() và cin.ignore() và trạng thái của cin trong vòng lặp này. Nếu tôi xóa cin.ignore(), vòng lặp sẽ trở thành vô hạn. Tại sao điều này? Tôi có thể làm gì để biến nó thành một vòng lặp hoạt động thanh lịch? Cảm ơn bạn.

Answer 1

Trong chi nhánh không phải số nguyên, bạn đang gọi thêm cin các phương thức để cin.good() được đặt lại thành true.

Bạn có thể thay đổi mã của bạn để một cái gì đó như thế này:

while(1) { // <<< loop "forever" 
    cout << "Please enter an integer."; 
    cin >> n; 

    if (cin.good()) 
    { 
     if (n < 0) {cout << "Negative.";} 
     else { cout << "Positive."; break; } 
    }       // ^^^^^ break out of loop only if valid +ve integer 
    else 
    { 
     cout << "Not an integer."; 
     cin.clear(); 
     cin.ignore(INT_MAX, '\n'); // NB: preferred method for flushing cin 
    } 
} 

cout << "\ndone."; 

hoặc bạn có thể đơn giản hóa nó hơn nữa như thế này:

while (!(cin >> n) || n < 0) // <<< note use of "short circuit" logical operation here 
{ 
    cout << "Bad input - try again: "; 
    cin.clear(); 
    cin.ignore(INT_MAX, '\n'); // NB: preferred method for flushing cin 
} 

cout << "\ndone."; 

Answer 2

int n; 

while (!(cin >> n)||n<0)//as long as the number entered is not an int or negative, keep checking 
{ 
cout << "Wrong input. Please, try again: "; 
cin.clear();//clear input buffer 

} 
//only gets executed when you've broken out of the while loop, so n must be an int 
cout << "Positive."; 

cout << "\ndone.";//finished! 

Nên làm những gì bạn muốn.