OutputIteratorをとり、それに書き込むC++関数

Question

OutputIteratorをとり、値を書き込むC++関数を記述しようとしていますが、さまざまな問題が発生しています。リスト<>イテレータを渡すと、リストは埋められません。ベクトル<>イテレータを渡すと、（Linux上で）セグメント化エラーが発生します。

私は"populating an std::[container] from a function by passing an output iterator"と"How to write a function that takes an iterator or collection in a generic way?"からの議論に続きました。

アイデア？私は他のSTL関数std：copy（）と同じように、出力コンテナのbegin（）でinsertValues（）コードを呼び出したいと思います。

#include <cstdio> 
#include <list> 
#include <vector> 

using namespace std; 

template<typename OutputIterator> 
void insertValues(OutputIterator result) 
{ 
    for (int i = 0; i < 10; i++) 
    { 
     *(result++) = i; 
    } 
} 

int main(int argc, char **argv) 
{ 
    // This code produces 0 items in the list. 
    list<int> values_list; 
    insertValues(values_list.begin()); 
    printf("values has %d items\n", (int) values_list.size()); 

    // This code produces a seg fault. 
    vector<int> values_vector; 
    insertValues(values_vector.begin()); 
    printf("values has %d items\n", (int) values_vector.size()); 
} 

Answer 1

リストやベクターに十分な要素がないため、コードが壊れます。出力イテレータはを追加せず、要素を追加するだけで、の既存の要素に書き込むことに注意してください。次のように

したがって、あなたはそれを書き換えることができ：

template<typename Container> 
void insertValues(Container &result) 
{ 
    for (int i = 0; i < 10; i++) 
    { 
     result.push_back(i); 
    } 
} 

とmainで：

insertValues(values_list); 
insertValues(values_vector); 

代わりに、push_back動作を行う反復子を使用することができます。これは、標準ライブラリーのback_inserter（#include <iterator>）という名前です。それを使用してinsertValuesコードを変更しないでください：

insertValues(back_inserter(values_list)); 
insertValues(back_inserter(values_vector));