charとshortの比較

[ななし]

char[2]とshortを比較の方法を教えてください。


short s = 0x3031;
char c[2] = {0x30, 0x31};

if(memcmp(s, c, 2) == 0)
{
　printf("sとcは同じです。\n");
}

上記のようにコーディングすると、memcmpでエラーが出ます。
エラー：makes pointer from integer without a cast

[さかまき]

sじゃなくて&sにする必要があります。
私のインテルはいってるＰＣでは
short s = 0x3130;
char c[2] = {0x30, 0x31};

if(memcmp(&s, c, 2) == 0)
{
printf("sとcは同じです。\n");
}

で「同じです」になります。

[softya]

メモリの比較はエンディアンなどの問題で環境依存しますので、数値比較はmemcmpしない方が良いです。
if( c[0]==((s>>8)&0xff) && c[1]==(s&0xff) ) {
　printf("sとcは同じです。\n");
}
にすべきです。
まぁ、無理やり出来なくないですがWindowsパソコンだと確実に"sとcは同じです"にはなりません。

さかまきさんの方法は、エンディアンに完全に依存した書き方ですので、よく分からないなら使うべきではありません（確実にバグの温床になります）。

[たかぎ]

どんな目的でどんな比較をしたいのかによるでしょう。
まずはhttp://dixq.net/board/board.htmlを熟読して、必要な情報を補足してください。

[フェアレディ]

＞さかまきｓ
回答ありがとうございます。
ｓと＆ｓにすることでエラーは消えました。
ただ、char配列とshortのビットの並び順が同じで比較したいです。
言葉不足ですみません。


＞softyaｓ
1バイトずつ比較する方法で、問題を解決することが出来ました。
ありがとうございます。
また、エンディアンといった環境依存について教えていただきありがとうございます。


＞たかぎｓ
「規約と注意事項」を読まずに投稿してしまいました。
指摘していただきありがとうございます。
今後とも、よろしくお願い致します。

[さかまき]

>ただ、char配列とshortのビットの並び順が同じで比較したいです。
>言葉不足ですみません。

エンディアンの関係で、私のインテルはいってるＰＣでは
char c[2] = {0x30, 0x31};
と同じであるには
short s = 0x3031;
ではなくて
short s = 0x3130;
ですよという意味合いで書きました。
結果が「同じです」にならないだけですので、どのように比較されても結構だと思います。

[たかぎ]

> 「規約と注意事項」を読まずに投稿してしまいました。
> 指摘していただきありがとうございます。

そう思うなら、どうして是正しないのでしょうか？

[toyo]

適当な名前は規約に従って修正されたようです
環境に依存する場合なので使っているＯＳ名・コンパイラ名・ライブラリ名も明記しましょう。