C++マージファイルの問題

Question

私は2つのファイルの番号を取り、昇順に別のファイルに出力するプログラムを作成することになっています。プログラムはエラーなしでコンパイルし、必要に応じて出力ファイルを作成しますが、ファイルは空です。私は間違って何をしていますか？

まずファイル：

1 
3 
5 
7 
9 

セカンドファイル：

2 
4 
6 
8 

-

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

void merging(ifstream& instream, ifstream& instream2, ofstream& outstream); 

int main() 
{ 
    ifstream instream, instream2; 
    ofstream outstream; 

    instream.open("merge1"); 
    if (instream.fail()) 
    { 
     cout << "Failed to open input file.\n"; 
     exit (1); 
    } 

    instream.open("merge2"); 
    if (instream2.fail()) 
    { 
     cout << "Failed to open input file.\n"; 
     exit (1); 
    } 

    outstream.open("merge3"); 
    if (outstream.fail()) 
    { 
     cout << "Failed to open output file.\n"; 
     exit (1); 
    } 

    merging(instream, instream2, outstream); 

    instream.close(); 
    instream2.close(); 
    outstream.close(); 

    cout << "Finished merging files.\n"; 

    return 0; 
} 

void merging(ifstream& instream, ifstream& instream2, ofstream& outstream) 
{ 
    int number1, number2; 

    if (instream >> number1) 


    if(instream) 
    { 
     if (instream2 >> number2) 
    { 

     while (instream && instream2) 
     { 
      if (number1 <= number2) 
      { 
      outstream << number1 << " "; 
      } 

      if (! (instream >> number1)) 
      { 
      outstream << number2 << " "; 

      } 

     } 

      else 
     { 
      outstream << number2 << " "; 


      if (! (instream >> number1)) 
      { 
       outstream << number1 << " "; 

      } 
      else 
     } 
     } 
    } 
     else 
    { 
     outstream << number1 << " "; 

    } 
    } 

    while (instream >> number1) 
    { 
     outstream << number1 << " "; 

    } 

    while (instream2 >> number2) 
    { 
     outstream << number2 << " "; 

    } 


} 

Answer 1

問題がここにあります：

instream.open("merge1"); 
... 
instream.open("merge2"); 

instreamを2回開いた。

#include <iostream> 
#include <fstream> 
#include <vector> 
#include <algorithm> 

... 
std::vector<int> vec; 

int i; 
while (instream1 >> i) 
    vec.push_back(i); 

while (instream2 >> i) 
    vec.push_back(i); 

//sort the values 
std::sort(vec.begin(), vec.end()); 

//output 
for (auto n : vec) 
    outstream << n << "\n"; 

Answer 2

Barmak Shemiraniは、正しい答えを持っている：あなたは、おそらくファイルの準備ができたら、単にstd::vectorにコンテンツを読み、次のように並べ替え、instream、その後instream2

を開くことを意味しました。

弱い順序で改善されているstd:set<>は、すでに弱い順序で並べられています。つまり、別の並べ替え手順は必要ありません。

std::set<int> numbers; 
int i; 

while (instream >> i) 
    numbers.insert(i); 

while (instream2 >> i) 
    numbers.insert(i); 

for (auto n : numbers) 
    outstream << n << "\n";