关于"circular reference"，我用了weak_ptr，但还是发生了内存泄漏

Question

我阅读了：

How to avoid memory leak with shared_ptr?

我知道我需要使用 weak_ptr 来避免 循环引用 。

所以我创建了一个小程序来播放循环引用。

将调用以下对象(spyder)

class spyder {
 public:
  spyder(std::string _name): 
    m_name(_name), finger(nullptr)
  {  }
  inline const std::string ask_name() const{
    return m_name;
  }
  std::shared_ptr<spyder> finger;
 private:
  std::string m_name;
};

我在主代码中使用 shared_ptr 和 weak_ptr:

调用 spyder

int main(){
  auto sA = std::make_shared<spyder>("A");
  auto sB = std::make_shared<spyder>("B");
  std::weak_ptr<spyder> wp_sA(sA);
  std::weak_ptr<spyder> wp_sB(sB);  
  sA->finger = wp_sB.lock();
  sB->finger = wp_sA.lock();
}

以上代码发生了内存泄漏（使用valgrind检查）。

==20753== Memcheck, a memory error detector
==20753== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==20753== Using Valgrind-3.14.0 and LibVEX; rerun with -h for copyright info
==20753== Command: ./t
==20753== 
==20753== 
==20753== HEAP SUMMARY:
==20753==     in use at exit: 128 bytes in 2 blocks
==20753==   total heap usage: 3 allocs, 1 frees, 72,832 bytes allocated
==20753== 
==20753== LEAK SUMMARY:
==20753==    definitely lost: 64 bytes in 1 blocks
==20753==    indirectly lost: 64 bytes in 1 blocks
==20753==      possibly lost: 0 bytes in 0 blocks
==20753==    still reachable: 0 bytes in 0 blocks
==20753==         suppressed: 0 bytes in 0 blocks
==20753== Rerun with --leak-check=full to see details of leaked memory
==20753== 
==20753== For counts of detected and suppressed errors, rerun with: -v
==20753== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

但是我将上面的代码修改为：

int main(){
  spyder sA("A"), sB("B");
  std::weak_ptr<spyder> wp_sA( std::make_shared<spyder>("A") ) ;
  std::weak_ptr<spyder> wp_sB( std::make_shared<spyder>("B") );
  sA.finger = wp_sB.lock();
  sB.finger = wp_sA.lock();
}

没有发生内存泄漏，

==20695== Memcheck, a memory error detector
==20695== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==20695== Using Valgrind-3.14.0 and LibVEX; rerun with -h for copyright info
==20695== Command: ./t
==20695== 
==20695== 
==20695== HEAP SUMMARY:
==20695==     in use at exit: 0 bytes in 0 blocks
==20695==   total heap usage: 3 allocs, 3 frees, 72,832 bytes allocated
==20695== 
==20695== All heap blocks were freed -- no leaks are possible
==20695== 
==20695== For counts of detected and suppressed errors, rerun with: -v
==20695== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

我对此感到困惑。

Answer 1

你根本没有解决循环依赖

在您的第一个版本中，

sA 由 shared_ptr 管理，它存储 shared_ptr 到 sB。 sB又被shared_ptr管理，存储一个shared_ptr到sA。这意味着它们的引用计数永远不会变为 0。

相反，finger 应该是 std::weak_ptr 类型，您只需要 lock() 就可以使用它。

class spyder {
 public:
  spyder(std::string _name): 
    m_name(_name), finger()
  {  }
  inline const std::string ask_name() const{
    return m_name;
  }
  std::weak_ptr<spyder> finger;
 private:
  std::string m_name;
};

int main(){
  auto sA = std::make_shared<spyder>("A");
  auto sB = std::make_shared<spyder>("B");
  sA->finger = sB;
  sB->finger = sA;
}

你的第二个版本

从临时 shared_ptr 构造 weak_ptr，这意味着在构造后您的 weak_ptr 立即变为 expired()。 sA.finger 和 sB.finger 都存储 nullptr 在此版本中（但没有内存泄漏）。

Answer 2

您的代码泄漏是因为它仍然有一个循环引用。 sA -> sA::finger -> sB -> sB::finger -> sAstd::weak_ptr的要点是存储对指针的引用。为避免内存泄漏，您的 class 源代码应如下所示：

class spyder {
 public:
  spyder(std::string _name): 
    m_name(_name), finger(nullptr)
  {  }
  inline const std::string ask_name() const{
    return m_name;
  }
  std::weak_ptr<spyder> finger;
 private:
  std::string m_name;
};

关于"circular reference"，我用了weak_ptr，但还是发生了内存泄漏

About "circular reference", I used weak_ptr but memory leak still happened

c++

smart-pointers

shared-ptr

circular-reference

weak-ptr

在您的第一个版本中，

你的第二个版本