コマンドライン引数の取り扱いについて

Question

表題の件、次のことを実現したいのでご協力ください。

最終的に実現したいこと。
コマンドライン引数より ファイルオープン分類(数字) ファイル形式 ファイル名を入力しfopen関数の引数として
利用する。

今回実現したいこと
fopen関数の第2引数である、オープンモードの文字列をコマンドライン引数の「ファイルオープン分類」、「ファイル形式」から作成する関数を作りたいと思っています。作成するにあたって以下が諸条件です。

条件
・main関数とは分離する
・ファイルオープン分類として入力する数値とオープンモードの対応は以下の通り
1:r, 2:w, 3:a, 4:r+, 5:w+

・ファイル形式は以下の通り
c:テキスト,b:バイナリ
→ファイル形式を指定しているのはbが選択された場合に対応するためです。

困っている個所
以下のようにコードを書いていますが、実行するとファイルダンプが発生します。
どのように変更すればopenModeStrにオープンモードの文字列を格納し、表示させることが出来るでしょうか。

#include<stdio.h>
#include<string.h>


int makeModeStr(char openMode, char fileType, char *openModeStr);

int main(int argc, char *argv[])
{
    char openModeStr[4];
    int check = makeModeStr(argv[1][0], argv[2][0], openModeStr);

    printf("openModeStr = %s\n", openModeStr);

    return 0;
}

int makeModeStr(char openMode, char fileType, char *openModeStr){


    if (openMode == '1') {
        strcpy(openModeStr, "r");
    }

    else if (openMode == '2') {
        strcpy(openModeStr, "w");
    }
    else if (openMode == '3') {
        strcpy(openModeStr, "a");
    }
    else if (openMode == '4') {
        strcpy(openModeStr, "r+");
    }
    else {
        strcpy(openModeStr, "w+");
    }
	
	if (fileType == 'b') {
		strcat(openModeStr, "b");
	}

    return 0;
}

Answer 1

openModeはchar型なので、char型の値と比較して下さい。

// if (openMode == "1") {
if (openMode == '1') {

Comments

[ken21]

回答ありがとうございます。
比較の個所はそれで問題なくなりました。
※コードは修正しました。

しかし、実行後にファイルダンプが発生します。
何か致命的な個所がありますでしょうか？
何か

[LouiS0616]

コマンドライン引数はどのように渡していますか？

[ken21]

すいません、コマンドライン引数を exe 数字　でしか渡していませんでした。
exe 数字　ファイルタイプで入力したら問題なく実行できました。

ありがとうございました。
最後に2点だけ確認させてください。
今回のような関数呼び出しをすると変数openModeとfileTypeにはargvが持っているアドレスが格納されていると思います。(printfを使って確認したらopenとfileの変数にはアドレスが入っていました)
makeModeStr(argv[1][0], argv[2][0], openModeStr);

int makeModeStr(char openMode, char fileType, char *openModeStr)

その場合だとしての疑問ですが、、

①なぜif文で文字と比較ができたのでしょうか。
(下の書き方だとアドレスと文字の比較になってエラーが起きる気がします)
②仮引数側にアドレスが入っていたのは呼び出した際に仮引数がポインタ変数にキャストされている？

if (openMode == '1')

[LouiS0616]

> 変数openModeとfileTypeにはargvが持っているアドレスが格納されていると思います。
いいえ。
argv[i] は i番目のコマンドライン引数（もっと正確に言えば、i番目のコマンドライン引数の先頭アドレス）であり、argv[i][0] はその0文字目です。

> なぜif文で文字と比較ができたのでしょうか。
openModeが文字だからです。アドレスでは無いです。

[ken21]

なるほど文字を渡しているでよかったのですね。
ただ、下記のように記入した場合※にargv[1][0]とopenModeの出力が同じなのが気になりました。
この結果をもってopenModeへアドレスが渡ったと判断したのですが何か根本的に考え方が間違ってますでしょうか？

int main(int argc, char *argv[])
{
printf("argv[1][0] = %p\n", argv[1][0]);
printf("argv[2][0] = %p\n", argv[2][0]);

char openModeStr[4];
int check = makeModeStr(argv[1][0], argv[2][0], openModeStr);

printf("openModeStr = %s\n", openModeStr);

return 0;
}

int makeModeStr(char openMode, char fileType, char *openModeStr){

printf("openMode = %p", openMode);
printf("fileType = %p", fileType);

[LouiS0616]

%pで出力できることは、アドレスが渡っていることの証左にはなりません。
%dでも%cでも出力できる筈です。

[LouiS0616]

ところで、コンパイルの際に警告が出るように設定しているでしょうか。
かなりの潜在バグを検知できます。

[ken21]

cygwinを使っていますが警告などの設定をどのようにするかはわかっておりません。
コンパイルする際はそのような設定をしたほうが良いのでしょうか。

[LouiS0616]

cygwinは実行環境ですよね？
コンパイラは何をお使いですか。gccとかclangとかclとかです。

[ken21]

gccです。
cygwinがコンパイラだと思っておりました、、、

[LouiS0616]

gcc -Wall -Wextra プログラム.c でコンパイルしてみて下さい。

[ken21]

コンパイルしてみました。
利用してないcheckの変数やprintf(%p= ....,openMode)の個所でエラー表示が出ました。

%pの個所は「format argument is not a pointer (arg 2)」と出たのでその変数がポインター、つまりアドレスを持っていないということですね？

[LouiS0616]

%pで出力されるべきではない、って意味ですね。

> その変数がポインター、つまりアドレスを持っていないということですね？

アドレスを持っていないというとちょっと語弊がありますが、
当該変数が直接保持している値がアドレスではない（=変数がポインタではない）のはそうです。

[ken21]

なるほど。このような調べ方もあるんですね。
大変勉強になりました。

Answer 2

コマンドラインで入力した数字、ファイル形式の文字をchar型の仮引数へ渡したいのですが

やりたいことが間違っています。charには1文字しか入れられませんので、複数文字が来うる文字列の受け渡しには使えません。

Comments

[LouiS0616]

複数文字は引き渡さない前提で良いように見えます。

> ファイルオープン分類として入力する数値とオープンモードの対応は以下の通り
1:r, 2:w, 3:a, 4:r+, 5:w+
> ファイル形式は以下の通り
c:テキスト,b:バイナリ

[maisumakun]

あっ、ほんとだ。

[ken21]

はい、argv[1][0]のように指定して一文字で渡そうと考えております。
※この記述で文字が渡せるかはわからないですが、、、