C语言函数条件判断失效，首字母大小写转换逻辑异常求助

嘿，我一眼就看出问题出在哪儿了——你的upper函数里的条件判断写法在C语言里完全不符合预期，这就是为什么不管首字母是不是小写，代码都会错误执行转换逻辑的原因。

错误根源：连续比较的陷阱 #

C语言不支持像数学表达式那样的连续比较，97 <= *a <= 122会被编译器从左到右拆解成：

(97 <= *a) <= 122

• 如果*a是小于97的字符（比如大写字母'A'是65），97 <= *a的结果是0（假），然后0 <= 122的结果是1（真），条件成立；
• 如果*a是大于等于97的字符，97 <= *a的结果是1（真），1 <= 122也为真，条件依然成立。

所以不管输入的首字母是什么，这个条件永远为真，导致每次都会执行*a -= 32的操作，自然会错误处理非小写字母的情况。

修正方案 #

1. 修复条件判断逻辑

把连续比较改成用逻辑与&&连接的两个独立判断，同时建议用字符常量'a'和'z'代替魔法数字97、122，代码可读性更高：

void upper(char *a) {
    if (*a >= 'a' && *a <= 'z') {
        *a -= 32;
    }
}

2. 补充函数声明（避免隐式声明警告）

在main函数之前声明upper函数，或者把upper函数的定义放在main函数前面，否则编译器会默认upper返回int类型，可能引发潜在问题：

#include <stdio.h>
#include <stdlib.h>

// 先声明upper函数
void upper(char *a);

int main() {
    char name[30];
    fgets(name, sizeof(name), stdin);
    printf("name: ");
    puts(name);
    upper(name);
    puts(name);
    return 0;
}

void upper(char *a) {
    if (*a >= 'a' && *a <= 'z') {
        *a -= 32;
    }
}

更优雅的优化：使用标准库函数 #

C标准库的<ctype.h>头文件提供了专门的字符判断和转换函数，比手动判断ASCII码更安全（能兼容非ASCII编码的字符集），推荐这样写：

#include <stdio.h>
#include <ctype.h> // 包含ctype头文件

void upper(char *a);

int main() {
    char name[30];
    fgets(name, sizeof(name), stdin);
    printf("name: ");
    puts(name);
    upper(name);
    puts(name);
    return 0;
}

void upper(char *a) {
    // 用unsigned char避免字符为负时的未定义行为
    if (islower((unsigned char)*a)) {
        *a = toupper((unsigned char)*a);
    }
}

这样修改后，代码就能正确只转换首字母为小写的情况，对非小写字母的输入不会做错误处理了。

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