Question

我使用以下代码从我的android手机整数作为字节发送（我只放置了所需的块）：

private byte[] intToByteArray ( final int i ) throws IOException {      
    ByteArrayOutputStream bos = new ByteArrayOutputStream();
    DataOutputStream dos = new DataOutputStream(bos);
    dos.writeInt(i);
    dos.flush();
    return bos.toByteArray();
}
//thread sender
        data = new byte[16];
        ...
        try {
            temp = intToByteArray((int) (roll*100));
        } catch (IOException e1) {
            // TODO Auto-generated catch block
            e1.printStackTrace();
        }
        data[3] = temp[0];
        data[4] = temp[1];
        data[5] = temp[2];
        data[6] = temp[3];
        ...
        p = new DatagramPacket(data, 16);
        udpSocket.send(p);

发送代码有效，因为我已经尝试了收到的Java，结果是正确的。真正的接收器必须使用嵌入式设备实现，Murata SN8200：我收到数据包，但我的整数结果（例如，roll）不正确。我用于从byte转换为int的代码如下：

void getSubArray(char* dest, char* source, int start, int end){
   uint8_t i, j = 0;
   for(i=start; i<end; i++){
       dest[j] = source[i];
       j++;
   }
}
int byteToInt(char* bytes)
{
  int ch1 = bytes[0];
  int ch2 = bytes[1];
  int ch3 = bytes[2];
  int ch4 = bytes[3];
  if ((ch1 | ch2 | ch3 | ch4) < 0)
      WPRINT_APP_INFO ( ("boh\r\n"));
  return ((ch1 << 24) + (ch2 << 16) + (ch3 << 8) + (ch4 << 0));
}
...
//thread receiver
wiced_packet_get_data(packet, 0, (uint8_t**)&rx_data, &rx_data_length, &available_data_length);
traffic_type = rx_data[0];
        id = rx_data[1];
        takeOffPressed = rx_data[2];
        getSubArray(temp, rx_data, 3, 7);
        roll = byteToInt(temp);
        WPRINT_APP_INFO ( ("Rx: \" Packet Type: %c Id: %c  TakeOffPressed = %c \r\n  roll: %d \r\n ", (unsigned char )traffic_type, (unsigned char )id, (unsigned char)takeOffPressed, roll);

阅读我已经尝试过的其他讨论：roll =（int *）（temp）;正如另一篇文章中所建议的，但它对我没有用（别名编译器错误）。有什么想法解决这个问题吗？感谢

Answer 1

#include <sdint.h>
#include <string.h>
#include <netinet/in.h>

uint32_t byteToInt(char* bytes)
{
 uint8_t b[4];
 memcpy(b, bytes, 4);
 if ((b[0] | b[1] | b[2] | b[3]) < 0)WPRINT_APP_INFO ( ("boh\r\n"));
 return ntohl(*((uint32_t *)b));
}
 ...
 //thread receiver
wiced_packet_get_data(packet, 0, (uint8_t**)&rx_data, &rx_data_length, &available_data_length);
traffic_type = rx_data[0];
id = rx_data[1];
takeOffPressed = rx_data[2];
memcpy(temp, rx_data+3, 4);
roll = byteToInt(temp);
WPRINT_APP_INFO ( ("Rx: \" Packet Type: %c Id: %c  TakeOffPressed = %c \r\n  roll: %d \r\n ", (unsigned char )traffic_type, (unsigned char )id, (unsigned char)takeOffPressed, roll);

不需要

void getSubArray(char* dest, char* source, int start, int end)

您可以用

替换它

memcpy(void* dest, void* src, size_t t);//from string.h

另外我已经将你的byteToInt函数修改为一个更有意义的东西

Answer 2

你有一些可能的签名混淆。考虑一下您的byteToInt()功能：

int byteToInt(char* bytes)
{
  int ch1 = bytes[0];
  int ch2 = bytes[1];
  int ch3 = bytes[2];
  int ch4 = bytes[3];
  if ((ch1 | ch2 | ch3 | ch4) < 0)
      WPRINT_APP_INFO ( ("boh\r\n"));
  return ((ch1 << 24) + (ch2 << 16) + (ch3 << 8) + (ch4 << 0));
}

假设发送的4字节值为511（十进制）。然后发送的字节为0x00 0x00 0x01 0xff。现在假设接收系统具有签名的，两个补码char，这并不罕见。在这种情况下，在接收器上，这些字符将被解释为值0 0 1 -1（十进制）。现在有两个问题：

将移位运算符应用于负左操作数的行为（即我的示例中的ch4的值）未定义。
-1 << 0的合理可能实际行为是产生-1（评估其左操作数值的零位移位）。在这种情况下，总计算值为256 + -1 = 255。

当然，这仅仅是一个例子，但只有十六个32位int中的一个具有清除每个字节的高位。

可以通过使用适当的无符号数量执行计算来解决这两个问题。这可以这样做：

int byteToInt(char* bytes)
{
  uint8_t ch1 = bytes[0];
  uint8_t ch2 = bytes[1];
  uint8_t ch3 = bytes[2];
  uint8_t ch4 = bytes[3];
  return ((ch1 << 24) + (ch2 << 16) + (ch3 << 8) + (ch4 << 0));
}

另请注意，提取子数据以传递给byteToInt()是浪费的。最好只使用一些指针算法：

roll = byteToInt(rx_data + 3);

byte到int类型转换网络应用程序

2 个答案: