1Byte→Hex文字列変換がうまく行かない。

PICマイコン用にCを勉強中です。

1Byte→Hex文字列変換関数を作ろうとしています。
メモリを出来るだけ使いたくないのでコンパイル後のメモリを極少なくするようにしたいと思い以下のようなコードを書きましたが、うまくいきません。

void ByteToHex(unsigned short b, char *p) {
　　unsigned short t;
　　t = b >> 4
　　*p = t + (t < 10 ? 0x30 : 0x41);
　　t = b & 0x0f;
　　*(p+1) = t + (t < 10 ? 0x30 : 0x41);
　　*(p+2) = 0;
}

(p+2)：ポインタのアドレスは加減算できるんかな？？

ちなみに使っているコンパイラは"mikroC"です。

No.1 ベストアンサー 回答者： chie65536 回答日時： 2005/07/30 11:03

void ByteToHex(unsigned char b, char *p) {

　　int t;
　　t = (b >> 4) & 0x0f;
　　*p++ = t + (t < 10 ? 0x30 : 0x41);
　　t = b & 0x0f;
　　*p++ = t + (t < 10 ? 0x30 : 0x41);
　　*p = '\0';
}

No.4 回答者： noname#15691 回答日時： 2005/07/30 16:52

＃1，＃2，＃3の方の指摘を全て修正すれば動作するので、他の方法を紹介します。

void ByteToHex(unsigned short b, char *p)
{
static char *ch = "0123456789ABCDEF";
p[0] = ch[0x0f & b>>4];
p[1] = ch[0x0f & b];
p[2] = '\0';
}

メモリの消費量はコンパイルしてみないとわかりません。
なぜなら、
　見た目が短い　＝　メモリの消費量が少ない
ではないからでです。
複雑な演算式をごちゃごちゃ書いた方が案外大きくなります。

>> (p+2)：ポインタのアドレスは加減算できるんかな？？

全く問題ありません。

この回答へのお礼

ポインタを使う方法や配列を使う方法などみなさんのアドバイスとても参考になりました。
ありがとうございました。

お礼日時：2005/07/31 23:42

No.3 回答者： xcrOSgS2wY 回答日時： 2005/07/30 12:48

セミコロン抜けのほかに、

　　t < 10 ? 0x30 : 0x41

の0x41は (0x41 - 10) でないとマズくないですか。

No.2 回答者： ChateauAres 回答日時： 2005/07/30 11:07

t = b >> 4

にセミコロンがないけど？

後、どのようにうまくいかないかを書いた方がいいです。

「(p+2)：ポインタのアドレスは加減算」は可能ですが、
やらない方がいいですよ。