我如何停止我的 tcp echo 服务器以正确的方式工作？答案

【问题标题】：How do i stop my tcp echo server to work the right way?我如何停止我的 tcp echo 服务器以正确的方式工作？
【发布时间】：2015-09-17 00:35:45
【问题描述】：

// A simple echo server
// ./server port_no

#include<stdio.h>
#include<stdlib.h>
#include<sys/socket.h>
#include<sys/types.h>
#include<netinet/in.h>
#include<error.h>
#include<strings.h>
#include<unistd.h>
#include<arpa/inet.h>



#define ERROR -1
#define MAX_CLIENTS 2
#define MAX_DATA 1024



main(int argc, char **argv)
{
  struct sockaddr_in server;
  struct sockaddr_in client;
  int sock;
  int new;
  int sockaddr_len = sizeof(struct sockaddr_in);
  int data_len;
  char data[MAX_DATA];


  if((sock = socket(AF_INET , SOCK_STREAM, 0)) == ERROR)
  {
       perror("server socket: ");
       exit(-1);   
  }

  server.sin_family = AF_INET;
  server.sin_port = htons(atoi(argv[1]));
  server.sin_addr.s_addr = INADDR_ANY;
  bzero(&server.sin_zero,8);


  if((bind(sock , (struct sockaddr *)&server, sockaddr_len)) == ERROR)
  {
          perror("bind : ");
          exit(-1);
  }

  if((listen(sock, MAX_CLIENTS)) == ERROR)
  {
         perror("listen");
         exit(-1);
  }          

  while(1) //Better signal handling required
  {
      if((new == accept(sock, ( struct sockaddr *)&client , &sockaddr_len)) == ERROR)
      {
         perror("accept"); 
         exit(-1); 
      }


 printf("New Client connected from port no %d and IP %s\n",ntohs(client.sin_port),inet_ntoa(client.sin_addr));
 data_len = 1;

 while(data_len)
    {      
       data_len = recv(new,data , MAX_DATA, 0);

             if(data_len)
               {
                    send(new, data, data_len, 0);
                    data[data_len] = '\0' ; // null the last byte to detect string
                    printf("Sent mesg: %s", data);
               }
    }

 printf("Client disconnected\n");

 close(new);
}


close(sock);

}

您好，我一直在学习 C 中的套接字编程。最近我为 tcp echo 服务器编写了一个代码。当我使用 netstat 命令时，服务器确实显示已连接，但是一旦我使用 telnet 运行代码，它就会开始打印无限消息，而不是等待我在其他终端中写入消息。请帮我弄清楚我哪里出错了。

【问题讨论】：

请告诉我们实际输出是什么。正在打印的消息显然很重要，因为它会告诉程序卡在哪里。但是对于初学者来说，while(data_len) 和 if (data_len) 是错误的。 recv 在错误时返回 -1，在 C 中，任何非零值在用作条件时都会计算为真。所以这些检查不能正确区分成功和错误。

标签： c sockets server client

【解决方案1】：

死循环的原因是代码

f((new == accept(sock, ( struct sockaddr *)&client , &sockaddr_len)) == ERROR)

应该是

f((new = accept(sock, ( struct sockaddr *)&client , &sockaddr_len)) == ERROR)

这是 == 与 =。使用 ==，accept() 返回的套接字永远不会分配给 new，因此在其上调用 recv() 会返回 -1（在这种情况下为错误），并且因为代码不检查错误，正如 Alan 指出的那样au，结果是死循环。

顺便说一句，如果您观看编译器警告或使用开关启用更多警告，例如-Wall 与 gcc，您通常可以很好地了解代码可能存在的问题。

【讨论】：