4

Delphi で書かれた古いアプリケーションを Java に移植しています。送信されたメッセージの CRC を計算するこの関数に問題があります。

元のコードは次のとおりです。

       if(ReceivedMessage[slot].Message[1] = $FD) and
          (ReceivedMessage[slot].MessageLength in [1..16]) and
          (ReceivedMessage[slot].Message[counter] = $FF) then
       begin
        index := 2;
        ReceivedMessage[slot].DataReady := TRUE;

        for counter := 1 to ReceivedMessage[slot].MessageLength do
        begin
          Inc(index);

          if ReceivedMessage[slot].Message[index] < $F8 then
            ReceivedMessage[slot].Data[counter] := ReceivedMessage[slot].Message[index]
          else
          if ReceivedMessage[slot].Message[index] = $F8 then
          begin
            Inc(index);

            ReceivedMessage[slot].Data[counter] := ReceivedMessage[slot].Message[index] or $F0;
          end
          else
            ReceivedMessage[slot].DataReady := FALSE;  // Invalid data
        end;

        if ReceivedMessage[slot].DataReady = TRUE then
        begin
          Inc(index);

          if ReceivedMessage[slot].Message[index] < $F8 then
            ReceivedMessage[slot].CRC := ReceivedMessage[slot].Message[index] shl 8
          else
          if ReceivedMessage[slot].Message[index] = $F8 then
          begin
            Inc(index);

            ReceivedMessage[slot].CRC := (ReceivedMessage[slot].Message[index] or $F0) shl 8;
          end;

          Inc(index);

          if ReceivedMessage[slot].Message[index] < $F8 then
            ReceivedMessage[slot].CRC := ReceivedMessage[slot].CRC or ReceivedMessage[slot].Message[index]
          else
          if ReceivedMessage[slot].Message[index] = $F8 then
          begin
            Inc(index);

            ReceivedMessage[slot].CRC := ReceivedMessage[slot].CRC or        ReceivedMessage[slot].Message[index] or $F0;
          end;

そして、ここに私のJavaコードがあります:

 if(array[1]==0xFD && (array[2]>0 && array[2]<17) && array[pos]==(byte)0xFF)
 {
index=2;
for(int counter=1;counter<splMsg.nbytes+1;counter++)
{
     index++;
 if(array[index]<0xF8)
     data[counter]=array[index];
 else
     if(array[index]==0xF8)
     {
      index++;
      data[counter]=(byte)(array[index] | 0xF0);
     }
     else
         return 0xFC; 
}


index++;
short crc=0x0000;

if(array[index]<0xF8)
     crc=(short) (array[index]<<8);
else
   if(array[index]==0xF8)
   {
    index++;
    crc=(short)((array[index] | 0xF0) << 8);
   }
index++;
if(array[index]<0xF8)
    crc=(short) (crc | array[index]);
else
   if(array[index]==0xF8)
   {
    index++;
    crc=(short)(crc | array[index] | 0xF0);
   }

msgcrc=new byte[] {(byte)(crc >> 8 & 0xff),(byte)(crc & 0xff)};

私の関数はほとんどの場合、送信された CRC コードを返しますが、時々失敗してメッセージの最後の 2 バイトを返します。メッセージの最後の 3 バイトは、CRC コード (2 バイト) とメッセージの終わりの 0xff バイトです。

何か助けはありますか？

ありがとう、ペドロ

4

1 に答える 1

7

覚えて：

Java バイトは署名されています。そう：

byte test = (byte)255;
System.out.println(test);

出力します：-1

と

    byte test = (byte)255;
    System.out.println(test == 255);

印刷されますfalseが、

    byte test = (byte)255;
    System.out.println((test & 255) == 255);

あなたが達成したいと思うことをします（この場合は印刷しますtrue）。

符号なし (バイト) 値を取得するには、 を使用します(array[index] & 255)。

どこでも 0xff でマスキングを行う必要があります。そうしないと、127 より大きいバイト値の符号拡張整数が得られます。

比較の場合は、いつでもbyte(eg if(test == (byte)255)) にキャストすることもできますが、単一の変換に固執する必要があると思うので、& 0xffマスキングを使用することをお勧めします。

于 2013-02-15T11:19:54.317 に答える