二次方程求解代码遇非数值输入终止，如何实现正常循环？

Let's get your code working properly so it handles non-numeric inputs gracefully and only exits when any coefficient is zero. Here's what was wrong with the original code, plus the fixed version:

Issues in the Original Code #

• When you enter non-numeric values, cin enters a failure state—all subsequent cin operations will fail immediately until you clear this state. Your current code just tries to read again without fixing this, so it gets stuck in a loop or exits unexpectedly.
• The input validation logic doesn't clear the invalid data from the input buffer, so the bad input keeps being reprocessed.
• The termination check (if ((a == 0) || (b == 0) || (c == 0))) runs even if the input failed, which can lead to using uninitialized values.

Fixed Code #

#include <iostream>
#include <cmath>
#include <limits> // Needed for numeric_limits

using namespace std;

int main() {
    while (true) { // More readable infinite loop
        float a, b, c, D, x1, x2, real_part, im_part;
        
        cout << "Please enter the Quadratic Coefficients (a, b, c): " << endl;
        cin >> a >> b >> c;

        // Handle invalid input
        while (cin.fail()) {
            // Clear the failure flag
            cin.clear();
            // Discard all invalid characters in the buffer up to the newline
            cin.ignore(numeric_limits<streamsize>::max(), '\n');
            cout << "Error! Please enter valid numerical values: " << endl;
            cin >> a >> b >> c;
        }

        // Check termination condition ONLY after valid input
        if (a == 0 || b == 0 || c == 0) {
            cout << "One of the coefficients is zero. Exiting program." << endl;
            break;
        }

        // Calculate discriminant and roots
        D = b*b - 4*a*c;
        if (D < 0) {
            real_part = -b/(2*a);
            im_part = sqrt(-D)/(2*a);
            cout << "Roots are Complex" << endl;
            cout << "x1 = " << real_part << "+" << im_part << "i" << endl;
            cout << "x2 = " << real_part << "-" << im_part << "i" << endl;
        } else if (D == 0) {
            x1 = (-b + sqrt(D)) / (2*a);
            cout << "Real and Repeated Roots" << endl;
            cout << "x1 = x2 = " << x1 << endl;
        } else {
            x1 = (-b + sqrt(D)) / (2*a);
            x2 = (-b - sqrt(D)) / (2*a);
            cout << "Real and Distinct Roots" << endl;
            cout << "x1 = " << x1 << endl;
            cout << "x2 = " << x2 << endl;
        }
    }
    return 0;
}

Key Changes Explained #

1. Input Validation Loop: We added an inner while (cin.fail()) loop that runs until the user enters valid numbers. This loop:
   • Uses cin.clear() to reset the error state of cin so it can read input again.
   • Uses cin.ignore(...) to throw away all the invalid characters left in the input buffer (otherwise, the bad input would be read over and over).
2. Termination Check Placement: We moved the exit condition check after we've confirmed the input is valid, so we only check a, b, c when they hold valid values.
3. Readability: Swapped the for(;;) loop for a while(true) loop (just a style choice, both work, but this is more intuitive).
4. Minor Improvements: Renamed variables like real to real_part (since real is a keyword in some contexts), and added a clearer message when exiting.

Now your program will:

• Prompt the user again if they enter non-numeric values
• Only exit when any of a, b, or c is zero
• Calculate roots correctly for valid inputs

内容的提问来源于stack exchange，提问作者Nick Chr