ポインタのベクトルへのポインタ

Question

私は他の誰かが書いたコードを変更しています。コードのある部分では、ポインタのベクトルへのポインタを使用しています。

  class2 *pointofclass2 = new class2(); 

     class1 * pointofclass1 = new class1(.....) 

     pointofclass2->push_back(pointofclass1); 

は、その後、コード

delete pointofclass2; 

は今、私は、これはメモリリークを引き起こしていると思います。私はこれが原因であるかどうかを確かめる前にこれを遭遇したことはありませんか？どんな助けでも大歓迎です。もしそうなら、メモリを解放する正しい方法は何ですか？

Answer 1

pointofclass2を削除していますが、class1のデストラクタでclass1のポインタを保持するベクターを削除していない可能性があります。新しいクラス1（...）で作成されたオブジェクトはどこに削除されていますか？これはC++であり、自動ガベージコレクションはありません。

class2がベクトルの場合、実際にはにはが必要です。ベクトルはヒープ上にメモリを割り当てますので、ポインタを作成する必要があります。

Answer 2

push_back()で追加された要素がclass2のdtorで破棄されない限り、これはリークです。