2012-02-15 59 views
3

我想写一个复杂的算法列表,必须应用于范围序列。我想使用类似于以下代码的语法来嵌套许多算法。我唯一的问题是它不会编译。有什么建议么?结合增强范围算法,例如。升压::复制和提升:: remove_if

bool pred(double x); 

double d[]={1,2,3,4}; 
std::vector<double> x(d,d+4); 
std::vector<double> y; 
boost::copy(x, std::back_inserter(y)); // OK 
boost::copy(boost::remove_if(x,&pred), std::back_inserter(y)); // ERROR 

对于这种语法的工作,我不想为内部算法指定模板参数。这可以通过使用auto关键字来简化,但我需要保持代码向后兼容。

这是错误消息的片段:

1>c:\dev\thirdparty\boost\boost-1.48.0-windows-vc90-x32-p64925\installed\include\boost-1_48\boost/range/iterator.hpp(63) : error C2039: 'type' : is not a member of 'boost::mpl::eval_if_c<C,F1,F2>' 
1>  with 
1>  [ 
1>   C=true, 
1>   F1=boost::range_const_iterator<std::_Vector_iterator<double,std::allocator<double>>>, 
1>   F2=boost::range_mutable_iterator<const std::_Vector_iterator<double,std::allocator<double>>> 
1>  ] 

1>  c:\dev\thirdparty\boost\boost-1.48.0-windows-vc90-x32-p64925\installed\include\boost-1_48\boost/range/concepts.hpp(256) : see reference to class template instantiation 'boost::range_iterator<C>' being compiled 
1>  with 
1>  [ 
1>   C=const std::_Vector_iterator<double,std::allocator<double>> 
1>  ] 
1>  c:\dev\thirdparty\boost\boost-1.48.0-windows-vc90-x32-p64925\installed\include\boost-1_48\boost/concept/detail/has_constraints.hpp(42) : see reference to class template instantiation 'boost::SinglePassRangeConcept<T>' being compiled 
1>  with 
1>  [ 
1>   T=const std::_Vector_iterator<double,std::allocator<double>> 
1>  ] 
1>  c:\dev\thirdparty\boost\boost-1.48.0-windows-vc90-x32-p64925\installed\include\boost-1_48\boost/concept/detail/msvc.hpp(53) : see reference to class template instantiation 'boost::concepts::not_satisfied<Model>' being compiled 
1>  with 
1>  [ 
1>   Model=boost::SinglePassRangeConcept<const std::_Vector_iterator<double,std::allocator<double>>> 
1>  ] 
1>  c:\dev\thirdparty\boost\boost-1.48.0-windows-vc90-x32-p64925\installed\include\boost-1_48\boost/range/algorithm/copy.hpp(33) : see reference to class template instantiation 'boost::concepts::require<Model>' being compiled 
1>  with 
1>  [ 
1>   Model=boost::SinglePassRangeConcept<const std::_Vector_iterator<double,std::allocator<double>>> 
1>  ] 
1>  ..\..\..\..\..\source\yotta\libraries\snl\unittests\StackOverflow.cpp(7) : see reference to function template instantiation 'OutputIterator boost::range::copy<std::_Vector_iterator<_Ty,_Alloc>,std::back_insert_iterator<_Container>>(const SinglePassRange &,OutputIterator)' being compiled 
1>  with 
.... 
+0

执行范围必须按引用传递?或者他们可以是一个包含对范围的引用的类? – Adrian 2012-02-15 12:14:44

+0

+1“我唯一的问题是它不会编译”。 'cout << the_answer_to_it_all();'唯一的问题是不能编译... – 2012-06-19 11:09:26

回答

6

的问题是remove_if(和所有的算法,对于这个问题)返回单个迭代,而不是一个范围,如果你不指定应该返回的范围类型(​​您似乎不需要)。

一个简单的解决将是使用the adaptor filtered

#include <boost/range/adaptor/filtered.hpp> 
#include <boost/range/algorithm/copy.hpp> 
#include <iterator> 
#include <vector> 
#include <iostream> 

bool pred(double x){ 
    return x > 2.0; 
} 

int main(){ 

    double d[]={1,2,3,4}; 
    std::vector<double> x(d,d+4); 
    std::vector<double> y; 
    boost::copy(x | boost::adaptors::filtered(pred), std::back_inserter(y)); 
    boost::copy(y, std::ostream_iterator<double>(std::cout, " ")); 
} 

输出:3 4

+0

谢谢。现在我看到了问题。 – Adrian 2012-02-15 12:49:58

+1

@Adrian:如果这个答案解决了您的问题,您可能需要考虑通过点击旁边的勾号来接受它。 :) – Xeo 2012-02-22 11:33:46

+0

第一个转换可以优化为'boost :: push_back(y,x | filtered(pred));'它不仅更短,而且调用向量的范围插入成员,而不是一系列推回。 – TemplateRex 2014-02-14 17:50:41