4

我有一个小问题，我无法弄清楚为什么这段代码不起作用：

std::for_each(users.begin(), users.end(), [](Wt::WString u)
{
    std::cout << "ilosc: " << users.size() << std::endl;
    userBox_->addItem(u);
});

我在编译时遇到的错误：

GameWidget.cpp: In lambda function:
GameWidget.cpp:352:30: error: 'users' is not captured
GameWidget.cpp:353:4: error: 'this' was not captured for this lambda function
GameWidget.cpp: In member function 'virtual void GameWidget::updateUsers()':
GameWidget.cpp:354:3: warning: lambda expressions only available with -std=c++11 or -std=gnu++11 [enabled by default]
GameWidget.cpp:354:4: error: no matching function for call to 'for_each(std::set<Wt::WString>::iterator, std::set<Wt::WString>::iterator, GameWidget::updateUsers()::<lambda(Wt::WString)>)'
GameWidget.cpp:354:4: note: candidate is:
In file included from /usr/include/c++/4.7/algorithm:63:0,
                 from GameWidget.h:11,
                 from GameWidget.cpp:9:
/usr/include/c++/4.7/bits/stl_algo.h:4436:5: note: template<class _IIter, class _Funct> _Funct std::for_each(_IIter, _IIter, _Funct)
GameWidget.cpp:354:4: error: template argument for 'template<class _IIter, class _Funct> _Funct std::for_each(_IIter, _IIter, _Funct)' uses local type 'GameWidget::updateUsers()::<lambda(Wt::WString)>'
GameWidget.cpp:354:4: error:   trying to instantiate 'template<class _IIter, class _Funct> _Funct std::for_each(_IIter, _IIter, _Funct)'

我正在使用gcc 4.7.3，所以我的编译器可能支持 C++11。

userBox_是一个集合，BOOST_FOREACH适用于此代码：

BOOST_FOREACH(Wt::WString i, users)
{
    std::cout << "ilosc: " << users.size() << std::endl;
    userBox_->addItem(i);
}

感谢您的任何回答，我很好奇为什么会这样。

4

3 回答 3

9

您编写的 lambda 不会捕获任何上下文变量。为此，最简单的方法是添加&到 lambda 的捕获列表中。这将通过引用捕获所有上下文变量，您将能够在 lambda 中访问它们。

std::for_each(users.begin(), users.end(), [&](Wt::WString u)
{
    std::cout << "ilosc: " << users.size() << std::endl;
    userBox_->addItem(u);
});

我不明白你为什么要users.size()在循环中打印，因为看起来每次迭代的输出都是一样的。如果您将其移到循环之外，并且希望对 lambda 捕获的内容进行更细粒度的控制，您可以修改捕获列表以仅捕获this指针。这将允许您访问成员变量userBox_。

std::cout << "ilosc: " << users.size() << std::endl;
std::for_each(users.begin(), users.end(), [this](Wt::WString u)
{
    userBox_->addItem(u);
});

MSDN 有一篇很好的文章，非常详细地解释了lambda 表达式语法。

最后，在您的情况下，不需要std::for_eachand lambda。for使用基于范围的循环将项目添加到集合中会更简洁。

for( auto const& u: users ) {
  userBox_->addItem(u);
}
于 2013-05-26T21:36:51.397 回答
1

您需要知道的一切都在您收到的错误中。

您通过使用“[]”明确告诉 lambda 不要捕获任何内容，这意味着它在函数体内可以访问的唯一变量是参数和全局变量。

userBox_ 是什么类型无关紧要，它是一个成员变量，因此 lambda 需要捕获“this”。

最后，您通过值传递，这意味着您将复制每个 Wt::WString。您可能想改用

std::for_each(users.begin(), users.end(), [&](const Wt::WString& u) {
...
});

“&”通过引用捕获并将为您捕获“this”。

http://en.cppreference.com/w/cpp/language/lambda

于 2013-05-26T21:40:41.777 回答
0

捕获变量 users 意味着将您的 lambda 声明为：

    [users](Wt::WString u){...}

它将users作为只读变量传递。

为了能够修改users，您需要将您的 lambda 声明为：

    [&users](Wt::WString u){...}
于 2013-05-26T21:34:11.503 回答