ベクトルポインタvalgrindメモリリークを引き起こす

Question

私はC++とvalgrindを学ぼうとしています。だから私はそれをテストするために次のコードを書いた。しかし、私はいくつかのメモリリークを取得します。誰でもメモリリークの原因を説明できますか？前もって感謝します。 valgrindの

==28548== HEAP SUMMARY: 
==28548==  in use at exit: 4 bytes in 1 blocks 
==28548== total heap usage: 3 allocs, 2 frees, 16 bytes allocated 
==28548== 
==28548== 4 bytes in 1 blocks are definitely lost in loss record 1 of 1 
==28548== at 0x4C27CC1: operator new(unsigned long) (vg_replace_malloc.c:261) 
==28548== by 0x400C36: main (in a.out) 
==28548== 
==28548== LEAK SUMMARY: 
==28548== definitely lost: 4 bytes in 1 blocks 
==28548== indirectly lost: 0 bytes in 0 blocks 
==28548==  possibly lost: 0 bytes in 0 blocks 
==28548== still reachable: 0 bytes in 0 blocks 
==28548==   suppressed: 0 bytes in 0 blocks 
==28548== 
==28548== For counts of detected and suppressed errors, rerun with: -v 
==28548== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 4 from 4) 

から

#include <vector> 
#include <iostream> 
using namespace std; 

class test 
{ 
    int c; 
     public: 
     void whatever(); 
}; 
void test:: whatever() 
{ 
    vector<test*> a; 
    if(true) 
    { 
      test* b = new test(); 
      b->c = 1; 
      a.push_back(b); 
    } 
    test* d = a.back(); 
    cout << "prints: " << d->c; 
    delete d; 
} 

int main() 
{ 
    test* a = new test(); 
    a->whatever(); 
    return 1; 
} 

は、私は、ポインタのコピーから削除することは許されないか、私は間違って何かをやっているのですか？

Answer 1

aでdeleteに電話することは決してありません。

ここで重要なのは、ポインタのvectorを使用していることです。なぜあなたはこれをやるのですか？ベクトルがあなたのためにメモリ管理をしてくれます！

Answer 2

delete a;の最後にmain()があります。

あなたが書いたものはすべてにならないようにしてください。は実際のコードになります。ダイナミックアロケーション（new）を絶対に使用する必要はありません。そして、それをここに書くのは良い方法ですが、あなたは教育目的のためにポインタのベクトルを維持したいと仮定すると、

---

：

#include <vector> 
#include <memory> // for unique_ptr 

// intentionally left blank; NO abusing namespace std! 

struct Foo 
{ 
    int c; 

    void whatever() 
    { 
     std::vector<std::unique_ptr<test>> v; 

     if (true) 
     { 
      v.emplace_back(new test); 
      v.back()->c = 1; 
     } 

     // everything is cleaned up automagically 
    } 
}; 

int main() 
{ 
    Test a;  // automatic, not dynamic 
    a.whatever(); 

    return 1; 
} 

これは単に教育目的のために、まだです。実際の生活では、vectorがであり、すでにの動的データ構造であり、余分な間接レベルの必要性はほとんどないので、平野で行うことは非常に難しいです。std::vector<test>

Answer 3

メモリリークは主に発生しています。割り当てられたtestオブジェクトは削除しません。