C 中的简单字符串运行时错误？

Question

这段代码编译得很好，但运行时出现分段错误错误？ 谁能告诉我为什么吗？

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

int main() {
    const char s2[] = "asdfasdf";
    char* s1;

    strcpy(s1, s2);
    printf("%s", s1);

    return 0;
}

Answer 1

您没有为 s1 分配内存。 您有一个指向 s1 的指针，但没有为 strcpy 分配内存来将 s2 的值复制到其中。

char *s1 = malloc( strlen(s2) + 1 );

strcpy( s1, s2 );

Answer 2

您尚未为 s1 分配任何内存。 它是一个空无一物的指针。

char* s1 = malloc(sizeof(s2));
strcpy(s1, s2);
printf("%s", s1);
free(s1);

Answer 3

问题是 s1 没有任何与之关联的内存。 strcpy 不会调用 malloc()。

您可以这样做：

char s1[10];

或

char *s1 = malloc(10);

Answer 4

您为单个指针 s1 分配了空间，但没有为s1 指向的字节分配空间。

解决方案是为 s1 动态分配内存：

s1 = (char *)malloc(strlen(s2) + 1);
strcpy(s1, s2);

请记住，您需要比数字多分配一个字节的内存（调用 malloc 时的 +1） s2 中的字符数，因为末尾有一个隐式的 NULL 字节。

有关详细信息，请参阅 C 内存管理（堆栈溢出）。

Answer 5

他们都说，你需要为 s1 分配空间。 其他人发布的内容都可以正常工作，但是，如果您想要一种更简单的方法来为现有字符串分配空间并将其复制到新指针中，请像这样使用 strdup ：

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

using namespace std;

int main() {
    const char s2[] = "asdfasdf";
    char* s1;

    s1 = strdup(s2);
    printf("%s", s1);

    return 0;
}

有人之前提到过 strdup，这将是一种使用方法它。 大多数系统应该支持它，因为它位于标准 C 库中。 但显然有些人不这么认为。 因此，如果它返回错误，请使用已经提到的方法编写自己的错误，或者仅使用已经提到的方法；）

Answer 6

目前还没有人指出 strdup（字符串重复）解决这一问题的潜力。

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

using namespace std;

int main() {
    const char s2[] = "asdfasdf";
    char* s1;

    s1 = strdup(s2);  // Allocates memory, must be freed later.
    printf("%s", s1);

    free(s1);         // Allocated in strdup, 2 lines above
    return 0;
}

Answer 7

您需要分配目标（并且 using namespace std; 不是 C，而是 C++，其余代码是 C）。

Answer 8

您必须为指针 s1 分配内存。 如果你不这样做，它将指向未知的地方，从而出现分段错误。 正确的代码应该是：

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

int main() {
    const char s2[] = "asdfasdf";
    char* s1 = malloc(21 * sizeof(s2[0]));
    strcpy(s1,s2);
    printf("%s",s1);
    return 0;
}