Ошибка в коде? [C]

Question

Я застрял в бесконечном цикле, по какой-то причине программа не заканчивается при вводе \n. Кроме того, точка кода заключается в обратном вводе, но я пытаюсь сделать это, используя указатель вместо целого, чтобы отслеживать позицию в массиве.

#define MSG_LENGTH 80 

int main(void) 
{ 
    char msg[MSG_LENGTH], *p; 

    printf("Enter a message: "); 

    for (p = msg; p < &msg[MSG_LENGTH];) 
    { 
     *p++ = getchar(); 

     if (*p == '\n') 
      break; 
    } 

    printf("Reversal is: "); 

    for (p--; p >= msg;) 
     putchar(*p--); 

    return 0; 

} 

Answer 1

for (p = msg; p < &msg[MSG_LENGTH];) 
{ 
    *p++ = getchar(); <-- Getting character in *p and incrementing p 

    if (*p == '\n') <-- Checking value of next character, not the one just read 
     break; 
} 

Возможная коррекция:

for (p = msg; p < &msg[MSG_LENGTH]; p+=sizeof(char)) 
{ 
    *p = getchar(); 

    if (*p == '\n') 
     break; 
} 

Answer 2

Попробуйте это:

#define MSG_LENGTH 80 

int main(void) 
{ 
    char msg[MSG_LENGTH], *p; 

    printf("Enter a message: "); 

    for (p = msg; p < &msg[MSG_LENGTH];) 
    { 
     *p = getchar(); 
     if (*p == '\n') 
      break; 
     p++; 
    } 

    printf("Reversal is: "); 

    for (p--; p >= msg;) 
     putchar(*p--); 

    return 0; 

} 

Вы были приращение p указателя до состояния break. Таким образом, даже если *p содержит '\n', проверка делалась после увеличения местоположения указателя.