C++ STL：ポインタ付きリスト - Iteratorがアクセスできない？

Question

私は "オブジェクト"オブジェクトのポインタを保持するSTLリストで苦労しています。

Iが宣言：

this->objectlist.push_back(new Object(address,value,profit)); 

や地図などのように反復することを試みた：介し

list<Object*> objectlist; 

挿入を

プリント（）はパブリックメソッドである

list<Object*>::iterator iter; 

iter = this->objectlist.begin(); 

while(iter != this->objectlist.end()) 
{ 
    iter->print(); 
} 

クラスObjectの。

ここで何が間違っていますか？

イテレータからリストのオブジェクトにアクセスできません。

Answer 1

あなたは、矢印デリファレンスあなたはポインタにイテレータを持っているので、あなたが最初のデリファレンスに（あなたObject*を取得）イテレータを持ってObject *を(*iter)->print();

を必要とし、印刷の呼び出しを可能に。

(*iter)->print() 

Answer 2

は、次の方法を使用する必要があります！あなたのwhileループはそうのようなforループに変更します。

for (list<Object*>::const_iterator iter = this->objectlist.begin(), 
    end = this->objectlist.end(); 
    iter != end; 
    ++iter) 
{ 
    (*iter)->print(); 
} 

（またiter逆参照ポインタに、他の回答が指摘しているよう。）

Answer 3

あなたのイテレータをインクリメントされていません。

Answer 4

あなたがして、イテレータが指す値にアクセスすることができます* iter

また、各反復でイテレータをインクリメントすることを忘れないでください。さもなければ、あなたは無限ループで立ち往生します。このよう

：タイプTのリストについては、イテレータを逆参照するタイプTの価値を生み出すよう

iter = this->objectlist.begin(); 

while(iter != this->objectlist.end()) 
{ 
    (*iter)->print(); 
    iter++; 
} 

Answer 5

イテレータは、リスト内の要素を参照します。

あなたの場合、Tは実際にはポインタObject*です。したがって、iter->呼び出し（イテレータを逆参照する）によってポインタが生成されます。実際のオブジェクトに到達するには、そのポインタを逆参照する必要があります。

ではなく

(*iter)->print() 

を使用してみてください。

Answer 6

（可能であれば）C++ 11の方法を試すことができます。イテレータの間接参照は自由来る:)

for (auto& iter: this->objectlist) 
{ 
    iter->print(); 
} 

また、印刷は（）あなたが代わりにあなたがSTLのfor_each()を使用する必要があります - ループforの

for (const auto& iter: this->objectlist) 

Answer 7

使用する必要があるのconst関数の場合 - advantages of for_eachを参照します単純な上にfor -loop。

構文

#include <algorithm> // 'for_each()' is in there 

std::for_each(InputIterator first, // Iterator start position 
       InputIterator last, // Iterator end position 
       Function fn);  // Unary function - executed on all elements in range 

for_each例ここで

はあなたの例は、std::for_each()代わりのfor(...)どのように見えるかです：単項関数を使用して実装されて

list<Object*> objectlist; 

// insert some elements … 

// Deletes all elements of 'objectlist' 
std::for_each(objectlist.begin(), objectlist.end(), DeleteObj<Object*>()); 

単項機能

テンプレートなので、と入力すればとなります。 Tは、ポインタタイプです。

template<class T> class DeleteObj 
{ 
public: 
    bool operator()(T obj) const 
    { 
     delete obj; 
     return true; 
    } 
}; 

btw。 にはこれをテンプレートとして実装するためのがありません。そして：そのような実装の代わりにメンバー関数をバインドすることもできます！

ドキュメント

• http://www.cplusplus.com/reference/algorithm/for_each/
• http://www.sgi.com/tech/stl/for_each.html
• http://en.cppreference.com/w/cpp/algorithm/for_each
• Should I use std::for_each?