fstreamクラスでデータを読み込みたい

Question

#include <iostream>
#include <fstream>
#include <vector>
using namespace std;

int main(){

int num;
vector <int> array;

ifstream fin( "nums10.txt" ); // 皆様のご指摘で気づいて修正しました。 
if( !fin ){
cout << "Error." << endl;
return -1;
}

while( !fin.eof() ){ // ファイルの整数データを全て読み込むにはどうすれば？
fin >> num;
array.push_back( num );
}

for( int i = 0; i < array.size(); i++ )
cout << array[ i ] << endl;

return 0;
}

コード

-nums10.txt-
39 770 711 662 143 574 528 370 355 227

-出力結果-
39
770
711
662
143
574
528
370
355
227
227 // 重複して読み込んでしまう.

"nums10.txt" にある１０個の整数値を読み込みたいのですが、最後のデータを重複して読み込んでしまいます。

どうすればピッタリ読み込むように実装出きるでしょうか？

Answer 1

ofstream -> ifstreamや、重複して読み込む理由はChironianさんがおっしゃっている通りです。
私なりに書いてみました。

#include <iostream>
#include <fstream>
#include <vector>
using namespace std;

int main() {
    ifstream fin("nums10.txt");
    if (!fin) {
        cout << "Error." << endl;
        return -1;
    }

    vector<int> nums;
    int num;
    while (fin >> num) {
        nums.push_back(num);
    }

    for (int& n : nums) {
        cout << n << endl;
    }

    return 0;
}

Comments

[Chironian]

おお、これはスマートですね。

[carimatics]

競技プログラミング以外でC++を書かないので、一般的にif文の条件式内での読み込み処理がお行儀の良い作法かどうかは分かりませんが…。

Answer 2

こんにちは。

まず、ofstream fin( "nums10.txt" );はifstream fin( "nums10.txt" );の間違いですね？
次に、間違い易いのですが、データを読み取った直後はまだfin.eof()が成立しません。その次にデータを読み取ろうとして読めなかった時に成立します。
なので、下記のようなループにすれば良い筈です。

while(fin.good())
{
    fin >> num;
    if (fin.eof())
break;
    array.push_back( num );
}

Answer 3

入力ファイルの末尾に改行または空白文字が入っていると、そのような結果になります。
最終データが入力できなかったにもかかわらず配列に追加してしまいます。

以下のように判定を加えるとよいでしょう。
C++でストリームほとんど使ったことないので、これで正しいか確信は持てないですが。

while( !fin.eof() ){ // ファイルの整数データを全て読み込むにはどうすれば？
    if( fin >> num){ // 「入力できた」ら追加
        array.push_back( num );
    }
}

Answer 4

出遅れたー

#include <iostream>
#include <fstream>
#include <iterator>
#include <vector>
using namespace std;

int main(){
  vector<int> array(istream_iterator<int>(ifstream("nums10.txt")), istream_iterator<int>());
  for ( auto item : array ) { cout << item << ' '; } // 読めたかな?
}