копирование массива символов в другой массив дает значение для мусора

Question

Я делаю простую программу шифрования на основе ascii. ниже мой код, и я не уверен, почему я получаю значение мусора в конце при печати скопированной строки.

output results

#include <iostream> 
#include <conio.h> 

using namespace std; 

void encrypt(char []); 
void convertToAscii(char [], char []); 
int main() 
{ 
    char userString [] ="This is a string"; 
    const int size= sizeof(userString); 
    char copyString[size]; 
    convertToAscii(userString, copyString); 
    cout<<copyString; 
    _getch(); 
    return 0; 
} 

void convertToAscii(char s[], char* cs) 
{ 
    for(int i=0; s[i] != '\0'; i++) 
    { 
     int x = s[i]; 
     x= x+3; 
     char y= x; 
     cs[i]=y; 
     cout<< x<<" "<<y<<"\n"; 
    } 
} 

Answer 1

Просто добавьте Нуль в строку назначения

void convertToAscii(char s[], char* cs) 
{ 
    size_t i = 0; 

    for(; s[i] != '\0'; i++) 
    { 
     int x = s[i]; 
     x= x+3; 
     char y= x; 
     cs[i]=y; 
     cout<< x<<" "<<y<<"\n"; 
    } 

    cs[i] = '\0'; 
} 

Другой способ заключается в следующем

char * convertToAscii(cosnt char *s, char *cs) 
^^^^^^     ^^^^^ 
{ 
    char *p = cs; 

    for (; (*cs = *s) != '\0'; ++cs, ++s) *cs += 3; 

    return p; 
} 

Answer 2

В C вы должны обнулить-аннулирует ваши строки. Вы признаете, что, поскольку ваша функция convertToAscii() ищет нулевой ограничитель на входе; но он не помещает нулевой ограничитель на выходе, поэтому методы, такие как cout's operator<<, не знают, где заканчивается значение copyString.

Конечно, для того чтобы convertToAscii функции в нуль-прекратить вашу строку, вам необходимо выделить дополнительное пространство для '\0' в вызывающем:

char copyString[size + 1]; 
//     ^^^