2012-11-19 13 views
18

Das folgende Programm erfasst kompilieren nicht:Lambda einer Lambda: die Funktion nicht

#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; 
} 

weil:

error : 'f' is not captured 

Was ist das und wie das Problem zu lösen?

Antwort

37

Sie verwenden den f Parameter im Lambda innerhalb asort(), aber Sie erfassen es nicht. Versuchen Sie, f zu der Erfassungsliste hinzuzufügen (ändern Sie [], um [&f] zu lesen).

+7

Oder sogar erfassen Sie es durch Bezugnahme mit '[&]'. – Cameron

+0

Ja, Sie haben Recht, ich habe meine Antwort aktualisiert. Aus irgendeinem Grund habe ich zu schnell gelesen und dachte, 'f' sei ein Funktionszeiger, kein Funktionsobjekt. – cdhowie

7

Sie verweisen effektiv auf f, eine Variable im äußeren Bereich, in Ihrem Lambda. Sie sollten es in Ihrer Aufnahmeliste erfassen (am einfachsten ist es wahrscheinlich durch Bezugnahme [& f], oder [&] um alles durch Bezugnahme zu erfassen, wie Sie es sofort verwenden).

Auf einer anderen Anmerkung, std :: function hat einige Overhead, wie es Typ löschen führt, in Ihrem Fall hier könnte es besser sein, einen Schablonentyp einzuführen.

Verwandte Themen