0

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

#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;
}
4

1 回答 1

8

您将一个大小数组传递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函数需要一个固定大小的 size 数组10000，因此将函数声明为

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

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

于 2012-12-22T15:56:15.033 回答