４バイトでのサムチェックコード作成プログラミング

質問

下記のようにTx[1]からTx[19]までのサムチェックコードを
Tx[21](上位）、Tx[22](下位）に格納したいのですが上手くいきません。記述の訂正点やもっと良い方法があったら教えて下さい。

unsigned char Tx[30];
unsigned int sum;

for( i = 1; i < 20; i++ ) // サムチェックコード作成
sum ^= Tx[ i ];

Tx[ 22 ] = sum; //23 サムチェックコード２
sum = sum >> 8;
Tx[ 21 ] = sum; //22 サムチェックコード１

回答No.2

kabasan · ベストアンサー

^= はサムですから +=の間違いなんでしょうね。

変数のキャストが抜けています。
sum += (int) Tx[ i ];
Tx[ 22 ] = (char) sum;

あと、SUM = 0 と明示的に初期化しましょう。

お礼

そのとうりです。回答有難う御座いました。

回答No.1

bikkuri

なにが上手くいかないのか不明ですが、
sum ^= Tx[i];
とあるので、
char(8bit)のxorをいくらやっても、答えは8bitにしかなりません。
よって、Tx[21]は常に０が入っていると思います。

もしかして、sum += Tx[i]の間違い？

お礼

そのとうりです。回答有難う御座いました。