愚蠢的问题：strcat 和 char 指针

Question

在处理字符串时我通常不使用指针字符，但我通常认为它们更好，因为我可以打印整个字符串而无需迭代每个字符。

无论如何，因为我不使用它们，所以我真的不知道这两者是如何相互作用的。运行以下代码时程序崩溃，所以我不知道该怎么办，因为没有错误。

int main(){
char s[]="abcde";
char *p;
for(unsigned int i=0;i<strlen(s);i++)
    strcat(p,s+i);
cout<<*p;
}

我尝试将两个字符串都表示为指针，但它没有改变任何东西。只有当我尝试打印第二个数组时它才会崩溃。我尝试了 *p++=*s 或我在谷歌上找到的类似内容，但它仍然崩溃，我做错了什么？

Answer 1

指针 p 不指向任何地方，你需要为第二个数组保留内存。例如，

char *p = new char [strlen(s) + 1];

for 循环中的 strcat 也不是复制字符串的最佳方式。试试

strcpy(p, s);

（当然是在为 p 分配内存之后）。

补充：如果需要使用strcat代替strcpy，需要用空字符串初始化字符串p，即：

char *p = new char [strlen(s) + 1];
*p = '\0';

Answer 2

你没有为指针 p 分配内存。当您调用 strcat 而不分配内存时，您当然会遇到分段错误，因为 strcat 将尝试通过指针 p 写入内存指针。

您不需要用于连接字符串的 for 循环。 strcat 函数将完整的字符串复制到 p 指向的内存中。

如果你想打印整个字符串，不要遵循指针 p fi 你想打印整个字符串，遵循导致只打印字符串的第一个字符。

#include<iostream>
#include<cstring>

int main(){
char s[]="abcde";
//Allocating memory to so that p points valid memory.
char concatenatedString[50] = "";
//Initializing pointer to allocated memory.
char *p = concatenatedString;
// for(unsigned int i = 0;i < strlen(s); i++) //Not required.
    strcat(p,s);
//Here don't dereference if you want to print whole string.
std::cout<<p;
}