C言語 dumpコマンド作成 渡された引数の判定が上手くいかない

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 = 0;  //16行毎にヘッダーの出力 prt_headerflag; = 0

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

				opts->prt_charflag = 1;    //ダンプのあとに文字を表示(デフォルトは= 1     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;
			}
		}

		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 ;
	 opts.prt_charflag ;
	opts.prt_headerflag;
	opts.prt_help ;
	opts.offsetval ;
	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;
	}

	if (opts.infilename != NULL) {
		
		//fseek(*file, opts.offsetval, SEEK_CUR);
		
		file = fopen(opts.infilename, "rb");
		fseek(file, opts.offsetval, SEEK_END);
	}
	else {
		file = stdin;
		return;
	}

	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を出力しただけ足す
	}
}

引数でファイル名や　オプションで /h /c などを渡しても

ファイル指定が２つあります"　となってしまいます。

/? を渡してあげると /?の内容が表示されるので

下記のコードに問題がありそうなのですが、分からなかったのでアドバイス頂けると助かります。

 //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;
    }

----追記----

コマンドラインで実行すると
Debug Assertion Failed

Expression: stram != nullptr

とエラーが出てしまいます。

if (opts.infilename != NULL) {

		file = fopen(opts.infilename, "rb");
		//	fseek(file, opts.offsetval, SEEK_END);

	
	}
	else {
		file = stdin;
		return 0;
	}

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

		dump(bin_data, readnum, &opts);
	}

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

ここに問題があるとある気がするのですが
エラー内容があまり理解できなかったのでアドバイス頂けると助かります。

----追記２----

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


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

			fseek(file, opts.offsetval, SEEK_END);
		}
		else {                                                                  //ファイルがなければ
			//printf( "ファイルがありません");
			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);    //ファイルが開かれたらクローズ
	}
}

このように変更したところエラーメッセージは出なくなりましたが、ファイルの中身が表示されません。

Answer 1

下記のコードに問題がありそうなのですが

その部分も十分怪しい臭いがしますが、もうひとつ怪しいのは opts_analisys() 関数です。オプションを正しく解析できていないのではありませんか。

opts_analisys() 関数が返す情報は２種類あります。

• 関数の戻り値。即ち result 変数に返る値
• opts 構造体変数にセットする値。即ち infilename 〜 optcheck メンバの値。

これらの値を質問者自身で確認することをお勧めします。プログラムを思い通りに動かしたかったらデバッグ作業は不可欠です。他人に頼っていたら思い通りになりません。デバッガが使えるに越したことはありませんが、このケースは所謂printfデバッグが十分役に立ちそうです。
例えば opts 構造体の内容を表示する関数として

void printOpts(opts_t *opts)
{
    printf("infilename     = %p\n", opts->infilename);
    printf("prt_charflag   = %d\n", opts->prt_charflag);
    printf("prt_headerflag = %d\n", opts->prt_headerflag);
    printf("prt_help       = %d\n", opts->prt_help);
    printf("offsetval      = %d\n", opts->offsetval);
    printf("filecheck      = %d\n", opts->filecheck);
    printf("optcheck       = %d\n", opts->optcheck);
    printf("\n");
}

を追加して、opts_analisys() 関数呼出しの前後を

    printOpts(&opts);                // デバッグ表示
    result = opts_analisys(argc, argv, &opts); // オプション解析
    printf("result = %d\n", result); // デバッグ表示
    printOpts(&opts);                // デバッグ表示

とすれば、result と opts 変数の値を確認できます。printOpts() は opts_analisys() の中からも呼び出せます。
Enjoy !

Comments

[kokok]

ありがとうございます。
変数の値を一つずつ確認してみます。

Answer 2

とりあえず、気が付いた箇所のみコメントしておきます。
//追加
//変更
のコメントをいれておきます。

//オプションの解析
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; = 0

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

                opts->prt_charflag = 1;    //ダンプのあとに文字を表示(デフォルトは= 1     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) {

        //fseek(*file, opts.offsetval, SEEK_CUR);

        file = fopen(opts.infilename, "rb");
        fseek(file, opts.offsetval, SEEK_END);
    }
    else {
        file = stdin;
        return 0;
    }

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

        dump(bin_data,  readnum, &opts);
    }

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

mainの最初の部分

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;

Answer 3

if (*argv[cnt] == '/') {

この if ()を抜けた後、どうなっているでしょうか?

ファイル指定が２つあります

との事なので、エラーフラグ立てる時に、そのファイル名を出力してみましょう。

なお、"&" "&&" が混在してますが、使い分けましょう。(論理積はむ、&&)
opts.offsetval 初期化されてないみたいですが、、他は?

細かく、質問立ててますが、デバッガとは言わなくても、途中経過を出力するとかで、ある程度分かりませんか?