マップイテレータを使ってofstreamに書き込む方法

Question

私はofstreamというマップを持っています。私はメッセージのパケットを調べ、特定のシンボルに関する各メッセージをそれぞれのファイルに書きたいと思っています。本質的には、パケットファイルをより小さなものに分割します。 ofstreamにI <<が書き込まれるとファイルに書き込まれます。しかし、マップイテレータ経由でアクセスしたofstreamへの<<を試してみると、うまくコンパイルされてもクラッシュします。

私がマップを使用している理由は、パケットヘッダーが与えられたシンボルに対して複数回書き込まれないようにするためです。

void write_packet_to_symbol_file(packet p) 
{ 

    string path = "E:\\20170131\\"; 
    map<string,ofstream&> outs; 
    for (message m : p.messages) { 
    map<string,ofstream&>::iterator it = outs.find(m.symbol.name); 
    if (it == outs.end()) { 
     string full_path = path + m.symbol.name + ".CAP"; 
     ofstream of; 
     of.open(full_path, ios_base::app); 
     // write packet header since first message for that symbol 
     of << p.get_header(); 
     outs.emplace(m.symbol.name,of); 
    } 

    map<string,ofstream&>::iterator it2 = outs.find(m.symbol.name); 
    if (it2 != outs.end()) 
     it2->second << m.get_message_content(); 
} 

私は間違っていますか？

Answer 1

あなたのofstreamインスタンスはif文のローカル変数で、if文が終了すると閉じられ、破棄されます。マップのような参照のコレクションを使うのではなく、ストリームポインタ（できればスマートポインタ）のマップを作成し、ストリームを動的に作成し、既に書いたすべてのコードをスクラップする必要があります。

Answer 2

mapの文字列からのストリーム参照があるようです。私はそれがどのようにコンパイルされているのか分かりませんが、参照されたオブジェクトが破壊されるので、間違いなくバグです。

私はあなたがマップに、あなたはit->second.open()を呼び出す必要が返さイテレータであなたべき.insert().emplace()その後map<string,ofstream> を使うべきだと思います。

ifstreamがコピー可能ではなくmake_pairコールをスキップしている、emplaceに対する同様

map.insert(make_pair(some_key, ifstream(...))); //this will work, calls move 

ifstream x; 
map.insert(make_pair(some_key, x)); //this will NOT work, calls copy 

map.insert(make_pair(some_key, std::move(x))); // should work 

ように移動可能であるので、これは少し難しいかもしれません。

Answer 3

他の人はすでに説明したように、ifブロック内に一時的なストリームオブジェクトのみを作成しています。 ifブロックが終了したときにこのオブジェクトが破棄され、マップに格納された参照が無効になります。

map<string,ofstream>が必要なのは、マップがストリームオブジェクトを参照するだけでなく実際に格納する必要があるためです。

今、私たちは問題を抱えています。一時的なストリームをどのようにマップに入れるのですか？ストリームにはコピーコンストラクタや代入演算子がないため、ストリームオブジェクトのコピーはできません。

オペレータ[]を使用してストリームオブジェクトをマップ内に直接作成できるため、一時オブジェクトは不要です。

したがって、一部のユーザーの提案に応じて、ポインタやセマンティクスは必要ありません。

void write_packet_to_symbol_file(packet p) 
{ 

    string path = "E:\\20170131\\"; 
    map<string,ofstream> outs; 
    for (message m : p.messages) { 
    map<string,ofstream>::iterator it = outs.find(m.symbol.name); 
    if (it == outs.end()) { 
     // Create stream object in the map and get reference 'of' to it 
     ofstream& of = outs[ m.symbol.name ]; 

     string full_path = path + m.symbol.name + ".CAP"; 
     of.open(full_path, ios_base::app); 
     // write packet header since first message for that symbol 
     of << p.get_header(); 

     // No need to insert 'of' into 'outs', because it is already in there! 
    } 

    map<string,ofstream>::iterator it2 = outs.find(m.symbol.name); 
    if (it2 != outs.end()) 
     it2->second << m.get_message_content(); 
}