【C++】 vector<unique_ptr<int>>にpush_backしたい

Question

vector<unique_ptr<int>>に、push_backしたいのですができません。
以下のコードで、intsにiをpush_backしたいです。

#include<memory>
#include<vector>

int main()
{
    std::vector<std::unique_ptr<int>> ints;

    std::unique_ptr<int> i;
    i.reset(new int);

    // コンパイルエラーになってしまう
    ints.push_back(i);

    return 0;
}

以下は、コンパイルエラーの抜粋です。（全くわかりません・・・）

/usr/include/c++/7/ext/new_allocator.h:136:4: error: use of deleted function ‘std::unique_ptr<_Tp, _Dp>::unique_ptr(const std::unique_ptr<_Tp, _Dp>&) [with _Tp = int; _Dp = std::default_delete<int>]’
  { ::new((void *)__p) _Up(std::forward<_Args>(__args)...); }

どうしたらいいのでしょうか・・・。

Answer 1

push_back+std::moveでもいいのですがもっとシンプルに効率的にするには

#include<memory>
#include<vector>

int main()
{
    std::vector<std::unique_ptr<int>> ints;
    ints.emplace_back(std::make_unique<int>());
}

Comments

[torimingo]

ご回答ありがとうございました。
make_uniqueを使うという方法もあるのですね。
勉強させて頂きました。

[yumetodo]

むしろnewすらも書くべきではないですね

[torimingo]

なるほど、make_uniqueはunique_ptr用のnewってことですね

Answer 2

unique_ptr はコピーできんからフツーにpush_backできない。

#include <memory>
#include <vector>
#include <utility>
#include <iostream>

int main() {
    std::vector<std::unique_ptr<int>> ints;

    // do this
    for ( int i = 0; i < 10; ++i ) {
      ints.push_back(std::make_unique<int>(i));
    }

    // or this
    for ( int i = 0; i < 10; ++i ) {
      ints.emplace_back(new int(i));
    }

    // かくにん
    for ( auto& item : ints ) {
      std::cout << *item.get() << ' ';
    }
    return 0;
}

呈示された元コードを活かすなら:

#include<memory>
#include<vector>
#include <iostream>

int main() {
    std::vector<std::unique_ptr<int>> ints;

    for ( int i = 0; i < 10; ++i ) {
      std::unique_ptr<int> up;
      up.reset(new int(i));
      ints.push_back(std::move(up)); // std::moveせよ
    }
    
    // かくにん
    for ( auto& item : ints ) {
      std::cout << *item.get() << ' ';
    }
 
    return 0;
}

vector<int*> を使っても構わんけど、リークの要因になるからオススメしない。

#include<memory>
#include<vector>
#include <iostream>

int main() {
    std::vector<int*> ints;

    for ( int i = 0; i < 10; ++i ) {
      ints.push_back(new int(i));
    }
    
    // かくにん(と解放)
    for ( int* item : ints ) {
      std::cout << *item << ' ';
      // これ↓を忘れがちだからポインタの集合はお薦めしかねる
      delete item;
    }
 
    return 0;
}

Comments

[torimingo]

ご回答ありがとうございました。
move()が必要なのですね。勉強になります。