C++状態パターンで宣言する場所を知る方法

Question

この質問はthis質問から来ます。 私はコンテナ（ゲーム）にshared_ptrを持つ状態パターンを実装しようとしています。

しかし、私は円形の包含に問題があり、宣言を転送する必要があります。

マイコード：

Game.h

#pragma once 
#include <memory> 

#include "BaseState.h" 
class Game : public std::enable_shared_from_this<Game> 
{ 
private: 
    std::shared_ptr<BaseState> currentState; 
public: 
    Game(); 
    void switchState(std::shared_ptr<BaseState> nextState); 
    void doSomething(char); 
    void runState(); 
}; 

CPP

#include "stdafx.h" 
#include <iostream> 

#include "Game.h" 
#include "SomeState.h" 

Game::Game() 
{ 
    currentState = std::make_shared<SomeState>(); 
} 

void Game::switchState(std::shared_ptr<BaseState> nextState) 
{ 
    currentState = nextState; 
} 

void Game::doSomething(char c) 
{ 
    std::cout << "Game : " << c; 
} 

void Game::runState() 
{ 
    currentState->handleCommand(shared_from_this()); 
} 

BaseState.h

#pragma once 
#include <memory> 

#include "Game.h" 

class BaseState 
{ 
public: 
    virtual void handleCommand(std::shared_ptr<Game>) = 0; 
}; 

SomeState.h

#pragma once 
#include "BaseState.h" 
class SomeState : 
    public BaseState 
{ 
public: 

    // Inherited via BaseState 
    virtual void handleCommand(std::shared_ptr<Game>) override; 
}; 

cppの前方宣言が、まだそれを得ることはありませんについて

#include "stdafx.h" 
#include "SomeState.h" 

void SomeState::handleCommand(std::shared_ptr<Game> game) 
{ 
    game->doSomething('S'); 
} 

I read他の質問。

私は何を試しましたか。

forward宣言BaseStateゲームでは、コードはコンパイルされますが、エラーはスローされます。 ConsoleApplication1.exeで0x73E9DAE8で

未処理の例外： のMicrosoft C++の例外：メモリ位置 0x00BBF5D4でのstd :: bad_weak_ptr。

はフォワードBaseStateにゲームを宣言します。 Dosntコンパイルも、未定義の型エラーの使用を与える

「doSomethingのは、」：doSomethingのためにコンパイル時の試合でロジックです 「のstd :: shared_ptrの」

のメンバーていないではありません関数のように宣言されているため;私はどこ別のクラスを宣言転送先を決めるん

class Game; 

はどのように、どの論理的なステップがありますか、私は1つだけ選んで作成チョイスの問題を修正する必要がありますか？あなたはBaseState.hに#include <Game.h>する必要はありません

Answer 1

、あなたは単にBaseState宣言はGameの内容を知っている必要はありませんので、これが機能すること

class Game; 

、前方宣言することができます。最初に試したことはOKです。同じことがGame.hの#include <BaseState.h>にも当てはまります。それを前方宣言BaseStateに置き換えてください。

std::bad_weak_ptr例外は何か他のものが原因でした。具体的には、おそらく

それだけのstd :: shared_ptrの管理するオブジェクトの以前の共有 オブジェクト、つまり上shared_from_thisを呼び出すことが許可されていると言うshared_from_thisについて少し詳細を逃しています。そうでなければ 挙動は

と

のstd :: bad_weak_ptr（デフォルト-構築weak_thisから shared_ptrのコンストラクタで）スローされます（C++ 17から）

を定義されていません

通常、オブジェクトをshared_ptrにインスタンス化することでこれを解決できます。

int main() { 
    auto myGame = std::make_shared<Game>(); 
    . . . 
    myGame->runState(); 
    . . . 
} 

EDIT

shared_ptrがそれを使用して関連付けられた特定のコストを持っていること、しかし心に留めておいてください。一般的には、指さしオブジェクトが使用されている関数呼び出しよりも先に指定されていることがわかっている場合は、BaseState::handleCommandの場合のように、参照で渡すほうが高速（かつ安全です）かもしれません。