vector<unique_ptr>の初期化方法

Question

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

using namespace std;

class X{};

int main()
{
	//vector<unique_ptr<X>> vec(100, nullptr); // コピーコンストラクタが呼ばれる

	vector<unique_ptr<X>> vec(100);	// こう書くしかない?
	for (size_t i = 0; i < vec.size(); i++) vec.at(i) = nullptr;
}

以上のように記述するとvector初期化時にコピーコンストラクタが呼ばれてしまいます。

より良い初期化方法がございましたら教えていただきたいです。ループさせるのをやめて短く書きたいです。

Answer 1

vector<unique_ptr<X>> vec(100);

この時点で、unique_ptrのデフォルトコンストラクタが呼び出されているので、
無理にnullptrを入れての初期化は必要ないと思います

Comments

[__ook]

デフォルトコンストラクタだとなにが入っているのかわからない認識なのですが正しいでしょうか。
nullptrかどうかで処理を分けようと考えていたのです…

[__ook]

あー…
getでnullptrかどうかチェックすればよいのですね…ありがとうございました。

[SaitoAtsushi]

std::unique_ptr には operator bool が定義されています。
条件式として使う文脈であればメンバ関数 get で陽にポインタを取り出す必要はありません。

#include <memory>
#include <iostream>

int main(void) {
std::unique_ptr<int> hoge;
if(not hoge) std::cout << "hoge is null" << std::endl;
return 0;
}

一旦スマートポインタに入れたものは不用意に取り出さずにスマートポインタのままで取り廻す方が良いです。
(取り出さざるを得ない場面はもちろんあるので程度問題ですが。)
取り出した生ポインタをうっかり再びスマートポインタに入れるようなことをすると複数個所で所有権を持つことになって破綻してしまいます。 間違いの元です。

[__ook]

！
ありがとうございます。
かなり処理がちゃんとかけた気がします。ありがとうございました！

[yohhoy]

> デフォルトコンストラクタだとなにが入っているのかわからない認識
FYI: std::unique_ptrのデフォルトコンストラクタは、スマートポインタがnullptrで初期化されることを保証しますよ。

https://cpprefjp.github.io/reference/memory/unique_ptr/op_constructor.html
https://ja.cppreference.com/w/cpp/memory/unique_ptr/unique_ptr
C++バージョンが上がるごとに改定されており読みづらいのは否めませんが、「値初期化したポインタ」や「何も所有しない」という表現がnullptrで初期化されることを意味しています。

[__ook]

質問した時点で確認不足でした。ありがとうございます、ちゃんと読むようにします。