Memset and characters

Question

Я хотел бы скопировать source строку в номер dest. Если я скомпилирую следующую программу:

#include <stdio.h> 

int main(void) { 
    char dest[6]; 
    char source[6]; 

    strcpy(dest,source); 

    while (*dest) { printf("%c",*dest++); } 
    while (*source) {printf("%c",*source++); } 

    return 0; 
} 

Я получаю ошибку времени выполнения. Я подозреваю, что это потому, что strcpy копирует от источника к месту назначения, пока не встретит \0. Однако он не столкнулся с нулевым символом и продолжал копировать из буфера до возникновения ошибки времени выполнения. Чтобы решить эту проблему, я изменил код следующим образом:

#include <stdio.h> 

int main(void) { 
    char dest[6]; 
    char source[6]; 


    memset(dest, '\0', 6*sizeof(dest)); //trying to set dest to '/0' 
    strcpy(dest,source); 

    while (*dest) { printf("%c",*dest++); } 
    while (*source) {printf("%c",*source++); } 

    return 0; 
} 

я получаю следующие ошибки:

prog.c:11:38: error: lvalue required as increment operand

while (*dest) { printf("%c",*dest++); } 
           ^

и

prog.c:11:38: error: lvalue required as increment operand

while (*dest) { printf("%c",*source++); } 
            ^

Почему это происходит?

Answer 1

Для начала он является исходным массивом, который должен быть нулевой прекращенным, если вы собираетесь скопировать его в других символьных массивах используя стандартную функцию C. strcpy. Таким образом, вместо этого заявления

memset(dest, '\0', 6*sizeof(dest)); 

вы должны по крайней мере написать

memset(source, '\0', 6*sizeof(source)); 
     ^^^^^^    ^^^^^^^ 

Однако даже это утверждение неверно, потому что переписывает память, выделенную для массива. sizeof(source) уже равна 6 байт, как это следует из декларации массива

char source[6]; 

Таким образом, вы должны написать

memset(source, '\0', sizeof(source)); 
        ^^^^^^^^^^^^^ 

На самом деле там было достаточно, чтобы написать либо как

char source[6] = { '\0' }; 

или аналогично

char source[6] = ""; 

или как

char source[6]; 
source[0] = '\0'; 

Массивы являются неизменяемой lvalues.Таким образом, вы не можете писать, например, следующим образом

while (*dest) { printf("%c",*dest++); } 

Вместо этого заявления можно было бы написать

for (char *p = dest; *p; ++p) { printf("%c", *p); } 

Примите во внимание, что ничего не будет выводиться, поскольку массив содержит пустую строку. Вы можете инициализировать исходный массив некоторым непустым строковым литералом.

Answer 2

strcpy не является надежной функцией, предпочитайте использовать strncpy.

Ошибка связана с тем, что вы пытаетесь увеличить массив, что является rvalue (т. Е. Константой, вы не можете поместить ее в левую сторону знака =).

Общий подход перебрать массив должны использовать указатель, как так:

char *p = dest; 
while (*p) { printf("%c",*p++); } 

Answer 3

следующий код полностью компилирует и выполняет требуемую операцию.

Различия между опубликованным кодом и этим комментарием.

#include <stdio.h> // printf() 
#include <string.h> // strcpy() 

int main(void) 
{ 
    char dest[6]; // declared, containing garbage 
    char source[6] = "12345"; // declared, containing the string "12345\0" 

    strcpy(dest,source); 
    // now both arrays contain the string "12345\0" 

    // best to use a 'for()' statement for indexing through an array 
    for(size_t i=0; dest[i]; i++) { printf("%c", dest[i]); } 
    printf("\n"); // output the buffered data to the terminal 
    for(size_t i=0; source[i]; i++) { printf("%c", source[i]);} 
    printf("\n"); // output the buffered data to the terminal 

    // note, the following lines contain a precedence problem in 
    // the increment expressions and 
    // the address of an array declaration cannot be incremented 
    //while (*dest) { printf("%c",*dest++); } 
    //while (*source) {printf("%c",*source++); } 

    //return 0;// with modern C compilers, 
      // this line is not necessary in a 'main()' function 
      // when returning 0 
} // end function: main