C语言手动实现tolower()函数时出现段错误的解决方法

你编写的代码触发zsh: segmentation fault是因为多处内存访问和逻辑错误，以下是具体问题和修复步骤：

核心错误分析 #

• 变量类型错误：char *variable[40];是包含40个未初始化char指针的数组，调用fgets(*variable, ...)时会直接访问野指针，触发段错误。需定义为字符数组来存储输入字符串。
• 函数参数设计错误：lower函数无需接收指针数组，C语言中字符串通过char*（字符指针）传递即可，直接操作字符串内存。
• 长度计算错误：sizeof(*s)/sizeof(*s[0])无法正确获取字符串长度——数组退化为指针传递到函数时，sizeof仅能得到指针字节数，而非数组长度。正确做法是用strlen(s)获取有效长度，或遍历到字符串结束符'\0'。
• 字符操作逻辑错误：原代码中*s[i]和s[i] +=32混淆了指针数组与字符数组的访问方式，直接操作字符数组下标即可修改对应字符。

修复后的完整代码 #

#include <stdio.h>
#include <string.h>

// 接收字符指针，直接修改原字符串
void lower(char *s){
    // 获取字符串有效长度（不含结束符'\0'）
    int length = strlen(s);
    int i;

    for(i = 0; i < length; i++){
        if (s[i] >= 'A' && s[i] <= 'Z'){
            // 用'a'-'A'替代硬编码32，可读性更强
            s[i] += ('a' - 'A');
        }
    }
}

int main(){
    // 定义字符数组，分配40字节存储空间
    char variable[40];
    printf("what would you like to do: ");
    // 读取输入到字符数组，sizeof(variable)为数组总长度
    fgets(variable, sizeof(variable), stdin);
    
    // 移除fgets读取的换行符（若存在）
    variable[strcspn(variable, "\n")] = '\0';
    
    lower(variable);
    printf("%s\n", variable);
    return 0;
}

额外说明 #

• 处理换行符：fgets会将输入的换行符\n读入字符串，用strcspn可快速定位并替换为结束符，避免输出时出现多余换行。
• 兼容写法：用'a' - 'A'替代32，虽ASCII中差值固定为32，但该写法更清晰，也兼容部分非ASCII编码中大小写差值相同的场景。
• 字符串传递逻辑：C语言中字符数组传递到函数时会自动退化为首元素指针，函数内修改指针指向的内存会直接影响原数组，无需额外返回值。

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