バイナリ４バイトをint型に変換する式はこれなのかどうか知りたい。

Question

以下の提示コードですがバイナリを4バイトのint型整数に変換しているビットシフトという式で正しいのでしょうか？確認しようもバイナリデータのため目視する方法がないため単純な質問をしました。出てきたものを信じるしかないため

int convInt(unsigned char* header, int start)
{
	int ret = (header[start + 3] << 24) | (header[start + 2] << 16) | (header[start + 1] << 8) | (header[start + 0]);
	printf("あああ %d\n",ret);
	return ret;
}

Comments

[dodox86]

> 確認しようもバイナリデータのため目視する方法がない
目視する方法が無いわけではありません。unsigned char* headerを頭から1バイトずつprintfなりsprintfで16進数ダンプすれば良いだけです。

[退会済みユーザー]

ダンプとはコピーするということでしょうか？

[dodox86]

> ダンプとはコピーするということでしょうか？

ちがいます。16進数ダンプとは例えば、printf("%02X %02X", *header, *(header + 1))　のように、内容を晒すことをいいます。ダンプ（dump）とはプログラミングでは頻出の言葉で、元は「ぶちまける」の意があります。詳しくはググってみてください。質問の本筋ではないのでコメントは以上です。

[dodox86]

そういったことを簡単にするのがy_waiwaiさんの回答でいう「デバッグ環境を整える」です。Visual Studio を含め、大抵のデバッガーには変数のみならず、メモリの直接のダンプ機能があります。本回答を含め、幾多のコメント、アドバイスをいただいているのですから、素直に従ってみませんか。（今度こそこれでコメント終了です）

Answer 1

ビッグリトルエンディアンの４バイト整数であればそれでいいです、が、出てくる数値は確認しときましょう

こういうコードを書くなら、デバッグできる環境を整えておきましょうよ。

---

C言語のコードを書くなら、デバッグできる環境を整えましょう。
Eclipseや、WindowsならVisualStudioなど。
コードの任意の場所で実行を止め、変数のナカミを見ることができます。そこから１行づつ実行して、コードの流れを見れるようになります
そうすれば、アテズッポでコードを書かなくて済むようになります。

＃BAになってしまったんで修正追記

Comments

[dodox86]

メモリ上の低位アドレスの1バイトを下位8ビットにしているので、リトルエンディアンではないでしょうか。
... (header[start + 1] << 8) | (header[start + 0]);

[y_waiwai]

あああ、そうですね。勘違いでした。
リトルエンディアンですね
ご指摘ありがとうございます

[Zuishin]

確かめたわけではないので疑問に思っただけですが、オーバーフローしませんか？

> header[start + 3] << 24

[退会済みユーザー]

間違えて居るのでしょうか？

[Zuishin]

確かめていません。自分で確かめてください。演算によって暗黙に int になったかもしれません。