12

通常のforループの代わりにfor_eachを使用しようとしています。しかし、私はC ++ 11に慣れていないので、ちょっと行き詰まっています。ここでの私の意図は、for_each式とlambda式を一緒に使用することです。何か案は ？Visual Studio2010を使用しています。
よろしくお願いします
。Atul

これがコードです。 

#include "stdafx.h"  
#include <algorithm>  
#include <memory>  
#include <vector>  
#include <iostream>

using namespace std;  

struct Point  
{
    union 
    {
        double pt[3];
        struct {double X,Y,Z;};
        struct {double x,y,z;};
    };

    Point(double x_,double y_,double z_)
      :x(x_),y(y_),z(z_)
    {}
    Point()
      :x(0.0),y(0.0),z(0.0)
    {}

    void operator()()
    {
        cout << "X coordinate = " << x << endl;
        cout << "Y coordinate = " << y << endl;
        cout << "Z coordinate = " << z << endl;
    }
};  

int _tmain(int argc, _TCHAR* argv[])  
{  
    std::vector<Point> PtList(100);

    //! the normal for loop
    for(int i = 0; i < 100; i++)
    {
        // Works well  
        PtList[i]();
    }

    //! for_each using lambda, not working
    int i = 0;
    for_each(PtList.begin(),PtList.end(),[&](int i)
    {
        // Should call the () operator as shown previously
        PtList[i]();
    });

    //! for_each using lambda, not working
    Point CurPt;
    for_each(PtList.begin(),PtList.end(),[&CurPt](int i)
    {
        cout << "I = " << i << endl;
        // should call the() operator of Point
        CurPt();
    });

    return 0;
}
4

2 に答える 2

17

の3番目のパラメータは、各インデックスではなく、各要素for_eachに適用する関数です。そうでなければ、従来のループでそれを使用する意味は何でしょうか？

したがって、パラメーターの代わりに、intパラメーターを取りPointます。PtListそして今、への参照は不要なので、何かをキャプチャする理由はありません。

// Should make operator() const as it doesn't modify anything
for_each(PtList.begin(),PtList.end(),[](Point const& p)
{
    p();
});
于 2012-07-16T10:38:46.410 に答える
4

あなたstd::for_eachは明らかに間違っています。ランバへの引数のタイプはPoint、またはPoint const&あなたが何をしたいか、そしてあなたが何をすることが許されているかに依存するべきです。

これである必要があります：

int count = 0;
for_each(PtList.begin(),PtList.end(), [&](Point const & p)
{
      cout <<"No. " << ++count << endl;
      p();
});

constoperator()メンバー関数を作成します。

于 2012-07-16T10:39:24.233 に答える