创建我自己的 strupr 函数，但它不起作用

Question

这个问题已经有答案了:

Crash or "segmentation fault" when data is copied/scanned/read to an uninitialized pointer (5 个回答)

已关闭 4 年前。

所以我刚刚创建了自己的toupper (单个字符大写)和 strupr (将字符串大写)。但这不起作用。所以，这是代码。

#include <stdio.h>

int toupper(const int character) {
    if (character >= 97 && character <= 122)
        return (character - 32);
    return character;
}

char *strupr(const char *string) {
    char *result;
    for (int a = 0; a < strlen(string); a++) {
        *(result + a) = toupper(*(string + a));
    }
    return result;
}

int main() {
    char myString[6] = "Hello";
    printf("myString (Before): \"%s\"\n", myString);
    printf("myString (After): \"%s\"\n", strupr(myString));
    return 0;
}

这是输出:

myString (Before): "Hello"

它只打印出第一行，之后程序就停止了。所以我需要帮助修复我的代码。

Answer 1

您正在尝试访问未初始化的指针结果。这会导致不确定的行为:

char * result;
for(int a=0; a<strlen(string); a++)
{
    *result[a] = toupper(string[a]);
}

在访问之前，您需要为该数组分配足够的内存:

char toupper(const char character)
{
    if(character >= 'a' && character <= 'z')
        return (character + ('A' - 'a'));
    return character;
}

void strupr(const char *string, char *result)
{
    for(int a = 0; a < strlen(string); a++)
        *(result + a) = toupper(*(string + a));
}


int main()
{
    char myString[6] = "Hello", res[6] = {};
    printf("myString (Before): \"%s\"\n", myString);
    strupr(myString, res);
    printf("myString (After): \"%s\"\n", res);
    return 0;
}

或者您可以使用 strupr 的另一个变体，它会更改输入字符串本身:

char* strupr2(char *string)
{
    for(int a = 0; a < strlen(string); a++)
        string[a] = toupper(string[a]);
    return string;
}

int main()
{
    char myString[6] = "Hello", res[6] = {};
    printf("myString (Before): \"%s\"\n", myString);
    printf("myString (After): \"%s\"\n", strupr2(myString));
    return 0;
}