2012-11-19 17 views
18

次のプログラムをキャプチャされていません。ラムダ:関数がコンパイルされません

#include <iostream> 
#include <vector> 
#include <functional> 
#include <algorithm> 
#include <cstdlib> 
#include <cmath> 

void asort(std::vector<double>& v, std::function<bool(double, double)> f) 
{ 
    std::sort(v.begin(), v.end(), [](double a, double b){return f(std::abs(a), std::abs(b));}); 
} 

int main() 
{ 
    std::vector<double> v({1.2, -1.3, 4.5, 2.3, -10.2, -3.4}); 
    for (unsigned int i = 0; i < v.size(); ++i) { 
     std::cout<<v[i]<<" "; 
    } 
    std::cout<<std::endl; 
    asort(v, [](double a, double b){return a < b;}); 
    for (unsigned int i = 0; i < v.size(); ++i) { 
     std::cout<<v[i]<<" "; 
    } 
    std::cout<<std::endl; 
    return 0; 
} 

理由:

error : 'f' is not captured 

それが何を意味し、どのように問題を解決するために?

答えて

37

asort()のラムダでfパラメータを使用しますが、キャプチャしません。キャプチャリストにfを追加してください([]を変更して[&f]と読み替えてください)。

+7

また、[[&] 'を使って参照することもできます。 – Cameron

+0

はい、あなたは正しいです、私は私の答えを更新しました。なんらかの理由で私はあまりにも速く読み込み、 'f'は関数オブジェクトではなく関数ポインタであると考えました。 – cdhowie

7

あなたはラムダの外側スコープの変数であるfを効果的に参照しています。キャプチャリストにキャプチャする必要があります(参照を使用してすべてをキャプチャするのは、おそらく参照[& f]または[&]です)。

std :: functionはタイプ消去を実行する際にオーバーヘッドがありますが、ここではテンプレートタイプを導入する方が良いかもしれません。

関連する問題