基于switch语句的成绩评级系统代码故障排查与优化请求

安全阿岳

2026-6-18

成绩评级代码问题排查与优化

问题根源

你这段代码的核心问题是完全误用了switch语句的匹配逻辑：

switch是用输入的分数值去匹配case后的常量，比如输入85，switch会找case 85:，但你写的case都是1、2、3这类固定数字，根本匹配不上任何有效分数，所以直接走到default输出F。
无效分数的判断放在case 1:里，只有输入1才会进入这个分支，其他无效值（比如101、-5）根本不会触发这个判断。
每个case里嵌套if，完全浪费了switch的作用，代码冗余且逻辑混乱。

优化后的代码

#include <stdio.h>

int main()
{
    int score;
    printf("请输入0-100之间的成绩：");
    scanf("%d", &score);

    // 先校验输入合法性
    if (score < 0 || score > 100) {
        printf("无效成绩，请输入0-100之间的数值\n");
        return 0;
    }

    // 通过score/10将分数映射到固定区间，适配switch匹配
    switch (score / 10) {
        case 10: // 100分单独处理
        case 9:  // 90-99分
        case 8:  // 80-89分
            printf("A+\n");
            break;
        case 7:  // 70-79分
            printf("A\n");
            break;
        case 6:  // 60-69分
            printf("A-\n");
            break;
        case 5:  // 50-59分
            printf("B\n");
            break;
        case 4:  // 40-49分
            printf("C\n");
            break;
        case 3:  // 32-39分，补充下限判断
            if (score >= 32) {
                printf("D\n");
            } else {
                printf("F\n");
            }
            break;
        default: // 0-31分
            printf("F\n");
            break;
    }

    return 0;
}