реализация strcat

код в порядке.

Похоже, у вас проблема в коде вызова.

Вы не забыли выделить достаточно памяти для целевой строки?

Я бы настоятельно рекомендовал использовать указатели, а не целочисленные индексы, из-за опасений целочисленного переполнения. Даже если size_t имеет то же количество битов, что и char *, вы добавляете индексы, а не указатели.

Думаю, это более или менее академично; если вы вызываете strcat() для многогигабайтных строк, у вас, вероятно, будут всевозможные проблемы.

Вот версия на основе указателя, для полноты картины:

char *
my_strcat(char *dest, const char *src)
{
    char *rdest = dest;

    while (*dest)
      dest++;
    while (*dest++ = *src++)
      ;
    return rdest;
}

Конечно, для возвращаемого значения rdest требуется еще один указатель, но я думаю, что это хороший компромисс.

Также обратите внимание, что вы не можете легально определить функцию с именем strcat() в обычном коде приложения; все это пространство имен (общедоступные функции с именами, начинающимися с str) зарезервировано для реализации.

dest должен иметь достаточно памяти для объединения в этой реализации. В этой реализации он должен быть выделен вызывающей стороной. Вы также должны быть уверены, что и dest, и src завершаются нулем. Если у dest недостаточно памяти, это перезапишет память, которая может быть использована чем-то другим.

У меня работает нормально, проверял.

    #include "stdio.h"


    char *strcat(char *dest, const char *src)

    {

    size_t i,j;

    for (i = 0; dest[i] != '\0'; i++)

        ;

    for (j = 0; src[j] != '\0'; j++)

        dest[i+j] = src[j];

    dest[i+j] = '\0';

    return dest;

}


void main(void)

{

    char a[10]={"abc"}, b[10]={"def"};

    strcat(a,b);

    printf("%s",a);

    getchar();

}

Выделите достаточно памяти для строки назначения.. т.е. по крайней мере (длина исходной строки + 1).