shared_ptr.reset() 삭제하지 않음

Question

나는 shared_ptr 및 weak_ptr에 대해 학습하기 시작했습니다. 이론 상으로는 모든 것이 단순 해 보입니다. 하지만 테스트를 시작했을 때, 음 ...이 간단한 프로그램이 있습니다.

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

class Second 
{ 
public: 
    Second() 
    { 
     cout << "Second created" << endl; 
    } 
    ~Second() 
    { 
     cout << "Second deleted" << endl; 
    } 
}; 

class Main 
{ 
public: 
    shared_ptr<Second> second; 
    Main() 
    { 
     cout << "Main created" << endl; 
     second = make_shared<Second>(*(new Second())); 
    } 
    ~Main() 
    { 
     second.reset(); 
     cout << "Main deleted" << endl; 
    } 
}; 

void fun() 
{ 
    shared_ptr<Main> main = make_shared<Main>(*(new Main())); 
} 

int main() 
{ 
    cout << "Program started" << endl; 
    fun(); 

    cout << "Program ended" << endl; 
    return 0; 
} 

문제는 두 번째가 삭제되지 않는다는 것입니다. 세드릭

Program started 
Main created 
Second created 
Main deleted 
Program ended 

: 여기 내 출력은? 나는 shared_ptr과 그 마지막 shared_ptr을 리셋하면 객체가 자동으로 삭제된다는 것을 알았다.

Answer 1

공유 포인터를 잘못 초기화하고 있습니다. make_shared을 사용할 때 new을 전혀 사용할 필요가 없습니다. make_shared에 전달하는 매개 변수는 만들고있는 클래스의 생성자로 전달됩니다. 다음과 같아야 코드에 따라서

, 당신의 make_shared 전화 :

shared_ptr<Main> main = make_shared<Main>(); 

하고 대신

shared_ptr<Second> main = make_shared<Second>(); 

, 당신은 새로운 사용하여 각 클래스 (버전을 유출 한 일 명시 적으로 삭제하지 않음) 누수 된 값을 두 번째 버전으로 복사합니다. 내가 말했듯이 상황이 매우 복잡하다,

Second *p = new Second; 
shared_ptr<Second> ss = make_shared<Second>(*p); // Calls the Second class copy constructor 
// p is never deleted, so it is leaked. 

Answer 2

, 그래서 스키마가 필요합니다 여러 줄로 표현을 분할함으로써, 우리는보다 쉽게 ​​잘못 한 일을 볼 수 있습니다.

class Main { 
public: 
    shared_ptr<Second> second; 
    Main() 
    { 
     cout << "Main created" << endl; 
     second = make_shared<Second>(*(new Second())); 
    } 
    ~Main() 
    { 
     second.reset(); 
     cout << "Main deleted" << endl; 
    } 
}; 

void fun() 
{ 
    shared_ptr<Main> main = make_shared<Main>(*(new Main())); 
} 

오른쪽 main의 생성 후, 우리는이 : main.second이 너무 <anon>.second에서 복사

<anon: Second> (leaked) 

<anon: Main> (leaked) 
      \ 
      |---> <Main.second: Second> 
      /
<main: Main> 

하는 것으로, 사람들은 Second의 동일한 인스턴스에 둘 점 shared_ptr<Second>이기 때문에.

<anon: Second> (leaked) 

<anon: Main> (leaked) ---> <Main.second: Second> (indirectly leaked) 

주 second.reset() 것을 반드시 지적이-하는 객체를 파괴하지 않습니다

그런 다음, main 다이, 우리는 남아 있습니다. 단순히 shared_ptr의 로컬 인스턴스를 재설정합니다. 이 개체를 가리키는 마지막 인스턴스 인 경우 개체가 삭제되지만 그렇지 않은 경우는 없습니다.