C言語 ファイルから数値読み込みの際、“ファイル名の一部を変更して繰り返す”方法

Question

txtファイルからのデータの読み込みについてです．
a_data_1224_1.txt
a_data_1224_2.txt
:
:
a_data_1224_100.txt

には数値が100個ずつ入っています．これらのファイルから数値を取り出し配列に格納，各ファイルの1,2,....100番目の数値の平均をとるプログラムを作ろうとしました．

＿＿＿＿＿＿＿＿＿＿＿＿
【追記】
1.txtの1番目、2.txtの1番目、...100.txtの1番目　の平均

1.txtの2番目、2.txtの2番目、...100.txtの2番目　の平均

...

1.txtの100番目、2.txtの100番目、...100.txtの100番目　の平均

をheikin[]に格納したいです。わかりづらくてすみませんでした。

＿＿＿＿＿＿＿＿＿＿

いちいちファイル名の数字を変えて100個ぶん読み込みのコードを書くのは現実的ではなかったので，ファイル名の数字だけ変えて繰り返すようにしたいのですが，ファイルが読み込めません.どこがおかしいのでしょうか．また，以下の形以外にも代案はありますでしょうか．
よろしくお願いいたします．

#include <stdio.h>
 
int main(void)
{
    int i;
    int k;
    double data[100];
    double copy_data[100];
    double heikin[100];
    char input_files[]={"a_data_1224_1.txt"};
    FILE *fp;
    for(i=0; i<100; i++)
    {
       heikin[i] = 0;
    } //初期化
    for(k=0; k<100; k++)
      {
           input_files[12] = '1'+k;
         //a_data_1224_""1"".txt この1を1～100に変えていく
     fp = fopen(input_files,"r");
 
 if(fp == NULL)
   {
     printf("file not open \n");
     return -1;
   }//ファイルが開けないときのエラー
 
 for(i=0; i<100; i++)
   {
   fscanf(fp, "%lf", &(data[i]));
   }
 fclose(fp);
 
 for(i=0; i<100; i++)
   {
     copy_data[i] = data[i];
     heikin[i] += copy_data[i]; //1+k個目のファイルまでの各データの平均
   }
  } 
}

Answer 1

char input_files[64];
...
sprintf(input_files,"a_data_1224_%d.txt",i);

ってのでどうでしょうか

Comments

[Cstdy_sun8]

ありがとうございます。sprintfを使うのですね

質問なのですが、
・なぜinput_filesの要素が64なのか？
・具体的に挿入・変更するとどうなるのか？

をお聞きしたいです。

[y_waiwai]

その６４というのはあんまし意味はなく、iが十分大きな値でも異常アクセスになりえない程度のサイズということでテキトーに決めた値です

> input_files[12] = '1'+k;
文字に足してますがこれでは意味不明です。
ｋが１０以上になると、１２文字目は数字ではない文字になってしまいます

[rubato6809]

y_waiwaiさんはテキトーに決めたんですね。私もテキトーに決めることが多いけど、この場合はガッチリ決めることができますよ。

input_files[] は　"a_data_1224_1.txt" 〜 "a_data_1224_100.txt" というファイル名を格納する文字配列ですから、この中で一番長いファイル名を格納できる大きさがあれば良いのです。
最長は "a_data_1224_100.txt" の１９文字でしょ。それに終端文字'\0' 分を足した２０バイトの大きさがあれば良いということ。
よって、 char input_files[20]; まで絞り込めます。

[rubato6809]

ホントに最近の質問見て知ったのだけど、これ
char input_files[ sizeof "a_data_1224_100.txt" ];
という書き方ができるんだって！！

[thkana]

ファイル名のフォーマットがどう変わっても対処できるようにFILENAME_MAXマクロを充てるのがいいような気が私はしますがどうでしょう? 過剰でしょうか。
(今の時代、PCベースなら数KBをケチる必要はないでしょう。RAMが2KBしかないArduino UNOでSDカードを使う、とかなら別ですが)
---
> char input_files[ sizeof "a_data_1224_100.txt" ];
それも提案しようかと思ってましたが(後出し:-)。その場合でも

#define FORMAT "a_data_1224_%d.txt"
#define MAX_DIGITS 3

char input_files[sizeof(FORMAT)+MAX_DIGITS-2]; //桁数の補正
sprintf(input_files,FORMAT,i);

とかのマクロ化はしたくなります。

[rubato6809]

そういうマクロを作る時点で、どういう範囲のデータに対応するか、設計があるわけで、そこに作成者の考えが反映されますね。
上の例は最低２０バイト必要だというように、極限のラインを正確に見積もることが大事だと思います。

[nob.]

ファイルのコピーやフォルダの作成が出来るなら、
・新しいフォルダを作る
・そのフォルダに対象のファイルすべてをコピーする
・そのフォルダに存在するファイル全てに、処理(平均をとる？)を施す
というふうにすれば、ファイル名を知る必要をないのですが…
こういう話はファイル名の形式やファイル数が変更になっても、うまく行くように考えたら？
(コピーする作業自体が大変かも？)

Answer 2

ご提示のコードは平均までは計算していないようです.

#define DATA_COUNTS 100
#define FILE_COUNTS 100

int main(void) {
  int i, j;
  double data;
  double heikin[DATA_COUNTS];
  char input_files[64];
  FILE *fp;

  //初期化
  for(i=0; i<DATA_COUNTS; i++) heikin[i] = 0;

  for(j=1; j<=FILE_COUNTS; j++) {
    //a_data_1224_""1"".txt この1を1～100に変えていく
    sprintf(input_files, "a_data_1224_%d.txt", j);
    fp = fopen(input_files,"r");
    if(fp == NULL) {
      printf("%s not open\n", input_files); //ファイルが開けないときのエラー
      return -1;
    }
    for(i=0; i<DATA_COUNTS; i++) {
      fscanf(fp, "%lf", &data);
      heikin[i] += data; //各データの合計
    }
    fclose(fp);
  }

  //各データの平均
  for(i=0; i<DATA_COUNTS; i++) heikin[i] /= FILE_COUNTS;
}

Comments

[Cstdy_sun8]

細かい回答ありがとうございます。やはりsprintfを使うのですね。
平均出し忘れ(割ってない)はミスでした。すみません。

heikin[0]＝1.txtの1番目、2.txtの1番目、....、100.txtの1番目　の平均
heikin[1]＝1.txtの2番目、2.txtの2番目、....、100.txtの2番目　の平均
.
.
.
heikin[99]＝1.txtの100番目、2.txtの100番目、....、100.txtの100番目　の平均

となるようにしたいのですが、その場合はこのコードどこを変更すればいいでしょうか...？わかりづらくてすみません。

[jimbe]

そのようなコードにしたつもりですが, なっていませんでしょうか.

[rubato6809]

> やはりsprintfを使うのですね
私もそれが一番やりやすいと思います。

念の為、input_files[12] = '1'+k;　は明確なバグですよ。k が９になったら、そこは数字じゃなくなります。どんな文字列になるか、そこだけ確かめると良いですよ。

[Cstdy_sun8]

＞jimbeさん
目的のものでした。頭であれこれ考える前に手を動かすべきでしたね。ありがとうございます。

＞rubato6809さん
はい、桁増えるのを考えていませんでした。ご指摘ありがとうございます。