1

私は次のタイプを持っています：

struct X { int x; X( int val ) : x(val) {} };
struct X2 { int x2; X2() : x2() {} };

typedef std::pair<X, X2>      pair_t;
typedef std::vector<pair_t>   pairs_vec_t;
typedef std::vector<X>        X_vec_t;

のインスタンスをpairs_vec_tからの値で初期化する必要がありますX_vec_t。次のコードを使用すると、期待どおりに機能します。

int main()
{
  pairs_vec_t ps;
  X_vec_t xs; // this is not empty in the production code

  ps.reserve( xs.size() );

  { // I want to change this block to one line code.
    struct get_pair {
      pair_t operator()( const X& value ) { 
        return std::make_pair( value, X2() ); }
    };
    std::transform( xs.begin(), xs.end(), back_inserter(ps), get_pair() );
  }

  return 0;
}

私がやろうとしているのは、を使用してコピーブロックを1行に減らすことですboost::bind。このコードは機能していません：

for_each( xs.begin(), xs.end(), boost::bind( &pairs_vec_t::push_back, ps, boost::bind( &std::make_pair, _1, X2() ) ) );

なぜ機能しないのかはわかりますが、余分な関数や構造体を宣言せずに機能させる方法を知りたいですか？

4

3 に答える 3

5

このようなもの？

using boost::lambda;
X2 x;
transform(..., (bind(std::make_pair<X,X2>, _1, ref(x))));

現時点では確認できませんが、メモリから正しくリコールされれば、上記は有効です。

于 2010-04-28T20:07:08.760 に答える
3 
std::for_each( xs.begin(), xs.end(),
               boost::bind( &pairs_vec_t::push_back, &ps,
// you need to pass a pointer —— at least for gcc.   ^ 
                            boost::bind( &std::make_pair<X,X2>, _1, X2() ) ) );
// you need to specify which make_pair to instantiate   ^^^^^^
于 2010-04-28T20:09:28.917 に答える
0

ブーストは必要ありません。std::bind2ndBoostからの採用を支持して非推奨になってstd::bindいますが、今のところ標準です。

pairs_vec_t ps( xs.size() );
transform( xs.begin(), xs.end(), ps.begin(),
    bind2nd( ptr_fun( make_pair<X,X2> ), X2() ) );

Boostを使用している場合は、反復または事前サイズ設定よりもinsert（の）範囲で効率的です。transform_iteratorpush_back

pairs_vec_t ps(
    make_transform_iterator( xs.begin(), bind2nd( ptr_fun( make_pair<X,X2> ), X2() ) ),
    make_transform_iterator( xs.end(), bind2nd( ptr_fun( make_pair<X,X2> ), X2() ) ) );

ワンライナーはどうですか？

于 2010-04-28T20:35:30.890 に答える