C言語 dumpコマンド作成 file読み込みが上手くいかない

Question

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#pragma warning(disable: 4996)

#define H_PRT 0x02// ヘッダ印字オプション 
#define C_PRT 0x01// 文字印字オプション
#define ROW 16 //　1行に表示する文字
#define TEXTBUF 16 //テキストファイル用のバッファ
#define  FILE_ERR 10// 
#define OPT_ERR 10


//オプション
typedef  struct {

	char* infilename;
	char prt_charflag;
	char prt_headerflag;
	char prt_help;
	int offsetval;
	int filecheck;
	int optcheck;
} opts_t;

//プロトタイプ宣言
void dump(unsigned char* staddr, int dsize, opts_t* opts);

//オプションの解析
int opts_analisys(int argc, char* argv[], opts_t* opts) {

	int cnt;
	int err = 0;
	for (cnt = 1; cnt < argc; cnt++) { //引数の数ループ
		if (*argv[cnt] == '/') {   //引数の最初の文字が/なら

			switch (*(argv[cnt] + 1)) {

			case'H':
			case'h':

				opts->prt_headerflag = 1;  //16行毎にヘッダーの出力 prt_headerflag; = 1

				break;
			case 'c':
			case 'C':

				opts->prt_charflag = 1;    //ダンプのあとに文字を表示

				break;
			case 'o':
			case 'O':
				opts->offsetval = 1;

				break;

			case '?':
				opts->prt_help = 1;
				break;

			default:    //オプションが無いとき、エラー

				opts->optcheck = OPT_ERR;
				err = OPT_ERR;   //opt err

				break;
			}
			continue;
		}
		if (opts->infilename == NULL) {
			opts->infilename = argv[cnt];  //ファイル名をopts.infilename へ
		}
		else {

			opts->filecheck = FILE_ERR;
			err = FILE_ERR;

			return err;
		}
	}
	return err;
}

int main(int argc, char* argv[]) {


	char bin_data[TEXTBUF]; //　テキストファイル読み込み用
	int readnum;
	opts_t opts;
	int result = 0;

	//構造体の初期化
	opts.infilename = NULL;
	opts.prt_charflag = 0;
	opts.prt_headerflag = 0;
	opts.prt_help = 0;
	opts.offsetval = 0;
	opts.filecheck = 0;
	opts.optcheck = 0;

	FILE* file;

	result = opts_analisys(argc, argv, &opts);  //オプションの解析

	//if(ファイルエラー　　　または　 オプションで？)
	if (((result & opts.filecheck) != 0) || ((result & opts.optcheck) != 0) || (opts.prt_help == 1)) {

		if ((result & opts.filecheck) != 0) {
			fprintf(stderr, "ファイル指定が２つあります");
		}
		if ((result & opts.optcheck) != 0) {

			fprintf(stderr, "オプション指定が間違っています");
		}
		if (opts.prt_help == 1) {
			fprintf(stderr, "構文:  mydump[<opts>] [<inpath>] [<opts>]\n機能:  ファイルやデバイスの内容を16進で表示する\nオプション:\n/ o[=]<hex> オフセットアドレスの指定(省略の場合は0000)\n/ h       16行毎にヘッダーの出力(デフォルトは = 0)\n	/ c	　　ダンプのあとに文字を表示(デフォルトは = 1)\n / ? 　　使用方法の表示(このメッセージを表示)");
		}
		return 0;
	}

	if (opts.infilename != NULL) {      //コマンドラインからファイルの入力があれば


		if ((file = fopen(opts.infilename, "rb")) != NULL) {    //ファイルがあれば

			fseek(file, opts.offsetval, SEEK_SET);
		}
		else {                                                                  //ファイルがなければ
			fprintf(stderr, "ファイルがありません");
			return 0;
		}
	}
	else {
		file = stdin;	
	}

	while ((readnum = fread(bin_data, sizeof(unsigned char), TEXTBUF, file)) > 0) {

		dump(bin_data, readnum, &opts);
	}

	if (opts.infilename != NULL) {
		fclose(file);    //ファイルが開かれたらクローズ
	}
}
//印刷
void  dump(unsigned char* staddr, int dsize, opts_t* opts) {

	int startcnt = 0;// ダンプするバイト数のカウント
	int bytecnt; // 16回ループさせる
	int savecnt;
	static int address = 0; //アドレスの表示
	static int onhold = 0; //文字出力が最後のバイトだった時の一時保留

	static  int headercnt = 0;

	char addrhead[] = "Addr";
	char hexa[] = "0 1  2 3  4 5  6 7  8 9  A B  C D  E F";
	char charprint[] = "0 2 4 6 8 A C E";

	while (startcnt < dsize) {

		if (onhold != 0) {  //前回最後が漢字だった場合、漢字を出力して改行

			printf("%c%c\n", onhold, staddr[startcnt]);
		}

		//初回ヘッダー表示　opt & H_PRT == 0だった時は16行毎にヘッダー表示
		if ((headercnt == 0) || ((opts->prt_headerflag & H_PRT) == 0) && (headercnt % ROW == 0)) {
			//16行毎にヘッダー

			printf("  %s     %s  ", addrhead, hexa);


			if ((opts->prt_charflag & C_PRT) != 0) {  //opt が文字表示の場合　文字ヘッダー表示

				printf("%s", charprint);
			}

			printf("\n");
			printf("--------  ---- ---- ---- ---- ---- ---- ---- ----");

			if ((opts->prt_charflag & C_PRT) != 0) { ////opt が文字表示の場合　文字罫線表示
				printf("  ----------------");

			}
			printf("\n");
		}

		headercnt++;

		printf("%08x  ", address);

		savecnt = startcnt;
		for (bytecnt = 0; bytecnt < ROW; bytecnt++) {

			if (startcnt < dsize) {
				printf("%02x", staddr[startcnt]); //16進数で出力
			}
			else {
				printf("  ");
			}
			if (startcnt % 2 != 0) {
				printf(" ");
			}
			startcnt++;
		}

		//文字の出力

		if ((opts->prt_charflag & C_PRT) != 0) {   //optで文字表示の場合　表示

			startcnt = savecnt;
			printf(" ");

			for (bytecnt = 0; bytecnt < ROW; bytecnt++) {

				if (onhold != 0) {  //前回最後が漢字だった場合 次の行の最初にスペースと初期化

					printf(" ");
					onhold = 0;
					bytecnt++;
					startcnt++;
				}

				if (startcnt < dsize) {

					//漢字だった場合
					if ((staddr[startcnt] >= 0x81 && staddr[startcnt] < 0xa0) || (staddr[startcnt] >= 0xe0 && staddr[startcnt] < 0xfd)) {

						if (bytecnt < ROW - 1) {  //最後のバイトでなければ

							printf("%c%c", staddr[startcnt], staddr[startcnt + 1]);
							startcnt++;
							bytecnt++;
						}
						else {//漢字で最後のバイトの場合、保留

							onhold = staddr[startcnt];
						}
					}
					else {

						if ((staddr[startcnt] >= 0x20 && staddr[startcnt] < 0x7f) || (staddr[startcnt] >= 0xa0 && staddr[startcnt] < 0xe0)) {

							printf("%c", staddr[startcnt]);
						}
						else {
							printf("");
						}
					}
				}
				else {
					printf(" ");
				}

				startcnt++;
			}
			if (onhold == 0) {  //最後が漢字でなければ改行
				printf("\n");
			}
		}
		else {  //optが文字表示ではない場合　改行

			printf("\n");
		}
		address += ROW; //addressを出力しただけ足す
	}
}

上記のコードをコマンドラインから実行しても、エラーも出力も何もでません。

/?　　や　ファイル名を複数入力した場合、オプションの解析でエラーがでるので

	if (opts.infilename != NULL) {      //コマンドラインからファイルの入力があれば


		if ((file = fopen(opts.infilename, "rb")) != NULL) {    //ファイルがあれば

			fseek(file, opts.offsetval, SEEK_SET);
		}
		else {                                                                  //ファイルがなければ
			fprintf(stderr, "ファイルがありません");
			return 0;
		}
	}
	else {
		file = stdin;	
	}

	while ((readnum = fread(bin_data, sizeof(unsigned char), TEXTBUF, file)) > 0) {

		dump(bin_data, readnum, &opts);
	}

	if (opts.infilename != NULL) {
		fclose(file);    //ファイルが開かれたらクローズ
	}
}

ここの部分に問題がある気がするのですが、なかなか解決できません。　　
ご指摘いただけると幸いです。

Comments

[KoichiSugiyama]

「上記のコードをコマンドラインから実行しても、エラーも出力も何もでません。」この時どう実行したのかコマンドラインに打ち込んだそのままを書いてもらわないと、なんとも判断できません。

[kokok]

mydump ファイル名 です。

[mit0223]

ちょっとやってみましたが、mac でも linux でも、それらしきものが出力され、動作しました。環境の問題かもしれません。OS, コンパイラのバージョン、コンパイル時のコマンド、テスト実行時のコマンドと結果を貼ってください。

[kokok]

os : windows10

コンパイラのバージョンとコンパイル時のコマンドとテスト実行時のコマンドとはなんでしょうか？

Visual Studioの標準のコンパイラを使っていると思います。

[KoichiSugiyama]

指定したファイルが0バイトということはないですか？私の環境（mac)で
$ touch test.bin
で作ったファイルを使って実行してみると何も出ませんが、
$ dd if=/dev/zero of=test.bin bs=1024 count=1
で1024バイトのファイルを作り直して実行するとdump結果が表示されます。

[KoichiSugiyama]

Windows10 Visual Studio 2017でビルドしたものでも指定したファイルのサイズが1024バイトだとちゃんとdump結果が出ます。0バイトだと出ません。
( > fsutil file createnew test.bin 1024 で作成)

[kokok]

1024バイトで作成してみましたが、ファイルがありません。となってしまいます。

[tatsu99]

実行ファイルとダンプ対象のファイルを同じフォルダ内に格納して、そのフォルダ内でコマンドを実行してください。

[kokok]

ありがとうございます。
pathを通してもフォルダ内まで移動して実行しないとダメなのでしょうか？

[tatsu99]

pathを通してOKとなるのは、実行ファイルに対してです。
ダンプ対象のファイルは、フルパスで指定するか、そのファイルがあるフォルダまで移動して、コマンドを打たないとだめです。そもそも、そのあたりの事情が不明なので、まずは、確実に成功する方法として、
実行ファイルとダンプ対象のファイルを同じフォルダに格納し、そのフォルダでコマンドを実行してくださいと書きました。この方法で、「ファイルがありません」という件をまず解決するのが先決かと。

[kokok]

ありがとうございます。

「ファイルがありません」という件は、そのファイルがあるフォルダまで移動して実行したところ解決できました。

[tatsu99]

それでは、本件は解決でしょうか。それとも、何の出力結果もなく終了してしまうということでしょうか。

Answer 1

VisualStudioを使ってるならデバッグしましょう。
ファイルのオープンが問題だと思うなら、そこんところにブレークポイントを設置して実行を止め、各変数の値を確認してください。また、その実行結果が何を返しているか確認しましょう。
そして１行づつ実行させ、あなたの想定から外れる動作のところを確認します

Comments

[kokok]

コマンドラインから引数など渡す場合は、どのようにデバッグすれば良いのでしょうか？

[y_waiwai]

プロジェクトのプロパティでコマンド引数の設定があります

このコードではファイル名の設定がうまくいかないですね

[kokok]

プロジェクトのプロパティでコマンド引数の設定は一応しております。

[y_waiwai]

ならそれでデバッグしましょうよ。
１行づつ実行させてみたらどうですか