套接字和 telnet 有问题

Question

我一直在学习套接字，并创建了一个基本服务器，您可以在其中远程登录并键入消息，然后按 Enter 键，消息将打印在服务器上。

由于是远程登录，每次按键都会发送到服务器。所以我基本上将所有发送的字节保存在一个缓冲区中，然后当收到回车符 ("\r\n") 时，我将其丢弃，并打印出客户端的当前缓冲区。然后我清除客户端缓冲区。

我的问题是每隔一段时间（我不太确定如何复制它），我发送的第一“行”数据会在每个字符上附加一个额外的空间。例如，我将在 telnet 客户端上键入“Test”，但我的服务器将接收它作为“T e s t”。我总是在接收任何数据之前清除接收缓冲区。一个明显的解决方案是删除服务器端的所有空格，但这会破坏我发送多个单词的能力。这只是我的 telnet 的问题，还是我可以在服务器上做些什么来解决这个问题？

我正在使用 WinSock2 API 和 Windows 10 Telnet。

编辑： 我检查了额外字符的十六进制值，它是 0x20。

编辑： 下面是接收和处理传入 telnet 数据的代码。

// This client is trying to send some data to us
                memset(receiveBuffer, sizeof(receiveBuffer), 0);
                int receivedBytes = recv(client->socket, receiveBuffer, sizeof(receiveBuffer), 0);
                if (receivedBytes == SOCKET_ERROR)
                {
                    FD_CLR(client->socket, &masterFDSet);
                    std::cerr << "Error! recv(): " << WSAGetLastError() << std::endl;
                    closesocket(client->socket);
                    client->isDisconnected = true;

                    continue;
                }
                else if (receivedBytes == 0)
                {
                    FD_CLR(client->socket, &masterFDSet);
                    std::cout << "Socket " << client->socket << " was closed by the client." << std::endl;
                    closesocket(client->socket);
                    client->isDisconnected = true;

                    continue;
                }

                // Print out the hex value of the incoming data, for debug purposes
                const int siz_ar = strlen(receiveBuffer);
                for (int i = 0; i < siz_ar; i++)
                {
                    std::cout << std::hex << (int)receiveBuffer[i] << " " << std::dec;
                }
                std::cout << std::endl;

                std::string stringCRLF = "\r\n"; // Carraige return representation
                std::string stringBS = "\b"; // Backspace representation
                std::string commandBuffer = receiveBuffer;
                if (commandBuffer.find(stringCRLF) != std::string::npos)
                {
                    // New line detected. Process message.
                    ProcessClientMessage(client);
                }
                else if (commandBuffer.find(stringBS) != std::string::npos)
                {
                    // Backspace detected,
                    int size = strlen(client->dataBuffer);
                    client->dataBuffer[size - 1] = '\0';
                }
                else
                {
                    // Strip any extra dumb characters that might have found their way in there
                    commandBuffer.erase(std::remove(commandBuffer.begin(), commandBuffer.end(), '\r'), commandBuffer.end());
                    commandBuffer.erase(std::remove(commandBuffer.begin(), commandBuffer.end(), '\n'), commandBuffer.end());

                    // Add the new data to the clients data buffer
                    strcat_s(client->dataBuffer, sizeof(client->dataBuffer), commandBuffer.c_str());
                }

                std::cout << "length of data buffer is " << strlen(client->dataBuffer) << std::endl;

Answer 1

你有两个主要问题。

首先，您有一个变量receivedBytes，它知道您收到的字节数。那你为什么打电话给strlen？您无法保证收到的数据是 C 风格的字符串。例如，它可以包含嵌入的零字节。不要打电话给strlen。

其次，您检查刚刚收到的数据是否为\r\n，而不是完整的接收缓冲区。并且您将数据接收到接收缓冲区的开头，而不是其中的第一个未使用空间。因此，如果对recv 的一次调用得到\r，而下一次调用得到\n，那么您的代码将做错事。

您实际上从未编写过代码来接收消息。你从来没有真正创建一个 message buffer 来保存收到的消息。

Answer 2

你的代码，我的 cmets：

memset(receiveBuffer, sizeof(receiveBuffer), 0);

你不需要这个。你不应该需要这个。如果你这样做了，你的代码后面会有一个错误。

int receivedBytes = recv(client->socket, receiveBuffer, sizeof(receiveBuffer), 0);
if (receivedBytes == SOCKET_ERROR)
{
    FD_CLR(client->socket, &masterFDSet);
    std::cerr << "Error! recv(): " << WSAGetLastError() << std::endl;
    closesocket(client->socket);
    client->isDisconnected = true;

    continue;

你的意思是“休息”。你有一个错误。你关闭了套接字。没有什么可以继续的。

}
else if (receivedBytes == 0)
{
    FD_CLR(client->socket, &masterFDSet);
    std::cout << "Socket " << client->socket << " was closed by the client." << std::endl;
    closesocket(client->socket);
    client->isDisconnected = true;

    continue;

同上。你的意思是“休息”。你有一个错误。你关闭了套接字。没有什么可以继续的。

}

// Print out the hex value of the incoming data, for debug purposes
const int siz_ar = strlen(receiveBuffer);

Bzzzzzzzzzzzzt。不能保证缓冲区中的任何地方都存在空值。你不需要这个变量。 receivedBytes 中已经存在正确的值。

for (int i = 0; i < siz_ar; i++)

那应该是`for (int i = 0; i

{
    std::cout << std::hex << (int)receiveBuffer[i] << " " << std::dec;
}
std::cout << std::endl;

std::string stringCRLF = "\r\n"; // Carraige return representation

没有。那是一个回车符 (\r) 后跟一个换行符 (\n)，通常称为 CRLF，因为您确实在变量名中有自己。这是 Telnet 中的标准行终止符。

std::string stringBS = "\b"; // Backspace representation
std::string commandBuffer = receiveBuffer;

Bzzt。此副本应由receivedBytes 进行长度分隔。

if (commandBuffer.find(stringCRLF) != std::string::npos)

正如@DavidShwartz 所说，您不能假设您在同一个缓冲区中获得了 CR 和 LF。

{
    // New line detected. Process message.
    ProcessClientMessage(client);
}
else if (commandBuffer.find(stringBS) != std::string::npos)
{
    // Backspace detected,
    int size = strlen(client->dataBuffer);
    client->dataBuffer[size - 1] = '\0';

这没有任何意义。您正在使用strlen() 告诉您尾随的空值在哪里，然后您将空值放在那里。您还有一个问题，即可能没有 尾随空值。无论如何，您应该做的是删除退格键和它之前的字符，这需要不同的代码。您还在错误的数据缓冲区上进行操作。

}
else
{
    // Strip any extra dumb characters that might have found their way in there
    commandBuffer.erase(std::remove(commandBuffer.begin(), commandBuffer.end(), '\r'), commandBuffer.end());
    commandBuffer.erase(std::remove(commandBuffer.begin(), commandBuffer.end(), '\n'), commandBuffer.end());

    // Add the new data to the clients data buffer
    strcat_s(client->dataBuffer, sizeof(client->dataBuffer), commandBuffer.c_str());
}