Question

所以我的兄弟正在制作一个程序，将字符串中的所有单词转换为主题标签，但由于某种原因，它总是在执行结束时出现“分段错误”错误。我试图找到可能导致它的原因，但还没有找到。这是代码：

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

char* setHashtag(char text[10000])
{
    int i, j;

    printf("Initial text = %s\n", text);

    for (i = 9998; i >= 0; i--)
    {
        text[i+1] = text[i];
    }
    text[0] = ' ';

    for (i = 0; text[i+1] != '\0'; i++)
    {
        if(text[i] == ' ' && text[i+1] != ' ')
        {
            for (j = 9998; j > i; j--)
            {
                text[j+1] = text[j];
            }
            text[i+1] = '#';

            printf("Partial text = %s\n", text);
        }
    }

    return text;
}

void execute() {
    char text[5000], textFinal[10000];

    gets(text);

    strcpy(textFinal, setHashtag(text));
    printf("%s\n", textFinal);
}

int main()
{
    execute();
    printf("Back to main\n");
    return 0;
}

Answer 1

您将大小为5000的数组传递给您的函数，但您可以访问其中的10000个元素。当然，它会崩溃。

函数声明中指定的数组大小无关紧要。它被编译器忽略。此

char* setHashtag(char text[10000])

等同于此

char* setHashtag(char *text)

即。该函数接收一个指向原始参数数组开头的指针，而不是参数数组的新本地副本（C中的裸数组不可复制）。

这意味着当您将功能称为

时

char text[5000];
...
setHashtag(text)

text数组不会神奇地成为char [10000]数组。它仍然是char [5000]数组，因为它最初是声明的。试图在函数内部访问text[9998]等导致未定义的行为。

由于您的setHashtag函数需要大小为10000的固定大小数组，因此将函数声明为

可能更好一点。

char* setHashtag(char (*text)[10000])

并将数组参数传递为setHashing(&text)。这将确保您无法传入错误大小的数组。在函数内部，您必须以(*text)[i]的形式访问数组。

为什么这个程序给我一个“分段错误”？

1 个答案: