Что не так с моей операцией сцепления?

Question

Я пытался реализовать strcat(), поэтому я придумал этот код. Но, я не знаю, что в этом плохого? Это дает мне ошибку сегментации.

Я думал, что это может быть беспорядок распределения памяти? Это оно? Как я могу исправить это, не используя malloc()?

#include <stdio.h> 

char *strcat(char *s,char *d){ 

while(*s++ != '\0') ; 
while(*s++ = *d++) ; 

*s = '\0'; 

return s; 
} 

int main(){ 

char s[20]= "source"; 
char d[20]= "dest"; 

printf("%s\n",strcat(s,d)); 

return 0; 
} 

Я должен Concat d опережая s.

Answer 1

я установил его

#include <stdio.h> 
#define SIZE 20 

char *strocat(char *s,char *d){ 

char *temp = s; 

while(*s++ != '\0'); 
*--s; 
while(*s++ = *d++); 

*s = '\0'; 

return temp; 
} 

int main(){ 

char s[SIZE] = "source"; 
char d[SIZE] = "dest"; 

printf("%s\n",strocat(s,d)); 

return 0; 
} 

Answer 2

1. Струны в памяти только для чтения
2. Строки s не будет достаточно

долго Чтобы исправить:

... 
    #define S_STR "source" 

    char *d= "dest"; 
    char *s= S_STR; 
    s = malloc(strlen(s) + strlen(d) + 1); 
    strcpy(s, S_STR); 
    printf("%s\n",strcat(s,d)); 
    free(s); 
    return 0; 
} 

Answer 3

эля, d строковых констант! Вы никогда не должны делать такие вещи. имеют большой массив, такой как char s [100], копировать его, а затем использовать вашу конкатенацию. Помните, что у вас должно быть место для размещения контента d!

Answer 4

Строки s и d, которые вы указали, являются постоянными строковыми литералами, их нельзя изменить. вы должны объявить два массива символов и убедиться, что тот, который вы копируете, достаточно велик, чтобы удерживать другой.

#include <stdio.h> 

char *strcat(char *s,char *d) 
{ 
    //Get length of s string 
    size_t len = strlen(s); 

    //declare new pointer and assign it the value of s 
    char *ptr = s; 

    //move the pointer to the end of the string 
    ptr += len; 

    //copy contentes of d string to s string 
    while(*d != '\0') 
    { 
     *ptr++ = *d++; 
    } 

    *ptr = '\0'; 

    //return s 
    return s; 
} 

int main() 
{ 
    //make sure s array length is big enough to accomodate d string 
    char s[50] = "source"; 

    char d[] = "dest"; 

    printf("%s\n",strcat(s,d)); 

    return 0; 
}