getter /によってコピーされないでアクセスされるマップの逆の反復

Question

ファイルを読み込み/解析し、キーと値のペアをマップに格納するクラスがあります。

typedef std::map<std::string, const int> Data; 
typedef std::map<std::string, const int>::reverse_iterator DataItReverse; 

は今、私は、「パーサ」クラスinline Data getData() const {return _data;}

すべて私が問題になっているマップを反復処理を逆にするまでは素晴らしいですからデータを取得するためのゲッターを持っています。反復する上記

for(DataItReverse it_reverse = _parser->getData().rbegin(); 
    it_reverse != _parser->getData().rend(); 
    ++it_reverse) 
{ 
    std::cout << it_reverse->first << std::endl; 
} 

コード通常順（逆ではない）、私は一時変数にデータをコピーする場合、すべてがうまく機能して：

Data tmpData = _parser->getData(); 
for(...) 

私は以来、データのコピーを作成することを回避したいと思いますそれは大きな地図になるでしょう。

---

その内部にデータを複数回に

Answer 1

をコピーされないよう_parserは、あなたが実際にあなたが使用した時間では存在しない一時的な反復処理を行うことにより、未定義の動作を取得しているポインタであることに注意してくださいイテレータ。

解決策として正しく識別されたとおりにコピーを作成するか、参照を返すように「ゲッター」を変更してください。

inline const Data & getData() const { return _data; } 

これは実際には標準的な方法です。データを派生していないか（クラスに格納されていない）、呼び出し後に変更されない限り、ゲッターから値で複雑な構造体を返すことは通常ありません（現在の状態を取得するためにロックを取得する必要があるマルチスレッド環境）：

次に、const-iteratorsを使用する必要があります。

for(auto it = _parser->getData().rbegin(); it != _parser->getData().rend(); ++it) 

あなたがコピーを作成するか、また、非const「ゲッター」を提供する必要があると思います非constイテレータを使用するには：私は、ループ内で混乱を保存するためにautoキーワードを使用する傾向があります。これは一般的に推奨、多分あなたの目的に合っていない。

inline Data & getData() { return _data; }