c++ - std::generate_n と std::back_inserter を組み合わせるこの正しい方法はありますか?

Question

私の探求では:)できるだけ多くのSTLを使用するために、std::generateとstd::back_inserterを組み合わせて使用​​して、次のコードと同じことを行うことができるのではないかと思いました:

static const size_t nitems=1024*1024;
std::string mrbig;
for (size_t pos=0; pos<nitems; ++pos)
    mrbig.push_back('a'+ (rand()%26));

私は試した

 std::generate_n(std::back_inserter(mrbig),nitems,[](){return 'a'+(rand()%26);});

問題なく動作しているようですが、何かを台無しにしていないことを確認したいと思います。

Answer 1

generate_nは、その最初の引数が を満たすことを要求しOutputIteratorます。back_insert_iteratoriterator_categoryoutput_iterator_tag

コードの潜在的な問題:

std::generate_n(std::back_inserter(mrbig),nitems,[](){return 'a'+(rand()%26);});

• 通話mrbig.reserve(nitems)がより効率的になります
• cの代わりにc ++03 std::randまたはc++11 uniform_int_distribution<>を使用する必要があります。<random> rand
• ラムダ キャプチャとラムダ本体の間の括弧()は、ラムダが引数を取らない場合は不要ですが、好む人もいます。
• 実装文字セットで、文字a-zが連続したブロックを形成することは保証されていません。たとえば、EBCDIC では、iとjは隣接していません。移植可能な唯一の形式は"abcd...xyz"、プログラムに文字列を格納し、それにインデックスを付けることだと思います: C で a から z および A から Z を実行する単一の for ループを作成するにはどうすればよいですか?