コマンドライン引数でtextファイルの名前を入れて、読ませたい

Question

前提・実現したいこと

コマンドライン引数で C:\sample11-1.txt を読ませて、計算をさせたい

発生している問題・エラーメッセージ

``
アドバイスをくださった方々ありがとうございます
gcc ファイル名.c でコンパイルすると無事にコンパイルできて、
./a.exe C:sample11-1.txt
と打ったのですが反応がありません
(何も表示せずにプログラムが終わってしまっています)
コマンドライン引数でもらったデータをcharに変換することができていないのかもと思いますが調べてもよく分かりませんでした。

該当のソースコード

ソースコード
#include <stdio.h> 
#include <stdlib.h>
#include <string.h>
#define MAX_DATA 1000

int main(int argc, char *argv) {
   if (argc != 2) { printf("使用法: %s <文字列>\n", argv[0]); return 1; }  

  char buf[256]; double weight[5]; double sum; int i, n; sum = 0;    
  FILE *fp;
  char *filename;
  *filename = argv[1];
  fp = fopen(filename, "r");
  if (fp == NULL) { printf("ファイル%sが開けません．\n", filename); return 1; }  //ファイルオープンのエラー
  for (n = 0; n < MAX_DATA; n++) {
  if (fgets(buf, 256, fp) == NULL) break;
  weight[n] = atof(buf);
  sum += weight[n];
  }
  fclose(fp);
  printf("平均 = %5.1lf\n", sum/5);
  return 0;
}

試したこと

./a.exe C:\sample11-1.txt で打ちました

補足情報（FW/ツールのバージョンなど）

Visual studio code を使っています。

Comments

[hoshi-takanori]

質問は編集できますので、ソースコードを貼ってくれないと何とも言えません。

[takasima20]

それ gcc が所定のファイル名以外を指定したときに出すエラーメッセージじゃ?

[takasima20]

あー、やっぱコンパイル時に指定してたか。
コンパイルしてできた実行ファイルに指定しましょう。
インタープリタあたりと混同してるのかな?

Answer 1

*filename = argv[1];
を
filename = argv[1];
に修正しましょう。

argv[1]は文字列の先頭アドレスなので、それを代入するのは*filenameではなく、filenameです。

初期化されていないfilenameを使って、*filenameに代入しようとすると、セグメンテーションフォールトしたりします。

Answer 2

引数は入力できたと思うのですが、実行しても何も起こりませんでした。

手元の環境(Linux)では異常終了のメッセージが出ましたが、Windowsでは何のメッセージ等も出ないんですか？

Segmentation fault (コアダンプ)

どんな処理をさせたいのかこちらでもちょっと把握しきれていない部分もあるかもしれませんが、
指定されたテキストファイルに書かれている5つの数字の平均値を求めるということなら次のような修正になります。(追記:コメントでのご指摘を受けてソースコード修正しました。)

• main()関数のargvの型がおかしいのを修正した。同時にargvの使用箇所も修正した。
• データ数として「MAX_DATA」を使うように統一した

--- hogeorg.c   2021-01-21 07:56:34.256784888 +0900
+++ hoge.c      2021-01-21 10:36:44.730806905 +0900
@@ -1,15 +1,15 @@
 #include <stdio.h> 
 #include <stdlib.h>
 #include <string.h>
-#define MAX_DATA 1000
+#define MAX_DATA 5
 
-int main(int argc, char *argv) {
+int main(int argc, char *argv[]) {
        if (argc != 2) { printf("使用法: %s <文字列>\n", argv[0]); return 1; }  
 
-       char buf[256]; double weight[5]; double sum; int i, n; sum = 0;    
+       char buf[256]; double weight[MAX_DATA]; double sum; int i, n; sum = 0;    
        FILE *fp;
        char *filename;
-       *filename = argv[1];
+       filename = argv[1];
        fp = fopen(filename, "r");
        if (fp == NULL) { printf("ファイル%sが開けません．\n", filename); return 1; }  //ファイルオープンのエラー
        for (n = 0; n < MAX_DATA; n++) {
@@ -18,6 +18,11 @@
                sum += weight[n];
        }
        fclose(fp);
-       printf("平均 = %5.1lf\n", sum/5);
+       if ( n == 0 ) {
+               printf("データ数が0なので平均の計算はしません\n");
+       }
+       else {
+               printf("平均 = %5.1lf\n", sum/n);
+       }
        return 0;
 }

Comments

[tatsu99]

平均は、読み込んだデータの件数で割るべきかと思います。

[hidezzz]

ご指摘ありがとうございます。おっしゃる通りですね。
MAX_DATAでなく「n」で割るように回答のソースコードを修正しました。

[ys003]

返信遅れてしまい申し訳ありません
回答ありがとうございました

Answer 3

gcc ファイル名.c C:\sample11-1.txt と打ちましたが

gcc ファイル名.c

で実行ファイル a.exe が生成されたら

.\a.exe C:\sample11-1.txt

Answer 4

C:\sample11-1.txt

といれてみてはどうでしょう。
これでダメなら、コードを提示しましょう

Comments

[ys003]

回答ありがとうございます。

ターミナルに
gcc ファイル名.c sample11-1.txt
と入力したのですがダメでした。
コードを乗せたのでもしよろしければ教えていただけると助かります

[y_waiwai]

そこに入れるもんじゃありません。根本的にやり方間違ってます
コンパイル結果のEXEファイルを実行するときに入れましょう

[ys003]

理解が足りていなかったみたいです。
C:sample11-1.txtと打つときはわかりました。
ありがとうございます。
引数は入力できたと思うのですが、実行しても何も起こりませんでした。
恐らく私の書いたプログラム(特にargvとかのあたり)がおかしいと思うのですが調べても分からなかったのでよろしければ教えていただきたいです

[y_waiwai]

コード自体はおかしなところはなさそうなので、ちゃんとコンパイルできてるのかを確かめてみては