파생 된 개체에 대한 포인터 컬렉션을 관리하는 가장 좋은 방법

Question

C#에서 C++을 처음 사용했습니다. 다음 코드는 작동하지 않으며 필요한 코드가 무엇인지 잘 모르겠습니다. 그것이 작동하지 않는 이유와 적절한 변화가 무엇인지 이해할 수 있도록하는 통찰력은 인정 될 것입니다. 내가 마지막 줄을 주석 경우

C2284 "cannot convert parameter 1 from 'std::unique_ptr<_Ty> *' to 'std::unique_ptr<_Ty> *&&'

, 그것은 잘 컴파일 :

// GamePlayScreen derives from ScreenBase 
std::unique_ptr<GameplayScreen> m_game(new GameplayScreen()); 

// MenuScreen derives from ScreenBase 
std::unique_ptr<MenuScreen> m_menu(new MenuScreen()); 

// PauseScreen derives from ScreenBase 
std::unique_ptr<PauseScreen> m_pause(new PauseScreen());   

std::vector<std::unique_ptr<ScreenBase>*> screens; 

screens.push_back(&m_game); // this gets the error 

나는 오류가 컴파일 얻을.

기본적으로 나는 파생 된 항목의 모음 (또는 그 대신 포인터)을 갖고 싶습니다. 나는 '가리키고 및/또는 참조하는'다양한 방법을 사용하여 매개 변수와 다양한 방법으로 T를 설정하는 시도했습니다 vector<T>하지만 솔루션은 나를 eludes.

Answer 1

나는 많이 읽고 있어요. shared_ptr을 메모리 소유권 디자인없이 사용하면주기적인 메모리 누수가 발생합니다.

예, unique_ptr도 주기적 메모리 누출을 만들 수 있습니다. 그러나 내 경험에 의하면 unique_ptr을 사용하면 설계자가 누가 디자인을 발전 시켰는지 이해하고 주기적 메모리 누수 가능성을 낮추며 오류가 발생할 때 쉽게 디버깅 할 수 있습니다.

그리고이 설계 프로세스 중에 설계자가 실제로 공유 소유권 의미론이 필요하다는 것을 발견하면 shared_ptr (아마도 해당주기를 깨기 위해 weak_ptr)까지 도달해야합니다.

마지막으로 가능한 빨리 생 포인터를 스마트 포인터에 바인딩하십시오. **이 전송-의 소유권 고유의를 맡길 경우 **,

#include <type_traits> 
#include <utility> 
#include <memory> 
#include <vector> 

// A general purpose factory function for unique_ptrs 
// Feel free to make this factory function more specific to your domain 
template <class T, class ...Args> 
typename std::enable_if 
< 
    !std::is_array<T>::value, 
    std::unique_ptr<T> 
>::type 
make_ScreenParts(Args&& ...args) 
{ 
    return std::unique_ptr<T>(new T(std::forward<Args>(args)...)); 
} 

// Your class hierarchy 
struct ScreenBase 
{ 
    // Don't forget to make your destructors virtual 
    virtual ~ScreenBase() = default; 
}; 

// Future-proof your code with typedef's 
// If/when you need to switch smart pointer types 
// or container types, you'll thank yourself 
typedef std::unique_ptr<ScreenBase> ScreenPtr; 
typedef std::vector<ScreenPtr> ScreenContainer; 

struct GameplayScreen 
    : public ScreenBase 
{ 
}; 

struct MenuScreen 
    : public ScreenBase 
{ 
}; 

struct PauseScreen 
    : public ScreenBase 
{ 
}; 

int main() 
{ 
    // raw pointers never exposed here... 
    ScreenContainer screens; 
    screens.push_back(make_ScreenParts<GameplayScreen>()); 
} 

Answer 2

unique_ptr 대신 shared_ptr을 사용하십시오. 여러 장소에서 화면 개체를 참조 할 것이므로 사용하십시오. 다음과 같이 마지막 두 줄을 변경

Answer 3

봅니다 : 당신은 아마, unique_ptr의 unique_ptr에 대한 포인터의 아닌 벡터를 벡터를 원했기 때문에

std::vector<std::unique_ptr<ScreenBase>> screens; 

screens.push_back(std::move(m_game)); 

나는 별표를 제거했습니다. 또한 이름이 의미하는대로 unique_ptr은 한 번에 하나의 unique_ptr 소유권을 가질 수 있음을 의미하므로 소유권을 이전하려면 명시 적으로 std::move해야합니다.

Answer 4

특정 유형의 다양한 화면을 관리하려는 경우처럼 기본 유형으로 화면의 다른 목록을 가지고 있기 때문에 모든 화면에서 특정 작업을 수행 할 필요가없는 것처럼 들리는 것 같습니다 특정 유형의 화면에 대한 세부 정보 당신은 당신이 화면을 파괴하는 경우에 매달려 화면의 포인터를 남기지 않도록 조심해야

// GamePlayScreen derives from ScreenBase 
std::unique_ptr<GameplayScreen> m_game(new GameplayScreen()); 

// MenuScreen derives from ScreenBase 
std::unique_ptr<MenuScreen> m_menu(new MenuScreen()); 

// PauseScreen derives from ScreenBase 
std::unique_ptr<PauseScreen> m_pause(new PauseScreen());   

std::vector<ScreenBase*> screens; 

screens.push_back(&*m_game); 

:

나는이 경우에 당신은 단지 원시 포인터를 원하는 생각합니다.

Answer 5

어떤 컴파일러는 여기에서 확인할 수 있습니다,하지만 난 당신이 할 수있는 확신 :

// GamePlayScreen derives from ScreenBase 
std::shared_ptr<GameplayScreen> m_game(new GameplayScreen()); 

// MenuScreen derives from ScreenBase 
std::shared_ptr<MenuScreen> m_menu(new MenuScreen()); 

// PauseScreen derives from ScreenBase 
std::shared_ptr<PauseScreen> m_pause(new PauseScreen());         

std::vector<std::shared_ptr<ScreenBase>> screens; 

screens.push_back(m_game); 

@처럼 환호 앤 HTH-ALF 당신은 아마 공유 포인터를 원했다. 나는이 때 확실 unique_ptr에 기본값으로 사람들을 격려 할

...you probably want shared_ptrs...

: