Использование for_each и boost :: bind с вектором указателей - PullRequest
9 голосов
/ 10 марта 2010

У меня есть вектор указателей. Я хотел бы вызвать функцию для каждого элемента, но эта функция берет ссылку. Есть ли простой способ разыменования элементов?

Пример:

MyClass::ReferenceFn( Element & e ) { ... }

MyClass::PointerFn( Element * e ) { ... }

MyClass::Function()
{
    std::vector< Element * > elements;
    // add some elements...

    // This works, as the argument is a pointer type
    std::for_each( elements.begin(), elements.end(),
                   boost::bind( &MyClass::PointerFn, boost::ref(*this), _1 ) );

    // This fails (compiler error), as the argument is a reference type
    std::for_each( elements.begin(), elements.end(),
                   boost::bind( &MyClass::ReferenceFn, boost::ref(*this), _1 ) );
}

Я мог бы создать маленькую грязную оболочку, которая берет указатель, но я подумал, что должен быть лучший способ?

Ответы [ 2 ]

15 голосов
/ 10 марта 2010

Вы можете использовать boost::indirect_iterator:

std::for_each( boost::make_indirect_iterator(elements.begin()), 
               boost::make_indirect_iterator(elements.end()),
               boost::bind( &MyClass::ReferenceFn, boost::ref(*this), _1 ) );

Это разыменует адаптированный итератор дважды в его operator*.

3 голосов
/ 10 марта 2010

Похоже, вы также можете использовать библиотеку Boost.Lambda .

// Appears to compile with boost::lambda::bind
    using namespace boost::lambda;
    std::for_each( elements.begin(), elements.end(),
                   bind( &MyClass::ReferenceFn, boost::ref(*this), *_1 ) );

Но я согласен с комментаторами по поводу предпочтения BOOST_FOREACH. «Алгоритм» for_each практически ничего не делает, и то, что он делает, основанный на диапазоне цикл for может сделать для вас гораздо меньше усилий.

...