1

我正在编写一个简单的服务器/客户端程序来将文件从客户端发送到服务器。我正在使用winsock2。我将每次发送数据的容量限制为 5000。

客户端（发送）：

int iResult = 0;
int totalBytesSent = 0;

while (length > 0){
    iResult = send( _connectSocket, data, MAX_TRANSIT_SIZE, 0 ); // MAX_TRANSIT_SIZE is 5000
    if (iResult == SOCKET_ERROR) {
        printf("send failed with error: %d\n", WSAGetLastError());
        return closeSocket();
    }
    totalBytesSent += iResult;
    length -= iResult;
    //cout << "Data sent (" << iResult << " Bytes)" << endl;
}
cout << "Total Bytes Sent: (" << totalBytesSent << ")" << endl;

return 0;

在服务器端（recv）：

    // Receive and send data
char recvbuf[MAX_DATA_SIZE];

int iResult = 0;
int totalBytesRead = 0;

// Receive until the peer shuts down the connection

do {
    totalBytesRead += iResult;
    iResult = recv(_clientSocket, recvbuf, MAX_DATA_SIZE, 0);

    if (iResult > 0) {
        //printf("RECEIVED DATA\n");
        //printf("Bytes received: %d\n", iResult);

    } else if (iResult == 0)
        printf("Connection closing...\n");
    else {
        printf("recv failed: %d\n", WSAGetLastError());
        closesocket(_clientSocket);
        WSACleanup();
        return 1;
    }

} while (iResult > 0);

cout << "Total Bytes Received: (" << totalBytesRead << ")" << endl;

问题：

运行客户端和服务器并发送文件后，它确实显示发送/接收的正确数据大小（当然是以字节为单位的文件大小），但是输出文件不同，当我用一些文本编辑器（记事本++ ) 我可以清楚地看到输出文件包含较少的数据（但 File->Properties 显示相同的文件大小）并且一些数据是重复的。

我的问题：

revc() 是如何工作的？如果它在许多调用中接收数据，它会在缓冲区中累积数据吗？（在我的情况下：recvbuf）还是重写缓冲区？据我所知，它确实会累积，所以我的代码是正确的？？

谢谢。

4

2 回答 2

2

我看不到您在 recvbuf 中的写入位置。每次调用 recv 时，它都会覆盖 recvbuf 中已有的内容。因此，如果您有注释掉的“接收数据”打印，您应该复制要保留在缓冲区之外的数据。

于 2013-03-02T13:20:05.867 回答
2

您的代码中有几个问题。

客户端：

send( _connectSocket, data, MAX_TRANSIT_SIZE, 0 );

  • 在这里，您永远不会更新data已发送的字节数，因此每次调用send它时都会一次又一次地发送相同的数据（缓冲区的第一个MAX_TRANSIT_SIZE字节）。data一个快速修复，假设data是指向任何字节类型（uint8_t，char，...）的指针将是：

    send( _connectSocket, data + totalBytesSent, MAX_TRANSIT_SIZE, 0 );

  • 您还应该限制您发送的数据大小，因为除非length最初是倍数，否则MAX_TRANSIT_SIZE当您到达数据末尾时会出现缓冲区溢出：

    send( _connectSocket, data + totalBytesSent, std::min(length, MAX_TRANSIT_SIZE), 0 );

服务器端：

recv(_clientSocket, recvbuf, MAX_DATA_SIZE, 0);
  • 就像send,recv没有“我在那个缓冲区中已经收到了什么”的概念。因此，每次调用recv它时，只需将它接收到的新数据放在缓冲区的开头，覆盖旧数据。这可能是也可能不是您想要的，这很难说，因为您没有向我们展示您如何使用该缓冲区。您可能希望使用与我刚刚为您的发送缓冲区解释的方法相同的方法来管理您的接收缓冲区。
于 2013-03-02T13:27:49.740 回答