0

我真的是 C 的新手，但我写 C++ 已经有一段时间了。我正在编写一个客户端服务器聊天程序。在用户输入用户名后，我需要在会话开始时向用户提示几个不同的选项。起初我试图使用 getchar() 函数，但由于某种原因，以下模式的任何语句都不会产生预期的结果：

int x = getchar();
if (x == '2') doSomething();

如果用户输入 2，它将永远不会进入“doSomething”区域。所以我尝试改用 fgets 和 strncmp 。但是现在，我不断在 strncmp 上遇到分段错误。这是代码中最相关的部分，在我尝试使用 getchar 时有一些注释掉的部分。诚然，这有点混乱，因为我只是把它放在一起作为测试。我想也许为字符串分配额外的空间将有助于防止段错误，但当然它没有。

for( ; ; )
{
  printf("\r\n1.List Users \r\n2.Chat \r\n3.Exit \r\n \r\n \r\n");

  char *x = malloc(5);

  fgets(x, 2, stdin);

  if (x[0] != NULL)
    {

      if (strncmp (x[0],"a",1) == 0)
        {
          printf("yay");
        }
    }


/* int x = getchar();
  if(x == 'a') // Compare input to 'q' character
    break;
  fprintf(stdout, "%d\n", x);*/

  /*x = c - '0';

  if (x == 1)
    getUsers(sockfd);

  if ( x == 2 )
    {

      pthread_create(&sndThread, NULL, do_send, (void *) sockfd);
      pthread_create(&rcvThread, NULL, do_recv, (void *) sockfd);

      pthread_join(sndThread, NULL);
      pthread_join(rcvThread, NULL);
    }

  if ( x == 3 )
    {
    close(sockfd);
    exit(0);
    }*/
}

您可以在剩余的评论中看到尝试做一些事情的剩余部分，例如用减法将 char 转换为 int。这来自我在互联网上找到的东西。我还在互联网上听说 getchar 将 \n 留在输入缓冲区中。

因此，这是我为客户端编写的完整代码，因此您可以将其放在上下文中：

int main(int argc, char **argv)
{
  int sockfd, i;

  char *myName = malloc(MSGSIZE);

  char c;

struct sockaddr_in servaddr;

int status;

pthread_t sndThread;
pthread_t rcvThread;

if(argc != 2)
  {
    printf("Error: expected IP address argument");
    exit(1);
}
  if( (sockfd = socket(AF_INET, SOCK_STREAM, 0)) < 0)
{

  error("Socket error");
}

 memset(&servaddr, 0, sizeof(servaddr));
 servaddr.sin_family = AF_INET;
servaddr.sin_port = htons(PORTNUM);

if(inet_pton(AF_INET, argv[1], &servaddr.sin_addr) <=0)
{
  printf("inet_pton error for %s \n", argv[1]);
  exit(3);
}

if(connect(sockfd, (SA *) &servaddr, sizeof(servaddr)) < 0)
{
  error("Connect error");
}

printf("Type in a username: \r\n");

while ( fgets(myName[i],MSGSIZE,stdin ) == NULL){}


printf(">%s<\n",myName);

send_userName(myName,sockfd);

for( ; ; )
{
  printf("\r\n1.List Users \r\n2.Chat \r\n3.Exit \r\n \r\n \r\n");

  char *x = malloc(5);

  fgets(x, 2, stdin);

  if (x[0] != NULL)
    {

      if (strncmp (x[0],"a",1) == 0)
        {
          printf("yay");
        }
    }


/* int x = getchar();
  if(x == 'a') // Compare input to 'q' character
    break;
  fprintf(stdout, "%d\n", x);*/

  /*x = c - '0';

  if (x == 1)
    getUsers(sockfd);

  if ( x == 2 )
    {

      pthread_create(&sndThread, NULL, do_send, (void *) sockfd);
      pthread_create(&rcvThread, NULL, do_recv, (void *) sockfd);

      pthread_join(sndThread, NULL);
      pthread_join(rcvThread, NULL);
    }

  if ( x == 3 )
    {
    close(sockfd);
    exit(0);
    }*/
   }

}

4

3 回答 3

2

x[0]是一个字符，但是x是一个char*. strncmp应该只是x作为一个论点，而不是x[0]. 也就是说，你不想

strncmp(x[0],"a",1)

反而

strncmp(x,"a",1)

或者，如果您真的想强调您从 的第一个字符开始x，您可以执行以下任一操作：

strncmp(x+0,"a",1)


strncmp(&x[0],"a",1)
于 2014-10-09T15:24:00.803 回答
0

的行为getchar()取决于终端的模式。大多数都在“熟”模式下运行，这意味着getchar()在您输入一整行（并按下回车键）后返回。终端这样做是为了允许行编辑。要getchar()立即返回，您需要将其切换为“原始”模式。

接下来，您应该启用所有编译器警告，因为它会告诉您上面的代码有什么问题：

strncmp()期望char*作为第一个参数，但您通过了char. 这意味着代码将从任意内存中读取。

x[0] != NULL也没有意义（将字符与空指针进行比较）。要知道是否fgets()没有返回任何内容，请查看其返回码。

char * success = fgets(x, 2, stdin);
if(success == null) { ... error handling... }

if (strncmp (x,"a",1) == 0) {
    printf("yay");
}
于 2014-10-09T15:26:16.423 回答
0

对未来的评论

#include从完整源代码中删除这些行通常没有帮助。如果我想去编译你的代码，我现在必须花几分钟时间把它们放在一起。这是浪费时间。

这是我需要添加的额外标题：

#include <arpa/inet.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <stdlib.h>
#include <stdio.h>
#include <string.h>

#define MSGSIZE 100
#define PORTNUM 12

#define SA struct sockaddr

void error(const char *);
void send_userName(const char *, int);

接着...

因此，如果我添加这些标头，并尝试编译您的代码，我会收到一些非常可怕的警告。让我们看看第一类警告。

此类是您将变量传递给正在寻找不同类型变量的函数的地方。

foo.c:59:19: warning: incompatible integer to pointer conversion passing 'char' to parameter of type 'char *'; take the address with & [-Wint-conversion]
    while ( fgets(myName[i],MSGSIZE,stdin ) == NULL){}
                  ^~~~~~~~~
                  &
/usr/include/stdio.h:236:30: note: passing argument to parameter here
char    *fgets(char * __restrict, int, FILE *);
                                ^
foo.c:74:18: warning: comparison between pointer and integer ('int' and 'void *')
        if (x[0] != NULL)
            ~~~~ ^  ~~~~
foo.c:77:26: warning: incompatible integer to pointer conversion passing 'char' to parameter of type 'const char *'; take the address with & [-Wint-conversion]
            if (strncmp (x[0],"a",1) == 0)
                         ^~~~
                         &
/usr/include/string.h:84:26: note: passing argument to parameter here
int      strncmp(const char *, const char *, size_t);
                             ^

第二类是使用未初始化变量的地方：

foo.c:59:26: warning: variable 'i' is uninitialized when used here [-Wuninitialized]
    while ( fgets(myName[i],MSGSIZE,stdin ) == NULL){}
                         ^
foo.c:18:18: note: initialize the variable 'i' to silence this warning
    int sockfd, i;
                 ^
                  = 0

你真的应该尝试修复这些，因为否则你的代码会被破坏。如果您不明白为什么会触发特定警告，那么您应该询问一下。

于 2014-10-09T15:32:23.157 回答