Winsock服务器打印消息时客户端名称重复显示问题求助

我来帮你分析这个问题的原因，以及怎么解决它！

首先看你的输出，每次客户端发消息都会出现两行带主机名的内容，一行有消息，一行空——这背后有两个主要原因：

问题原因 #

1. TCP是无边界的字节流
   PuTTY发送的消息（比如你输入hi后按回车，实际发送的是hi\r\n）可能被TCP协议拆分成多个数据包传输。你的服务器每次调用recv都会读取当前可用的数据，所以可能第一次读到hi，第二次读到\r\n。而你每次recv成功后都会执行输出语句，自然就会打印两次<host>，一次带消息，一次带换行符（看起来是空行）。

2. 未处理客户端发送的换行控制符
   客户端发送的消息里包含\r（回车）和\n（换行），当你直接用cout输出buf时，这些控制字符会被控制台解析，导致输出格式混乱，出现额外的空行。

另外，你的代码里还有一个小bug：send(clientSocket, 0, bytesReceived, 0);这里第二个参数传了0（空指针），这会导致未定义行为，根本没法正确把消息回发给客户端，得赶紧改掉！

解决方案 #

我们可以通过处理消息中的控制字符，同时优化recv后的输出逻辑来解决这个问题，步骤如下：

1. 截断消息中的换行符
   在输出前，把buf里的\r和\n去掉，只保留实际的消息内容。可以用strcspn函数快速找到第一个换行符的位置，然后截断字符串。

2. 只输出非空消息
   添加判断，避免因为读到单独的\r\n而输出空的<host>行。

3. 修复send语句
   把send的第二个参数改成buf，正确回发消息给客户端。

修改后的代码片段 #

这里是修改后的循环部分，你可以直接替换原代码里的循环：

while (true) {
    ZeroMemory(buf, 4096);
    // Wait for client to send data
    int bytesReceived = recv(clientSocket, buf, 4096, 0);
    if (bytesReceived == 0) {
        // Print Client disconnected on the console
        cout << endl << "Client disconnected (" << host << ")" << endl;
        // Close socket; because the client disconnected
        closesocket(clientSocket);
        system("pause>nul");
        break;
    }
    // 处理recv错误的情况，增强健壮性
    if (bytesReceived == SOCKET_ERROR) {
        cerr << "recv failed! Error code: " << WSAGetLastError() << endl;
        break;
    }
    // 找到第一个\r或\n的位置，截断字符串
    size_t newlinePos = strcspn(buf, "\r\n");
    buf[newlinePos] = '\0';
    // 只输出非空的消息内容
    if (strlen(buf) > 0) {
        cout << " <" << host << "> " << buf << endl;
    }
    // 修复send语句，正确回发消息
    send(clientSocket, buf, strlen(buf), 0);
}

额外说明 #

如果你需要更严谨的消息处理（比如避免TCP粘包/拆包的影响），可以和客户端约定一个消息边界，比如每条消息以特定字符（比如\n）结尾，服务器端循环读取数据直到遇到这个边界再处理——不过对于简单的测试服务器，上面的修改已经足够解决你的问题了。

内容的提问来源于stack exchange，提问作者Ryuk