最近,我使用boost-range在满足特定条件的元素上创建范围。在所有情况下,我总是使用相同类型的过滤范围,因此我试图将这种行为封装在外部函数中。
这就是我的问题开始的地方。考虑下面的例子。
#include <boost/range/adaptor/filtered.hpp>
#include <iostream>
#include <vector>
auto myFilter = [](const std::vector<int>& v, int r) {
return v | boost::adaptors::filtered([&r](auto v) { return v%r == 0; });
};
int main(int argc, const char* argv[])
{
using namespace boost::adaptors;
std::vector<int> input{ 1, 2, 3, 4, 5, 6, 7, 8, 9 };
for (auto& element : input | filtered([](auto v) {return v % 2 == 0; } ))
{
std::cout << "Element = " << element << std::endl;
}
std::cout << std::endl;
for (auto& element : myFilter(input,4))
{
std::cout << "Element = " << element << std::endl;
}
return 0;
}第一个for-循环的行为与预期的打印4和8相同,而第二个for-循环只打印4。为什么?
我的第二个想法是实现一个具有begin()和end()函数的类。这应该是range对象的薄包装器。
这就是解决方案,在修改了范围迭代器的类型之后。
struct MyFilter {
MyFilter(const std::vector<int>& c, int r) : c(c), r(r), f([&r](auto v) { return v%r == 0; }) {
}
boost::range_detail::filtered_range<std::function<bool(int)>, std::vector<int>>::iterator begin() {
return rng.begin();
}
boost::range_detail::filtered_range<std::function<bool(int)>, std::vector<int>>::iterator end() {
return rng.end();
}
std::vector<int> c;
int r;
std::function<bool(int)> f;
boost::range_detail::filtered_range < std::function<bool(int)>, std::vector<int>> rng=c | boost::adaptors::filtered(f);
};用法应该类似于:
for (auto& element : MyFilter(input, 4)) {
std::cout << "Element = " << element << std::endl;
}不幸的是,它只印了4张,哪一件对我来说很奇怪?
现在,我自己找到了解决办法。我必须删除我的lambda函数中的"&“才能使它工作!
发布于 2018-06-04 09:38:17
在:
auto myFilter = [](const std::vector<int>& v, int r) {
return v | boost::adaptors::filtered([&r](auto v) { return v%r == 0; });
};它返回另一个范围适配器,而引用捕获的r则成为一个悬空引用。要修复它,按值捕获r:
auto myFilter = [](const std::vector<int>& v, int r) {
return v | boost::adaptors::filtered([r](auto v) { return v%r == 0; });
}; ^
+--- capture by valuehttps://stackoverflow.com/questions/50673980
复制相似问题