将主机字节顺序转换为网络字节顺序有时会导致奇怪的结果

时间:2015-04-21 09:24:58

标签: java android c++ sockets networking

我目前正在用c ++编写一个服务器,由android客户端使用。基本上我使用谷歌协议缓冲区进行数据传输,在发送protobuf包之前,我以网络字节顺序发送包大小。这是我发送一个包的功能:

bool Client::sendOnePackage(const std::string & data) {
    int32_t packageSize = data.length();
    packageSize = htonl(packageSize);

    char packageSizeBuffer[sizeof (int32_t)];
    memcpy(packageSizeBuffer, &packageSize, sizeof (int32_t));

    if (send(con.sockfd, packageSizeBuffer, sizeof (int32_t), 0) == sizeof (int32_t)) {
        if (send(con.sockfd, data.c_str(), packageSize, 0) == packageSize) {
            return true;
        }
    }
    return false;
}

然后android部分将使用以下方法处理将发送的整数转换为主机字节顺序的包:

int answerLength = ByteBuffer.wrap(answerLengthData).getInt();

现在的问题:当我通过线路发送包大小为8时,它工作得很好,android接收字节数组0,0,0,8(总是4个字节)。但是当我通过线路发送值6时,android接收奇数字节数组0,114,0,0,这导致奇怪的大包(7471104)。

我在这里遗漏了什么吗?如果您需要更多信息或更多代码,请询问它,然后发布它。

1 个答案:

答案 0 :(得分:2)

您使用packageSize

的结果覆盖htonl()的值
packageSize = htonl(packageSize);

然后将其用作传递给send()的包大小:

send(con.sockfd, data.c_str(), packageSize, 0)

将锥形值保存在不同的变量中。并且不要使用临时字符缓冲区:

bool Client::sendOnePackage(const std::string & data) {
    int32_t packageSize = data.length();
    int32_t conv_size = htonl(packageSize);

    if (send(con.sockfd, reinterpret_cast<char*>(&conv_size), sizeof (int32_t), 0) == sizeof (int32_t)) {
        if (send(con.sockfd, data.c_str(), packageSize, 0) == packageSize) { //use original size
            return true;
        }
    }
    return false;
}