문제 몇 가지 문제에

Question

나는 사용자 정의 헤더 파일이 프로젝트를 만드는 작업을하고, 실행하고있다 :

이 코드에 정확히 무슨 일이 있었는지 모르겠지만, 후 내가 fgets() 함수를 사용하기 시작했다면, 그것은 내 코드에서 일부 printf()의 출력을 멈춘 것으로 보인다. 여기에 아마 여기에 몇 가지 #include 문을 누락 코드의 중요한 부분은 다음과 같습니다

main.cpp

#include "ichat.h" 

int main() { 

    prompts(P_IDLE, response); 
    prompts(P_USERNAME, response); 
} 

ichat.cpp

#include "ichat.h" 
int prompts(int p_tag, char *response) { 

    if (p_tag == P_IDLE) { 

     printf("\nWaiting for connections. Press [ENTER] to start a chat."); 
     char msg[50]; 
     char *ptr; 
     while (true) { 

      fgets(msg, strlen(msg), stdin); 
      ptr = (char*)memchr(msg, '\n', strlen(msg)); 
      if (ptr != NULL) { 

       break; 
      } 

    } else if (p_tag == P_USERNAME) { 

     printf("\nPlease enter a username you'd like to use: "); 
     .... 

    } ... 

ichat.h의 일부, 당신이 궁금했다 까봐 ...

#define P_IDLE 0 
#define P_USERNAME 1 

gcc main.cpp ichat.cpp.

fgets() 함수로 while 루프를 추가 할 때 printf() 함수 중 아무 것도 출력을 생성하지 않습니다. 인쇄 기능이 정상적으로 작동하고 있기 때문에 여기에 무슨 일이 일어나고 있는지 정확하게 알 수는 없으며 때로는 \n을 제거하거나 다른 printf() 기능을 추가하기도합니다. 꽤 혼란 스럽네요 ...

사용자가 입력하는 좋은 방법이 있습니까? 나는 나의 코드의 어떤 부분에 대한 어떤 힌트, 비평, 아이디어에 대해서도 열려있다.

Answer 1

문제의 요점은

char msg[50]; 
/* no initialisation of data in msg */ 
fgets(msg, strlen(msg), stdin); 

msg가 초기화되지 않은되고, 그래서 그 안에있는 모든 문자의 값은 불확정 - 그들은 제로 보장되지 않습니다. 형식적으로 심지어 값을 액세스 (strlen()은 '\0'을 검색)하면 정의되지 않은 동작이 발생합니다.

strlen(msg)을 sizeof msg (항상 50의 값)으로 변경해야합니다.

fgets()은 입력에 따라 msg을 수정합니다. 그런 다음 strlen()을 결과에 사용할 수 있습니다.

또한 memchr()과 같은 함수의 결과에 (char *) 유형 변환을 사용하지 말고 void *을 반환하십시오. #include <string.h>을 잊어 버린 경우 C에서만 필요합니다. 오른쪽 헤더를 포함하는 것이 유형 변환보다 나은 해결책입니다. C 컴파일러가 실제로 C++ 컴파일러 인 경우 변환이 필요하지만 C++은 대체로 더 선호되는 대체 방법을 제공합니다 (유형 안전성 등).