附加两个字符时出现分段错误 - C++

Question

我正在尝试附加两个字符，但由于某种原因我遇到了分段错误。

我的代码是这样的；

#include <string.h>
char *one = (char*)("one");
char *two = (char*)("two");

strcat(one, two);

我似乎在 strcat(one, Two) 处遇到分段错误，这是为什么？

Answer 1

http://www.cplusplus.com/reference/clibrary/cstring/strcat/

strcat 的第一个参数必须足够大以容纳生成的字符串

尝试：

//assuming a,b are char*
char* sum = new char[strlen(a) +strlen(b)+1];
strcpy(sum,a);
strcat(sum,b);

Answer 2

应该有足够的合法内存来保存整个字符串。

char *one = new char[128]; //allocating enough memory!
const char *two = "two"; //"two" is const char*

strcpy(one, "one");
strcat(one, two); //now the variable "one" has enough memory to hold the entire string

---

顺便说一句，如果你更喜欢在 C++ 中使用 std::string 而不是 char*，这样的事情会更容易处理：

#include<string>

std::string one = "one";
std::string two = "two";

one = one + two; //Concatenate 

std::cout << one;

输出：

onetwo

Answer 3

这有两个原因。

1. 如果您有一个初始化为字符串文字的指针，则该内存是只读的，修改它将导致未定义的行为。在这种情况下，如果您尝试将字符串附加到字符串文字，您将修改此类内存，这将导致问题。

2. 使用 strcat 时，您需要保证指定位置处存在用于连接字符串的空间。在这种情况下，您不能保证这一点，因为只能保证字符串文字有足够的空间来保存文字本身。

要解决此问题，您需要显式分配一个足够大的缓冲区来容纳两个字符串的串联，包括空终止符。这是一种方法：

char* buffer = malloc(strlen(one) + strlen(two) + 1);
strcpy(buffer, one);
strcat(buffer, two);

希望这有帮助！

Answer 4

段错误是因为您尝试写入只读内存。 strcat 的第一个操作是将“t”从“two”的第一个条目复制到“one”末尾的 null 中。因此，严格来说，段错误并不是由于缺乏存储而导致的——我们从来没有走到这一步。事实上，这段代码也可能会给你一个段错误：

char* one = "one";
char* two = "";
strcat(one, two);    

所有这些尝试做的就是将一个空值复制到一个空值上，但在只读内存中。我想优化器可能会在某些平台上阻止这种情况。

奇怪的是，下面的（不正确的）代码（可能）不会给你一个段错误，甚至给出“正确”的答案：

char one[] = "one";
char two[] = "two";
strcat(one, two);   
printf("%s\n", one);

这成功地将“onetwo”写入我的机器上的标准输出。我们得到了一堆乱码，我们碰巧侥幸逃脱了。

另一方面，这会导致段错误：

char* one = "one        "; // Plenty of storage, but not writable.
char two[] = "two";
strcat(one,two);    

因此解决方案：

const unsigned enoughSpace = 32;
char one[enoughSpace] = "one";
char two[] = "two";
strcat(one,two);    
printf("%s\n", one);

当然，问题在于，要留出足够的空间来存储即将到来的内容有多大？

因此函数 strncat 或 strcat_s 或更简单的 std::string 。

这个故事的寓意是：在 C++ 中，就像 C 一样，您确实需要知道您的内存布局是什么。

Answer 5

这里有几个问题。首先，尽管您已将字符串转换为可变版本，但它们实际上是字符串文字，因此不应写入。其次，您使用 strcat 它将完全写入字符串缓冲区忽略字符串缓冲区的长度（最好使用 strncat ，它需要您指定缓冲区的长度）。最后，由于这是 C++，所以使用以下方法会更好：

#include <string>

// ...

string one = "one";
string two = "two";
one.append(two); 

Answer 6

你从来没有为你的字符串保留一些空间。

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

int main(void){
    char str[20] = "";
    strcat(str, "one");
    strcat(str, "two");
    printf("%s", str);
}

将是执行此操作的一种正确方法。另一种（也是更好的方法）是使用 std::string 类。

#include <string>
#include <cstdio>

int main(void){
    std::string str;
    str += "one";
    str += "two";
    std::printf("%s", str.c_str());
}

Answer 7

您的目标字符串应该足够大以容纳目标字符串和源字符串。所以一个例子是
<代码>

char one[10] = "one";
char two[4] = "two";
strcat(one,two);

<代码>

Answer 8

strcat 需要一个“可写”缓冲区作为目标。在您的示例中，它是指向字符串常量（或文字）的指针，您无法写入该常量，因此会导致异常。目标缓冲区可以是堆栈上的缓冲区或动态分配的缓冲区（例如，使用 malloc）。

Answer 9

这不是“空间不够”的问题。

char *a = "str";

看上面的代码，指针a指向“静态内存”。字符串“str”存储在PCB中的静态位置，这意味着它不能被覆盖。

所以，下面的代码会更好：

#include <string>
using std::string;

string a = "stra";
string b = "strb";

a += b;