C++マップベクトル参照エラー

Question

私は、次のコードからプログラムのクラッシュを取得しています：

// declaration in header: 
std::vector<Animation> mAnimations; // animation objects 
std::map<std::string, Animation*> mBoneAnimations; // map strings to animation objects from "mAnimations" 

// source code in source file: 
this->mAnimations.push_back(Animation()); 
this->mBoneAnimations[bone_name] = &this->mAnimations.back(); 
// ... 
// at different points, we modify the LAST element in animation 
this->mAnimations.back().addTranslation(Vector3(...)); 

// this is where the problem occurs later on in the program: 
Animation test = *(this->mBoneAnimations.at("Body")); // CRASH 

このコードは、最後の行にクラッシュします。私はこの問題が、このデータ構造を設定した方法によって引き起こされたことを80％確信しています。私の推測では、 "back（）"を使ってベクター内の最後のオブジェクトへの参照を取得している可能性があります。私はそれに変更しようとしました：

this->mBoneAnimations[bone_name] = &this->mAnimations[this->mAnimations.size() - 1]; 

しかし、これも役に立たなかった。問題はこのクラッシュを引き起こす原因であり、具体的にはどのように修正できますか？

編集：私はそこに "ボディ"オブジェクトがあると確信しています。このクラッシュはメモリエラーです。

Answer 1

std :: vectorに追加すると、再割り当てが必要になることがあります。それを実行すると、要素を連続したメモリに保持するために、すべての要素を新しい場所に移動します。これはすべてのポインタ、参照、イテレータを無効にするので、mBoneAnimationsはダングリングポインタを格納しています。

バック/フロントにプッシュ/ポップしている間は、代わりにstd :: dequeを使用するのが簡単です。 push_backを両端キューに置いて、空き領域がなくなったら、すべての要素を再割り当てするのではなく、新しい要素ブロックを追加して古い要素をそのまま残します。 （イテレータの有効性に関する節を参照）。プッシュフロントとポップでも同じです。