c - C语言中的大写

Question

我有这个代码：

void changeToCapital(char* str)
{
    int i;
    for (i=0; i<strlen(str); i++) 
    {
        str[i] =str[i] -32;
    }
}

这个方法应该得到一个char*变量，并将其更改为大写。出于某种原因，我收到一条错误消息EXECUTE_BAD_ACCESS。

调用函数：

char* s = "itzik";
changeToCapital(s);
printf("%s\n",s);

我在这里做错了什么？

score 12 · Accepted Answer

这很可能是因为您向它传递了一个指向不可写内存的指针，例如从字符串文字中获得的指针：

char *ptr = "Hello";
changeToCapital(ptr); // <<== ERROR !

您可以更改调用以避免错误：

char ptr[] = "Hello";
changeToCapital(ptr);

附带说明，您对大写的更改仅在所有字母都为小写时才有效。您应该使用toupper(ch)函数而不是减去 32。

void changeToCapital(char* str) {
    for (; *str = toupper(*str) ; str++)
        ;
}

score 4 · Accepted Answer

第一的：

你一直在计算strlen。相反，您应该将字符串的长度存储在局部变量中。

第二：您可能正在调用这样的函数：

char *str = "Hello World";
changeToCapital(str);

这很糟糕。"Hello World"是const，并且不能被您的程序修改。相反，您应该将字符串指定为字符数组，以确保它是不可变的：

char str[] = "Hello World";
changeToCapital(str);

2 回答 2