web-dev-qa-db-ja.com

C ++ 11 for()ループを使用してvector <unique_ptr <mytype >>を反復処理する

次のコードバッチがあります。

std::vector<std::unique_ptr<AVLTree_GeeksforGeeks>> AVLArray(100000);

/* Let's add some objects in the vector */
AVLTree_GeeksforGeeks *avl = new AVLTree_GeeksforGeeks();
avl->Insert[2]; avl->Insert[5]; AVL->Insert[0];
unique_ptr<AVLTree_GeeksforGeeks> unique_p(avl);
AVLArray[0] = move(unique_p);
/* we do this for a number of other trees, let's say another 9...
...
...
Now the vector has objects up until AVLTree[9] */

/* Let's try iterating through its valid, filled positions */
for(auto i : AVLTree )
{
   cout << "Hey there!\n";    //This loop should print 10 "Hey there"s.
}

Ruh roh。 for()ループの最後の部分でのコンパイルエラー。

\DataStructures2013_2014\main.cpp||In function 'int main()':|
\DataStructures2013_2014\main.cpp|158|error: use of deleted function 'std::unique_ptr<_Tp, _Dp>::unique_ptr(const std::unique_ptr<_Tp, _Dp>&) [with _Tp = AVLTree_GeeksforGeeks; _Dp = std::default_delete<AVLTree_GeeksforGeeks>; std::unique_ptr<_Tp, _Dp> = std::unique_ptr<AVLTree_GeeksforGeeks>]'|
e:\codeblocks\mingw\bin\..\lib\gcc\mingw32\4.7.1\include\c++\bits\unique_ptr.h|256|error: declared here|
||=== Build finished: 2 errors, 0 warnings (0 minutes, 0 seconds) ===|

私が間違っていることに関するアイデアはありますか?

42

ループ

_for (auto i: AVLTree) { ... }
_

AVLTree.begin()およびAVLTree.end()の範囲の各要素のコピーを作成しようとします。もちろん、_std::unique_ptr<T>_はコピーできません。各ポインターに_std::unique_ptr<T>_は1つしかありません。実際には何もコピーしませんが、むしろsteal itです。それは悪いだろう。

代わりに参照を使用する必要があります。

_for (auto& i: AVLTree) { ... }
_

...または、変更しない場合

_for (auto const& i: AVLTree) { ... }
_
75
Dietmar Kühl