生成没有指针的大写字母

Question

我正在尝试编写一个大写函数，它将字符串中的所有小写字符转换为大写字符。

但是，我的代码中出现 Bus 10 错误。我知道不能在 C 中修改字符串文字；所以，我不确定这是否是正确的方法。

我的代码如下：

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

int uppercase(char source[])
{
 int i;

 for(i=0; i<=strlen(source); ++i)
    if (source[i]>= 'a' && source[i]<= 'z')
        source[i]= source[i]-'a' +'A';
    else 
        source[i]=source[i];
}

int main(){
    uppercase("cold");

    return 0;
}

理想情况下，这个函数应该返回 COLD。我想错误在于我的整个 if 语句。

Answer 1

您遇到崩溃的原因是您的代码修改了字符串文字。字符串文字中的字符被放置在受保护的内存区域中，因此不能更改：它是未定义的行为。

替换这个

uppercase("cold");

用这个：

char cold[] = "cold";
uppercase(cold);

现在字符串的字符被放置在内存的可修改区域中，允许您根据需要进行更改。

Answer 2

你在不知不觉中就使用指针。

在你的函数定义中

int uppercase(char source[])

char source[] 被编译器视为指向 char 的指针 (char *source)

因此，当将字符串文字传递给 uppercase() 时，您只需传递它的地址。然后在您的函数中尝试修改它会导致未定义的行为。

你也不能返回整个数组，所以你只返回一个指向它的指针。

char *uppercase(char source[])
{
     int i;
     size_t len = strlen(source);
     char *tmp;
     tmp = malloc(len+1);
     if (tmp!=NULL){
         memcpy(tmp, source, len+1);
         for(i=0; i<len; ++i){
            if (tmp[i]>= 'a' && tmp[i]<= 'z'){
                tmp[i]= tmp[i]-'a' +'A';
            }
        }
    }
    return tmp;
}

然后：

int main(){
    char *str = uppercase("cold");
    printf("%s", str);
    free(str);

    return 0;
}

你完成代码：http://ideone.com/BJHDIF