Как использовать std :: foreach с параметрами / модификациями

Question

Я обнаружил, что пишу

for(int i=0;i<myvec.size();i++)
   myvec[i]->DoWhatever(param);

много, и я хотел бы сжать это в foreach утверждение, но я не уверен, как получить param там, не вдаваясь в подробности. У меня также есть такие вещи, как

for(int i=0;i<myvec.size();i++)
   if(myvec[i]->IsOK())
      myvec[i]->DoWhatever(param);

и я бы тоже хотел переписать этого парня. Есть мысли?

О, также, по разным причинам, я не хочу использовать повышение.

Answer 1

#include <vector>
#include <algorithm>
#include <functional>

class X
{
    public:
        void doWhat(int x) {}
        bool IsOK() const {return true;}
};
class CallWhatIfOk
{
    public:
        CallWhatIfOk(int p): param(p) {}

        void operator()(X& x) const
        {   if (x.IsOK())    {x.doWhat(param);}}
    private:
        int param;
};

int main()
{
    std::vector<X>      myVec;

    std::for_each(  myVec.begin(),
                    myVec.end(),
                    std::bind2nd(std::mem_fun_ref(&X::doWhat),4)
                 );


    std::for_each(  myVec.begin(),
                    myVec.end(),
                    CallWhatIfOk(4)
                 );
}

Answer 2

  

О, также, по разным причинам, я не хочу использовать повышение.

Действительное решение, но, скорее всего, неправильное. Рассмотрим Boost как расширение STL. C ++ - это язык, управляемый библиотекой. Если вы не примете это во внимание, ваш код будет качественно хуже.

Хотя здесь можно использовать std::for_each, отсутствие лямбда-выражений в C ++ до C ++ 0x делает это утомительным. Я рекомендую использовать Boost.ForEach ! Это делает намного проще:

foreach (yourtype x, yourvec)
    if (x.IsOK())
        x.Whatever();

Answer 3

Обычно я предпочитаю написать функтор, который будет делать то, что мне нужно:

struct doWhatever {
  doWhatever(const Param& p) p(p) {}
  void operator(MyVec v&, Param p) {
    v.DoWhatever(param);
  }

private:
  Param p;
};

А затем цикл:

std::for_each(myvec.begin(), myvec.end(), doWhatever(param));

В зависимости от того, сколько вариантов у вас есть, это может быть слишком многословно. Есть много вариантов сделать это встроенным, хотя. boost :: lambda позволит вам создать нужную функцию на сайте вызова. boost :: bind (или стандартные функции связывания библиотеки) позволят вам связать параметр param с функцией, чтобы вам не приходилось каждый раз указывать его в качестве аргумента.

boost :: lambda, пожалуй, самый лаконичный и гибкий подход. Я обычно использую простой функторный подход, потому что синтаксис легче запомнить. ;)

Answer 4

хорошо, когда у нас есть компиляторы, которые поддерживают лямбда-выражения C ++ 0x, это становится простым и минимально инвазивным:

std::for_each(myvec.begin(),myvec.end(),[&](X& item){
     item->DoWhatever(param);
});

и второй пример может выглядеть так:

std::for_each(myvec.begin(),myvec.end(),[&](X& item){   
   if(item->IsOK())      
      myvec[i]->DoWhatever(param);
});

Answer 5

#include <vector>
#include <algorithm>
#include <boost/bind.hpp>
#include <boost/lambda/if.hpp>
#include <boost/lambda/bind.hpp>


struct A
{
  bool IsOK () { return true; }
  void DoWhatever (int param) {}
};

struct B
{
  bool IsOk (A * a) { return true; }
  void DoWhatever (A * a, int param) {}
};

typedef std::vector<A *> Myvec;

void main()
{
  Myvec myvec;
  int param = 1;
  B b;

  // first challenge using boost::bind (fnct in the same class)
  std::for_each (myvec.begin(), myvec.end(),
    boost::bind (&A::DoWhatever, _1, param));

  // first challenge using boost::bind (fnct in an external class)
  std::for_each (myvec.begin(), myvec.end(),
    boost::bind (&B::DoWhatever, &b, _1, param));

  // second challange using boost::lambda (fnct in the same class)
  std::for_each (myvec.begin(), myvec.end(),
    boost::lambda::if_then(
      boost::lambda::bind (&A::IsOK, boost::lambda::_1), 
      boost::lambda::bind (&A::DoWhatever, boost::lambda::_1, param)
    )
  );

  // second challange using boost::lambda (fnct in an external class)
  std::for_each (myvec.begin(), myvec.end(),
    boost::lambda::if_then(
      boost::lambda::bind (&B::IsOK, &b, boost::lambda::_1), 
      boost::lambda::bind (&B::DoWhatever, &b, boost::lambda::_1, param)
    )
  );

}

Вы можете упростить это, используя пространства имен ...

Answer 6

Если вы используете GCC, вы можете определить что-то вроде:

#define foreach(element, array) \
    for(typeof((array).begin()) element = (array).begin(), __end_##element = (array).end();\
        element != __end_##element;\
        ++element)

и используйте его после этого:

foreach(element, array){
    element->DoSomething(); //or (*element)->DoSomething() if type is already a pointer
}

Я использую это в пользовательском массиве, но он отлично работает и с std :: vector.