C语言中IF语句被忽略问题咨询（附相关代码片段）

Hey there! Let's work through the issues with your C code for calculating ring area, especially the broken if statements.

问题分析 #

Looking at your code, there are a couple of key reasons your if checks aren't working as expected:

• Unfinished if statement: The check for both diameters being negative (if ( out < 0 && in < 0)) is cut off mid-code—you only started writing pr... instead of a full printf call, and there's no closing } for this block. This syntax error will either cause a compile failure or make the compiler misinterpret your code, leading to the check being ignored entirely.
• Out-of-order validation: You're calculating the area before checking if the inputs are valid. Even if you fix the if blocks, invalid inputs will still generate incorrect area values before your error messages show up.
• Commented-out validation: The check for inner diameter being larger than outer diameter is commented out, so that critical validation isn't running at all.

修正后的完整代码 #

Here's a fixed version of your code that addresses all these issues:

#include <stdio.h>
#define PI 3.141593

int main (void) {
    float out, outrad, outarea, in, inrad, inarea, final;

    printf("Enter outer diameter: ");
    scanf("%f", &out);
    printf("Enter inner diameter: ");
    scanf("%f", &in);

    // First check for negative diameters
    if (out < 0 && in < 0) {
        printf("Invalid inputs: Both outer and inner diameters cannot be negative\n");
        return 1; // Exit program with error code
    }
    if (out < 0) {
        printf("Invalid input: Outer diameter cannot be negative\n");
        return 1;
    }
    if (in < 0) {
        printf("Invalid input: Inner diameter cannot be negative\n");
        return 1;
    }

    // Check if inner diameter is larger than outer
    if (in >= out) {
        printf("Invalid inputs: Inner diameter must be less than outer diameter\n");
        return 1;
    }

    // Calculate areas only if inputs are valid
    outrad = out / 2;
    outarea = PI * (outrad * outrad);
    inrad = in / 2;
    inarea = PI * (inrad * inrad);
    final = outarea - inarea;

    printf("The area of the ring is: %.2f\n", final);
    return 0;
}

关键改动说明 #

• Completed negative check: The unfinished if block is now fully implemented with a proper error message and a return 1; to exit the program immediately when invalid inputs are detected.
• Ordered validation: We first check all negative input cases, then verify that the inner diameter is smaller than the outer one. Only after all validations pass do we calculate the area.
• Explicit error handling: Each invalid case has a specific error message, making it easier to debug user input issues.
• Prevented invalid calculations: By moving area calculation after validation, we avoid wasting cycles on calculations that will never be used, and ensure we don't output nonsensical area values for bad inputs.

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