一般的に関数をC++テンプレートに渡す

Question

template<typename T, typename F> 
auto mapV(vector<T> v, F f) { 
    vector<T> ret = {}; 
    for(int i=0; i < v.size(); i++) { 
     ret[i] = f(v[i]); 
    } 
    return ret; 
} 

int main() { 
    vector<int> v = {1,2,3,4,5,6}; 
    vector<int> b = mapV(v, [](int &t){ return t*2; }); 
    for (int n : b) { 
     cout << "is " << n << endl; 
    } 
    return 0; 
} 

このコードはコンパイルされますが、実行するとsegfaultが生成されます。テンプレートを使用して一般的に関数を受け入れる正しい方法は何ですか？私は関数の構造も定義したいと思います.Tをとり、Tを返す関数だけを受け入れることができますが、それを表現する方法はあまりよく分かりません。

これは、空のベクターを作成し、私はstd::function<T(T)>を試みたが、それはうまくいきませんでしたどちらか

Answer 1

vector<T> ret = {}; 

。

for(int i=0; i < v.size(); i++) { 
    ret[i] = f(v[i]); 

これは、ベクトルretの値に何かを割り当てます。ベクトルretには既存の値はありません。 ret[0]はありません。 ret[1]などはありません。これは未定義の動作です。 operator[]はベクトルに新しい値を追加せず、既存の値のみを変更します。

これをpush_back()メソッドに置き換えます。