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

下記のように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 回答日時： 2003/10/14 19:07

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

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

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

この回答へのお礼

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

お礼日時：2003/10/14 19:13

No.1 回答者： bikkuri 回答日時： 2003/10/14 18:45

なにが上手くいかないのか不明ですが、

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

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

この回答へのお礼

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

お礼日時：2003/10/14 19:11