программа конкатенации в C

Question

Я написал программу для конкатенации двух строк, и она бросает ошибку сегментации во время выполнения в строке s1 [i + j] = s2 [j], in for loop ..... И i я не в состоянии понять, почему это происходит так .... Просьба координировать меня, где я иду не так.

char* concatenate(char *s1, char *s2) 
{ 
    int i,j=0; 

    for(i=0; s1[i] != '\0'; i++); 
    for(j=0; s2[j] != '\0'; j++) 
    { 
     s1[i+j] = s2[j]; 
    } 
    s1[i+j] = s2[j]; 

    return s1; 
} 

---

char *s1 = (char *) malloc(15);; 
    char *s2 ; 

    s1 = "defds"; 
    s2 = "abcd"; 

    s1 = concatenate(s1,s2); 
// printf("\n\n%s\n\n",s1); 

Answer 1

Вы не должны написать свои собственные конкатенации, уже функционирует в стандартном libary, чтобы сделать эту задачу для вас!

#include <string.h> 

char *strcat(char *dest, const char *src); 
char *strncat(char *dest, const char *src, size_t n); 

Answer 2

Во втором фрагменте кода вы выделяете буфер размером 15 и затем назначаете его s1. Затем вы назначаете «rahul» на s1, который утечки только что выделенной памяти и присваивает s1 6-байтовой памяти, которую вы, вероятно, не можете записать. Измените s1 = "rahul"; на strcpy(s1, "rahul");, и вам, возможно, повезет больше.

Я согласен с другими ответами, хотя функция конкатенации опасна.

Answer 3

s1 = "rahul"; 

Эта строка не копирует строку «Rahul» в буфер, на который указывает s1; он перераспределяет указатель s1, чтобы указать на (не изменяемый) строка «Рахул»

Вы можете получить требуемую функциональность, используя вашу concatenate функцию дважды:

char *s1 = (char *) malloc(15); 

s1[0] = '\0'; // make sure the buffer is a null terminated string of length zero 
concatenate(s1, "rahul"); 
concatenate(s1, "bagai"); 

Обратите внимание, что функция concatenate еще несколько небезопасно, поскольку он слепо копирует байты, как и strcat. Вам нужно быть уверенным, что буфер, который вы передаете, достаточно велик или изменить его, чтобы взять длину буфера, например, strncat.

Answer 4

Когда вы делаете s1 = "rahul";, вы переписываете выделенную память. Эта строка не копирует «rahul» в область malloc'ed, она меняет s1, чтобы указать на константу строки «rahul» и выбрасывает указатель на память malloc'ed.

Вместо этого, вы должны использовать strcpy, чтобы скопировать строку в malloc'ed области:

// s1 = "rahul"; 
strcpy(s1, "rahul"); 

Это зафиксирует ваш звонок concatenate так s1 теперь будет указывать на правильную область 15 байт памяти.

В качестве альтернативы, вы можете отказаться от динамического распределения и выделения + назначить начальную строку сразу:

char s1[15] = "rahul"; 

Это выделит 15 байт в стеке и скопировать «Rahul» в это пространство. Обратите внимание на тонкость здесь. В этом случае на самом деле правильно использовать =, тогда как неверно, когда s1 объявлен как char *s1.

Один важный урок отладки, который вы можете узнать из этого, заключается в том, что когда ваша программа выходит из строя на определенной строке кода, это не значит, что это ошибка. Часто вы делаете ошибку в одной части своей программы, и эта ошибка не проявляется в аварии до более позднего времени. Это часть того, что отлаживает такой восхитительно расстраивающий процесс!