strcat(); печать дополнительных символов

Question

Я сам выполнил функцию strcat(), но после добавления строки он печатает дополнительный символ ascii. Пожалуйста, скажите почему?

#include<stdio.h> 
    #include<conio.h> 
    #include<string.h> 
    void xstrcat(char string1[],char string2[]); 
    void main(void) 
{ char x[100]; 
    char string1[40],string2[40]; 
    printf("Enter a string:"); 
    gets(string1); 
    puts("Enter another string:"); 
    gets(string2); 
    xstrcat(string1,string2); 
    printf("%s",string1); 
    getch(); 
} 
    void xstrcat(char string1[],char string2[]) 
{ 
    int i,x,y; 
    x=strlen(string1); 
    y=strlen(string2); 
    for(i=0;i<y;i++) 
    { string1[i+x]=string2[i]; 
    }//for ends 
} 

Answer 1

Ваша xstrcat() функция не помещая нулевой терминатор символ в конце результирующей строки.

Одним из возможных решений может быть поставить следующий раз, прежде чем функция xstrcat() возвращает:

string1[x+y] = '\0'; 

Answer 2

В C, строки оканчиваются байт NUL (символ со значением 0). strlen расскажет вам, сколько символов есть от начала строки до NUL-байта, не считая самого байта NUL.

Итак, когда вы выполняете эту петлю:

for(i=0;i<y;i++) 
    { string1[i+x]=string2[i]; 
    } 

Вы никогда не копировать завершающий NUL байт из string2 в string1 и так string1 больше не имеет завершения NUL (вы перезаписали его NUL в начале цикла с первый символ string2). Когда строка пропускает свой завершающий NUL, функции, которые ее читают (например, printf), продолжат чтение за конечной точкой строки, пока они, в конце концов, не найдут нулевой байт где-то еще в памяти. Это может привести к печати дополнительных символов и/или сбою.

Либо изменение y к y+1 или явно вставить '\0' в положении x+y в string1.

Answer 3

void xstrcat(char string1[],char string2[]) 
{ 
    //int i,x,y; 
    size_t i, x, y; 
    // They could also be unsigned, but size_t is an unsigned big enough to hold the 
    // biggest in memory index possible 

    x=strlen(string1); 
    y=strlen(string2); 

    //for(i=0;i<y;i++) 
    for (i=0; i<=y; i++) // This picks up the null at the end 
    { string1[i+x]=string2[i]; 
    }//for ends 
} 

В качестве альтернативы вы могли бы сделать это так:

void xstrcat(char * string1, const char * string2) 
{ 
    while(*string1) { 
     string1++; 
    } 
    strcpy(string1, string2); 
} 

Это должно быть немного быстрее, потому что он не должен проходить либо строку, но один раз. Он также не требует столько дополнительных переменных.

Answer 4

ваш цикл

for (i = 0; i < y; i++) { 
    string1[i+x]=string2[i]; 
} 

вы читаете строку 2 до его длины-1, которая будет читать только ДО последнего символа до NULL, если вы не знаете, в С или С строк ++ оканчиваются с NULL (0).

Таким образом, ваша строка1 не будет иметь символа NULL в конце.

Итак, теперь, когда вы пытаетесь напечатать свою строку1, в конце концов она обязательно напечатает несколько символов, поскольку ваша строка имеет размер 40, эти символы - мусор.

В основном вы получите натянуть 40 символов печатается за исключением только тогда, когда на этих значений мусора является 0.

Таким образом, вы должны прочитать вторую строку Шифрование до 0, так что ваш цикл

for (i = 0; i <= y; i++) 

или вы можете просто добавить символ NULL в конце строки 1 после добавления в него строки2.

string1[x+y] = 0;