C言語、ファイル読み書き、コマンドライン引数、argc、argvについて

Question

input.txtを読み込み、計算結果をoutput.txtに書き出すプログラムを書いています

やりたいこと
・読み込み/書き出しのファイルを実行する時にargc,argvを用いて
コマンド行の引数で指定できるようにしたい
・引数の数のチェックをして足りないときはusageを表示して終了、
引数で受け取ったファイル名は別変数にコピーしてargvのまま用いないようにしたいです

補足
・読み込むファイル名は"input.txt"

//input.txt
1
2
3
4
5
6
7
8
9
10

・書き出すファイル名は"output.txt"

//空

現在自分で書いたコードは以下です(8/31追記を元に修正しました)

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#define N 10

int main(int argc, char *argv[])
{
  int i,data[N];
  int sum = 0;
  float ave = 0;
  float sqave = 0;
  float sd =0;
  const char* file1 = "input.txt";
  const char* file2 = "output.txt";
  FILE *fp1,*fp2;
  
//出力時の文字化けを直したい
  
//引数で受け取ったファイル名は別変数にコピーしてargvのまま用いないようにしたい
  file1 = argv[1];
  file2 = argv[2];

//引数の数のチェックをして足りないときはusageを表示して終了したい
  if (argc < 3){
      printf("usage: %s in_file out_fileを用いてください\n", argv[0]);
      return 1;
  }

//ファイルを開いて読み込み
  if( (fp1=fopen(file1,"r")) == NULL){
      printf("%sファイルを開くことが出来ませんでした\n",file1);
      return 1;
  }else{
      printf("%sファイルをオープンしました\n",file1);
      printf("%d 個の整数の合計を出力します\n", N);
  }
  
  for(i=0; i < N ; i++){
      fscanf(fp1,"%d\n", &data[i]);
      printf("読み込みデータ = %d\n", data[i]);
      sum += data[i];
      sqave += data[i]*data[i];
  }
  fclose(fp1);
  printf("%sファイルをクローズしました\n", file1);

//計算部分
  printf("%d 個の整数の平均と標準偏差を出力します\n", N);
  ave = (float)sum/N;
  sqave /= N;
  sd = (sqave - ave*ave)*N/(N-1);
  if(sd >= 0){
      printf("平均 = %.1f\n標準偏差 = %.1f\n", ave, sqrt(sd));
  }else{
      printf("平均 = %.1f\n標準偏差 = 存在しない値です\n", ave);
  }

//新規ファイルに書き出し
  if( (fp2=fopen(file2,"w")) == NULL){
      printf("%sファイルを開くことが出来ませんでした\n",file2);
      return 1;
  }else{
      printf("%sファイルをオープンしました\n", file2);
      fprintf(fp2,"平均 = %.1f\n標準偏差 = %.1f\n", ave, sqrt(sd));
      printf("平均と標準偏差を%sファイルに書き込みました\n", file2);
  }
  fclose(fp2);
  printf("%sファイルをクローズしました\n", file2);
  
  return 0;
}

文字化けを直したいです。
よろしくお願いします。

Comments

[hidezzz]

「読み込み/書き出しのファイルを実行する時にargc,argvを用いてコマンド行の引数で指定できるようにしたい」の部分についてどのようにコマンド行に指定しようとしているのかがよくわかりませんでした。
具体的にどのようなコマンドラインを指定したのかを質問を修正して追記してみてください。

「コンパイル出来たものの、プログラムが動きませんでした。」の部分は、具体的には実行後にどのようになりますか？(実行しても何も表示されない、何らかのエラーメッセージが表示される、自分が意図していない計算結果が表示される、…など) こちらも質問に追記してみてください。

また正しい出力結果について想定されているものがあると思います。これについてもどういう出力結果になって欲しいのかを追記してみてください。

[kazuma-s]

> ・変数の数のチェックをして足りないときはusageを表示して終了、
> 引数で受け取ったファイル名は別関数にコピーしてargvのまま用いないようにしたいです

変数というよりも、引数の数のチェックでは？
別関数ではなく、別変数にコピーでは？

[yatai]

@hidezzzさん
ありがとうございます。
>具体的にどのようなコマンドラインを指定したのかを質問を修正して

自分もよく分かっておらず、input.txtを扱いやすくするため？等、疑問です。

実行後の結果、想定される出力結果については後ほど追加します。

[yatai]

@kazama-sさん
ありがとうございます。
>変数というよりも、引数の数のチェックでは？
>別関数ではなく、別変数にコピーでは？

おっしゃる通りです。修正致します。
上記の解決策も教えていただけると幸いです。

Answer 1

-  int i,data[i];
+  int i, data[10];
 
-  for(i=0; i < argc ; i++)
+  for (i = 0; i < 10 ; i++)

-      data[i] = (int)argv[i];

この修正でとりあえず動くはずですが、
sd = (sqave - ave*ave)*10/(10-1); の意味が分かりません。
なぜ、10倍して 9で割っているのですか？

また、ファイル名が、input.txt と output.txt に固定で、
コマンドラインの引数で指定することができていません。

追記

・読み込み/書き出しのファイルを実行する時にargc,argvを用いて

#include <stdio.h>

int main(int argc, char *argv[])
{
	//・引数の数のチェックをして足りないときはusageを表示して終了、
	if (argc < 3) {
		printf("usage: %s in_file out_file\n", argv[0]); return 3;
	}

	// 引数で受け取ったファイル名は別変数にコピーしてargvのまま用いない
	char *ifile = argv[1], *ofile = argv[2];

	FILE *fp1 = fopen(ifile, "r");
	if (fp1 == NULL) { printf("can't open %s\n", ifile); return 1; }
	// データの読み込み
	fclose(fp1);

    // 計算

	FILE *fp2 = fopen(ofile, "w");
	if (fp2 == NULL) { printf("can't create %s\n", ofile); return 2; }
	// 計算結果の書き出し
	fclose(fp2);

	return 0;
}

Comments

[yatai]

ありがとうございます。
実行することが出来ましたが、文字化けしてしまいました。
対処方法はあるでしょうか？

[yatai]

sd = (sqave - ave*ave)*10/(10-1);
は10個の整数の不偏分散を表しています。

>また、ファイル名が、input.txt と output.txt に固定で、
コマンドラインの引数で指定することができていません。

学習途中ですみませんが、固定の際はコマンドラインの引数で指定が出来ないのでしょうか？

[hidezzz]

> 固定の際はコマンドラインの引数で指定が出来ないのでしょうか？

ソースコード中に固定で指定しているのなら、コマンドライン引数で重複して指定する必要は無いです。
とりあえずは、ソースコードに埋め込まれたファイル名固定にするか、コマンドライン引数で指定するかのどちらかに仕様を決めたほうが良いと思います。

ちなみに、出来るか出来ないかという質問の直接の回答としては、やりようによっては出来る、となります。
(仕様の一例としては、コマンドライン引数で指定した場合はそちらを使用するが、指定しない場合はソースコード中の固定のファイル名を使うみたいな感じになるしょうか？)
もしそのような仕様にするにしても、まずはプログラムとして正常な動作が出来るように仕上げるのが先かと思います。

[yatai]

コマンドライン引数で指定する仕様にしたいです。

>(仕様の一例としては、コマンドライン引数で指定した場合はそちらを使用するが、指定しない場合はソースコード中の固定のファイル名を使うみたいな感じになるしょうか？)

そのようなイメージにしたいと考えています。

まずは文字化けの原因を調べていますが、何が理由なのかいまいち分かりません、、。

[hidezzz]

少なくとも、私は質問へのコメントに書いた質問の修正がされない限りこれ以上回答するつもりはありません。
他の方の回答を待つか、自力で頑張ってください。

[yatai]

@kazuma-sさん
追記ありがとうございます。
自分で書いたコードのfile1,file2と追記のifile,ofileが共存できません
といったエラーが出たので統一したのですが、上手くコンパイル出来ませんでした、、

また、argv[0] = 実行ファイル.exe となったのですが、この場合argv[1],argv[2]は
どのように表記されるのでしょうか？

[kazuma-s]

「自分で書いたコードのfile1,file2」とは何ですか？
どうやって実行を開始していますか？
コマンドライン引数に input.txt と output.txt を指定しましたか？
コマンドラインが何か分かっていないのですか？

[yatai]

すみません。追記していただいた物を元に質問を修正しました。

今の疑問点
・最初に定義したfile1,file2と追記のifile,ofileをどのように統一すれば良いか
・コンパイルした際に文字化けするのはどうすれば直るか
・コマンドライン引数が何を指しているのか分からなくなってしまった

という感じです。お答えいただけると幸いです。

[kazuma-s]

文字化けもコマンドラインも環境に依存します。
どんなOS(Windows, Linux, MacOS など) で、どんなコンパイラ(VC++, gcc, ...)、
どんな IDE(Visual Studio, Eclipse, ...) で、文字コードの設定は何か、
どうやって実行を開始しているかなどを具体的に示してもらわないと回答できません。

[yatai]

そうなのですね。無知ですみません。
Windows,gcc,mingw-w64,UTF-8で行っています。

[kazuma-s]

minggw の gcc をつかっているということは端末はコマンドプロンプト(cmd.exe)で、
コマンドラインから
> gcc -o hoge hoge.c
> hoge input.txt output.txt
のようにしてプログラムを起動しているのでしょうか？
それとも端末は PowerShell ですか？
Cygwin だったら端末は mintty かもしれませんが。
それで、コード中の、argc は 3、argv[0] = "hoge"、
argv[1] = "input.txt"、argv[2] = "output.txt" となりませんか？

UTF-8 を使っているのなら、端末で chcp コマンドで確かめてください。
65001 なら UTF-8 ですが、932 なら Shift-JIS です。
ソースファイルは本当に UTF-8 ですか？　どうやって確かめましたか？
エディタは何ですか？
メモ帳(notepad.exe)なら、名前を付けて保存を選ぶと文字コードが分かります。
ANSI なら Shift-JIS、Utf-8 ならそう表示されるはずです。

[kazuma-s]

if (argc < 3) を if (argc != N) に変えたのはなぜですか？
argc はコマンドライン引数の個数。N はファイル内のデータの個数
ということが分かっていないのですか？

[yatai]

そこの理解も曖昧で、色々試してしまいました。
今回は｢実行ファイル　in_file out_file｣の3つが引数と言うことでしょうか。

コマンドプロンプトで｢実行ファイル　in_file out_file｣と入力することが分かったので
printf("usage: %s in_file out_file\n", argv[0]);の意味が理解できたと思います。

上記を修正したところ、後は文字化けを直すだけになりました。
引き続き直したいと思います。

[kazuma-s]

> 上記を修正したところ、後は文字化けを直すだけになりました。

本当ですか？
引数の argv[1]、argv[2] を参照した後、引数の個数 argc をチェックしても平気なのですか？

文字化けの問題は、最小限のプログラム
printf("abcあいう123\n");
などを使用して、別の質問にしたほうが良いと思います。
文字化けは、様々な要素が絡み合って生じるので、どういう環境で、
どういう設定で実行しているのかを詳しく書かないと解決しないでしょう。

[yatai]

>引数の argv[1]、argv[2] を参照した後、引数の個数 argc をチェックしても平気なのですか？

そこの仕組みについては勉強途中なのですが、プログラムとしては動くようになりました。

また、文字化けに関して、
｢gccによるコンパイルは、-fexec-charset=CP932 スイッチをつけてコンパイルする｣
という操作で直すことが出来ました。
仕組みはよく分かっていないのですが、、

Answer 2

Cのコードを組むなら、デバッグできる環境を整えましょう。
Windowsなら、VisualStudioなど。
コードの任意の場所で実行を止めることができ、変数の中身を見ることができます。
また、１行づつ実行して、変数の変化やコードの流れを見ることができます。

そうすれば自分の組んだコードがどう動くのかわからない、ってことはなくなるかと思います

Comments

[yatai]

ありがとうございます。
VisualStudioについて調べてみます。

Answer 3

うろ覚えで申し訳ないですが、ファイル関連だけ。。。

//新規ファイルに書き出し
  //if( (fp2=fopen(file2,"w")) == NULL){
  if( (fp2=fopen(file2,"wt")) == NULL){ //テキストモードでファイルを開く

もしくは

//新規ファイルに書き出し
  if( (fp2=fopen(file2,"w")) == NULL){
　if( (fp2=fopen(file2,"wb")) == NULL){//バイナリモードでファイルを開くことを明記
      printf("%sファイルを開くことが出来ませんでした\n",file2);
      return 1;
  }else{
      printf("%sファイルをオープンしました\n", file2);
      //fprintf(fp2,"平均 = %.1f\n標準偏差 = %.1f\n", ave, sqrt(sd));
      fprintf(fp2,"平均 = %.1f\r\n標準偏差 = %.1f\r\n", ave, sqrt(sd));//キャリッジリターンを追加
      printf("平均と標準偏差を%sファイルに書き込みました\n", file2);
  }

で動きませんか？

Comments

[yatai]

ファイル関連は動くようになりました!
ありがとうございます!