Что случилось с моей strcpy?

Question

Я сам пытался сделать strcpy. Он должен работать, я даже скопировал и вставлял (почти точный код) из сообщения someones здесь около strcpy. Оба дают мне «Ошибка сегментации».

char* strcpy(char * destination, const char * source) 
{ 
    while((*destination++ = *source++) != '\0') 
     ; 
    return destination; 
} 

Что не так с этим кодом?

char* a = "hello"; 
cout << strcpy(a, "Haha") << endl; 

Answer 1

Вы пытаетесь выполнить запись в сегмент данных, так как там хранится "hello".

Поэтому, когда вы вызываете strcpy, вы получаете ошибку сегментации.

Try:

char a[] = "hello"; 
cout << strcpy(a, "Haha") << endl; 

вместо этого.

EDIT: Внутри вашей функции strcpy после копирования destination укажет на конец строки, вам нужно вместо этого начать начало строки.

Answer 2

a является указателем на строку буквального:

char* a = "hello"; 

Попытка изменить строку литерал неопределенное поведение. Как предположил Карл, если инициализировать массив строкового литерала вместо этого он будет работать:

char a[] = "hello" ; 

Answer 3

Вы пытаетесь перезаписать строку литерал. Это вызывает неопределенное поведение. Объявить a как массив вместо:

char a[] = "hello"; 

Вашей strcpy реализация имеет ошибку, тоже (при условии нормальной семантики). Он должен вернуть указатель на начало буфера назначения, а не конец.

Answer 4

Помимо всего, что указано выше и ниже о строковых литералах, вы также возвращаете указатель на строку END, поэтому, даже если вы избегаете ошибки сегментации, вы будете печатать «».