C中的这个连接函数有什么问题？

Question

这是使用结构的家庭作业的一部分，我似乎无法理解这一功能。该函数是 string_t *concat (string_t *s1, string_t *s2)，它返回新的字符串结构。这就是我到目前为止所拥有的，只要达到它就会使编译器崩溃。程序可以编译，但是执行时出现“file”.exe has stop working 错误。任何帮助将不胜感激。谢谢！

typedef struct string{ //String struct (in .h file)

char *line;
int length;

} string_t;


string_t* concat(string_t *s1, string_t *s2) { //actual function (in .c)

int len1, len2;
len1 = length(s1);
len2 = length(s2);

int i, j, s;

string_t *newStr;
newStr = (string_t*)malloc(sizeof(string_t)*2);


for (i = 0; i<len1; i++) {
    *((newStr->line)+i) = *((s1->line)+i);
    }

for (j=0; j<len2; j++) {
    *((newStr->line)+(i+j)) = *((s2->line)+j);
    }

*((newStr->line)+(i+j))='\0';

return newStr;

}



concat(s1, s2); //tests function

Answer 1

newStr = (string_t*)malloc(sizeof(string_t)*2);

您为newStr 分配内存，但您没有为newStr->line 分配内存。尝试类似：

newStr = malloc(sizeof *newStr);
newStr->line = malloc(s1->length + s2->length + 1);

---

旁注：*((newStr->line)+i) 可以写成newStr->line[i]。

Answer 2

顺便说一句，这是一种不用那种难看的 ptr 数学语法的 cat 方法：

char* dest = newStr->line;

const char* src = s1->line;
while (*src)
{
  *dest = *src;
  ++dest;
  ++src;
}

src = s2->line;
while (*src)
{
  *dest = *src;
  ++dest;
  ++src;
}

*dest = '\0';