C言語でバイナリファイルファイルから読み込み二次元配列へ代入する方法

###前提・実現したいこと
学校の課題で簡単なパズルゲームみたいなものをC言語で作っています。
マップデータを16進数のバイナリファイルから読み込み、二次元配列へ代入したいのですが思うように動きません。
アドバイスよろしくおねがいします。

###ソースコード
自分の実行環境ではブルースクリーンになってしまいました。ご注意ください。

/*******************************************************
*		バイナリファイルから二次元配列へ
*
*///****************************************************
#include <stdio.h>

int main(void)
{
	FILE *fp ;
	unsigned char *p ;
	unsigned char dt ;
	unsigned char Qmap_001[13][13];

	p = &Qmap_001[0][0];

	fp = fopen("test.bin", "rb");

	/*---EOFにはFFが入っているためそれを表示しない対策---
	// ループの中の処理の最初に表示させて
	//最初の数字は先に読み込んでおく
	*/
	dt = fgetc(fp);

	while (feof(fp) == 0)
	{
		dt = fgetc(fp);

		*p = dt;

		(*p)++;
	}
	
	//配列に入ったかどうか確認
	int x , y ;
	for( y = 0 ; y < 13 ; y++ ) {
		for( x = 0 ; x < 13 ; x++ ) {
			printf( "0x%02x ," , Qmap_001[y][x] ) ;
		}
	}
	fclose(fp);

	return 0;
}

###補足情報(言語/FW/ツール等のバージョンなど)
gccコンパイラ

行動規範の内容に同意します

回答7件

いろいろ問題点があります。
既出の物も含めると、

１．fopenが失敗したときの対処が無い。
質問用に本筋で無いところを簡略化したと言うことであれば良いのですが。

２．13*13=169バイトのエリアに読み込むのに、読み取りサイズをチェックしていない。
ファイルが169バイトより大きいときは隣のメモリエリアを壊してしまいます。
ファイルが169バイトより小さいときは残りのメモリエリアが未代入。

３．ループの外で、1バイト読み捨てている。
dtに代入してそのままループ内で上書きしているため、そのデータは残りません。

４．ループ条件をfeofを使って判断している
feofは、最後まで読んだときに真になるのでは無く、最後の次を読もうとしたときに真になります。
従って、最後のデータをちょうど読んだ状態では、feofが偽で次のfgetcがEOFを返します。
そのデータ(0xFF)まで保存されてしまう。

５．(*p)++

改善案としては、fgetcを使うなら、

#include <stdio.h>

int main(void)
{
    FILE *fp ;
    unsigned char *p ;
    int dt ; /* int に変える必要有り */
    unsigned char Qmap_001[13][13];

    fp = fopen("test.bin", "rb");
    if(!fp){
        オープン失敗のエラー処理
    }

    p = &Qmap_001[0][0];
    while ((dt=fgetc(fp))!=EOF){
        if(p>=&Qmap_001[0][0]+sizeof(Qmap_001)){
            ファイルサイズ超過のエラー処理
        }
        *p = dt;
        p++;
    }
    if(p!=&Qmap_001[0][0]+sizeof(Qmap_001)){
        ファイルサイズが小さいエラー処理
    }
    ・・・・・・
}

forループで、169回回すのもあるかと思います。

freadを使った例は、maiko0318さんの回答へのcateyeさんのコメント参照。
ファイルが169を越えた場合の考慮が無いですが、まあ、そこまで考えなくて良いでしょう。
やっぱりサイズを決めて読むのはfreadが楽です。

サイト管理者殿：
コードブロック中で_で囲むとイタリックになってしまいますね。これはおかしい。

投稿2015/09/17 11:58

otn

総合スコア84555

"rb"でopenしてもバイナリファイルの読み込みは違います。fread()を使いましょうｄ＾＾
バイナリファイルの読み書き

蛇足ですがEOFはfgetc()呼び出し時にファイルの終端に達した場合帰ってくる値でintです。従ってcharで受けると判断できません。また、ファイルの中身は１バイトデータ(char)でしょうか？もしintでデータが格納されているとするとcharで読み込むと上下が入れ替わります。（ビックインディアン、リトルインディアンで検索してみてください）

投稿2015/09/16 13:51

cateye

総合スコア6851

_nyannyan_

2015/09/16 14:01

なぜfgetcではなくfreadを使うことを勧めるのでしょうか？ fgetcを使っているコードでも問題ないと思うので、わざわざ勧める理由もないと思うのですが。。。

naka_prg

2015/09/16 14:10

fread()試してみます！ありがとうございます！ 1バイトのデータで下位４ビットを数値、上位を判定で使っていて最小限でchar型なのですがその場合はint型で読み込んでから、char型にキャストすればいいのでしょうか？

cateye

2015/09/16 15:11

getc()は１バイトデータを読み込んでintに変換して返してきます。理由は、EOFがint型のためです。なので通常はfgetc()で読んでcharに代入してかまわないのですが・・・fgetc()がEOFを返した場合、ファイルの終端かエラーなのかが判断できないからです。

行動規範の内容に同意します

... 思うように動きません。 ...

どんなコードを書いて、どんな動作をしているかを提示していだだけると、回答がしやすくなると思います。
なにもコードの提示がないと、どんなレベルの回答をすればよいかが分かりません。

投稿2015/09/16 12:37

katoy

総合スコア22324

naka_prg

2015/09/16 13:32 編集

すみません。載せようと思ったのですがブルースクリーンになってしまったので、控えてしまいました。よろしくお願いします。

行動規範の内容に同意します

fread()する時は安全の為 dt = (char)(fread()&0xFF)をする方が良いかと思います。
ポインタを使うより p=&Qmap_001
for( i < 13 )
for( j < 13 )
Qmap_001[i][j] = dt;
の方が安全だと思います。

投稿2015/09/24 05:35

MasaakiIrie

総合スコア1021

cateye

2015/09/24 12:17 編集

Ｃでは配列はメモリ上に"連続して確保される"のですが、安全でないというにはどういった場合でしょうか？また、fread()は読み取った個数を返すのですが、それをdtに入れているのはなぜでしょう？

MasaakiIrie

2015/09/30 06:43

これは失礼しました。fread()はサイズを返すのでしたね。他の言語と勘違いしました。配列のポインタをいじるとときは、サイズ以上を参照しないようにしないと予期せぬ動作になります。範囲内であれば問題はありません。

cateye

2015/09/30 09:18

復帰値はサイズではないので、重ねて失礼しますが・・・fread()の定義は size_t fread(void *buf, size_t size, size_t n, FILE *fp); であって、パラメタはバッファポインタ、読み込むサイズ、読み込む数、ファイルポインタです。復帰値が読み込む数と同じなら正常に読めたという判断をします。（中身については、言及しません）たとえば、intを１０個読もうとしたら以下のようになります。 size_t cnt= fread(buf, sizeof int, 10, fp); この時、cnt==10であればＯＫという事です。

行動規範の内容に同意します

参考情報：