C++ ファイルからの読み込み

Question

こんばんは。
普段Cでコードを書いている学生です。
C++のvectorを使ってファイルから一行ずつ数字を読み込んで格納するファイルを作成中なのですが、
ファイルは読み込むのになぜかvectorに格納されません。
問題のコードは以下の通りです。二つの関数の宣言は別のヘッダファイルでしてあります。

#include <stdio.h>
#include <iostream>
#include <vector>
using namespace std;


FILE *fp1;
/*(1-3)*/
void file_input(vector<double> v){
  int leng = file_counter(0);
  v.reserve(leng);
  rewind(fp1);
  int i = 0;
  double j;
	while (1) {
    fscanf(fp1, "%lf",&v[i]);
    i++;
    printf("v=%f\n",v[i]);

    if(i >= leng)
      break;
}
	fclose(fp1);
}


/*(1-5)*/
int file_counter(int flag){
  char filename [256];
  int counter = 0;
  printf("ファイル名を入力してください：");
  scanf("%s",filename);

	if ((fp1 = fopen(filename, "r")) == NULL) {
		printf("file open error!!\n");
		exit(EXIT_FAILURE);
  }
  else{
    double fake;
    while(fscanf(fp1, "%lf", &fake) != EOF){
      counter++;
    }
  }
  if(flag){
    printf("ファイルから読み取ったデータ数は%dでした\n",counter);
    fclose(fp1);
  }

  return counter;
}

尚、ファイルからでなくキーボードから入力でvectorに格納する分にはうまく行きました。
以下のコードです。

vector<double> input_key(vector<double> v, int n){
    int i;
    for(i=0;i<n;i++){
        printf("data[%d]=",i);
        scanf("%lf",&v[i]);
    }
    return v;

この二つの違いがよくわかりません。
改善案などあれば教えてください。
稚拙な日本語とコードで申し訳ありませんが、よろしくお願いします。

Answer 1

void file_input(vector<double> v){

値渡しであるために、vectorのインスタンスがコピーされてしまっています。
つまり、呼び出し元のvと、関数内のvが別物です。

参照渡しするようにすれば良いでしょう。

尚、ファイルからでなくキーボードから入力でvectorに格納する分にはうまく行きました。

この二つの違いがよくわかりません。

後者のコードも値渡しですが、戻り値としてvectorのインスタンスを返しています。
結果的に二回コピーが生じ(※)、意図通りの出力が得られているだけです。

---

※ 実際には最適化される(はず)なので、これは方便としての嘘です。

Comments

[waewae96]

ありがとうございます。
調べてもvectorの参照渡しの方法がわかりません。
簡単にどのようにすれば良いか教えていただけますでしょうか。

[episteme]

void file_input(vector<double>& v) {...}
# &を型のｹﾂに追加

[waewae96]

ありがとうございます。ただ、
関数file_input内でもvectorであるvの値が変更されません。

FILE *fp1;
/*(1-3)*/
vector<double> file_input(vector<double> &v){
int leng = file_counter(0);
v.resize(leng);
rewind(fp1);
int i = 0;
while (1) {
fscanf(fp1, "%lf",&v[i]);
i++;
printf("v=%f\n",v[i]);

if(i >= leng)
break;
}
fclose(fp1);
return v;
}

file_counterについては質問に乗せたものと同じです。
上記のようにv[]の値を全て表示させてみたのですが、全てv[i]=0.0000となってしまいます。
v[]に読み込ませずに確認したところ、ファイルの読み込みは、うまくいっているようです。
v[]への書き込みがうまくいかないようです。
どこが間違っているでしょうか。

[episteme]

fscanf(fp1, "%lf",&v[i]); // 読んで
i++;
printf("v=%f\n",v[i]); // 次の要素をプリント(だめじゃん)

↑この出力を根拠にうまくいかないって言ってる?

[waewae96]

そうです！
全て0.00になってしまいます

[episteme]

だからー、それじゃダメなのあたりまえでしょ?
fscanf(fp1, "%lf",&v[i]); // 読んで
printf("v=%f\n",v[i]); // その要素をプリント
i++; // 次へ
じゃなきゃ。

[waewae96]

本当だ！
ありがとうございます笑
変なミスしてましたね笑

[episteme]

解決したんなら適切に closeしてねー

[LouiS0616]

@episteme さん
なにかとありがとうございます。。。

[episteme]

いやだから、"best-answer"なり"解決済み"にしろって。

Answer 2

呼び出し側のコードが提示されていないのでなんとも言えないですが、
下のバージョンのinput_keyを呼び出している時はvector<double> vにちゃんとn個分の要素があるvector<double>を渡して呼び出しているからうまく行ってるのでは？

上のコードでは

v.reserve(leng);

とやっていますが、std::vectorのreserveは領域の予約を行う関数で実際にleng分の要素を確保するわけではないです。

代わりに、

v.resize(leng);

とすることでうまく動くのではないかと思います。

C/C++はパフォーマンス上の理由から実行時に添字の範囲チェックなどはしないので、上のコードでも偶然動いてしまいうるんですが、Debugビルドだと実行時範囲チェックをしてくれる処理系も多いので、実はログなどに"Index is out of range"みたいな例外を出してプログラムが止まっているのではないでしょうか？本当に何も言わずにただただデータが入っていなかったのでしょうか？