C에서이 연결 함수에 어떤 문제가 있습니까?

Question

이것은 구조체를 사용하는 숙제의 일부이며이 기능을 이해하지 못하는 것 같습니다. 함수는 string_t * concat (string_t * s1, string_t * s2)이고 새로운 문자열 struct를 반환합니다. 이것은 내가 지금까지 가지고있는 것이며 컴파일러에 도달 할 때마다 충돌합니다. 프로그램이 컴파일되지만 "file".exe가 작업을 중단 할 때 실행 중 오류가 발생합니다. 어떤 도움이라도 대단히 감사하겠습니다. 감사!

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

BTW, 여기에 고양이에 대한 방법은 그 추한 PTR 수학 구문없이입니다 :

는

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';