调用toupper传参报错“makes integer from pointer without a cast”如何解决？

你代码存在4处核心错误，对应修复方案如下：

• 错误1：toupper() 函数入参要求为单个字符的int类型值（实际需传入unsigned char强制转换后的单个字符，或EOF），你直接传入char*类型的字符串指针input，类型不匹配，是编译报错的核心诱因。
• 错误2：toupper() 返回值是转换后的单个字符的int值，你将其赋值给char类型变量toUpper后，用%s格式符打印，%s要求传入字符串首地址，传入单个char会触发内存访问错误，导致崩溃或输出乱码。
• 错误3：要实现整个字符串转大写，需要遍历字符串的每一个字符单独调用toupper()处理，不能直接对整个字符串调用一次函数完成转换。
• 错误4：代码中("%s\n",toupper(input))属于无用的逗号表达式，前面的字符串常量不会产生任何实际作用，属于冗余写法。

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

int main(int argc, char *argv[]) {
    // 入参合法性校验，避免空指针访问
    if (argc < 2) {
        printf("请传入要转换的字符串参数\n");
        return 1;
    }
    char *input = argv[1];
    printf("原始字符串：%s\n", input);
    // 遍历每个字符逐位转大写
    int len = strlen(input);
    for (int i = 0; i < len; i++) {
        input[i] = toupper((unsigned char)input[i]);
    }
    printf("转大写后字符串：%s\n", input);
    return 0;
}

• 如果原字符串是只读常量（例如直接定义char *input = "test"），不能直接修改字符内容，需要先申请堆内存或者用栈数组拷贝原字符串后再修改，避免触发段错误。
• 调用toupper时强制转unsigned char是为了避免字符ASCII值为负的场景下出现未定义行为。

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