これらのクラスを正常にデカップリングする方法

Question

このスレッドの最適な名前はわかりませんでしたが、いくつかのコードをリファクタリングしてデカップリングを削除しようとしています。私は、Visual Studio 2005 C++を使用せずに使用することを余儀なくされました。質問しません。（C++ 11以降ではありません）

メッセージを受信するプロトコルがあり、プロトコルにはパーサとプロセッサが含まれています。パーサーは、メッセージから情報を抽出し、構造体に値を設定します。次に、構造体はプロセッサーに渡され、それ以上の処理が行われます。

私はこれをやってから私を妨げている唯一の問題は、私が持っている必要がありますです

class Protocol 
{ 
    Parser parser; 
    Processor processor; 

public: 
    Protocol() : parser(processor) 
    { 

    } 

    handleMessage(Message& message) 
    { 
     ParsedData parsedData; 
     parser.parse(message, parsedData); //This will not call the processor from within it 
     processor.process(parsedData); //This is the new 

    } 
} 

...それはもっとこのようになったので、パーサからプロセッサを除去するために期待していた

class Protocol 
{ 
    Parser parser; 
    Processor processor; 

public: 
    Protocol() : parser(processor) 
    { 

    } 

    handleMessage(Message& message) 
    { 
     ParsedData parsedData; 
     parser.parse(message, parsedData); 
    } 
} 

class Parser 
{ 
    Processor processor; 

public: 
    Parser() 
    { 

    } 

    Parser(Processor& p) : processor(p) 
    { 
    } 

    parse(Message& message, ParsedData& parsedData) 
    { 
     if(message.type == "whatever") 
     { 
      parseLevel2(message.message, parsedData); 
     } 
     //else if Other message types 
    } 

    parseLevel2(MessageLevel2& message, ParsedData& parsedData) 
    { 
     //Keep going down the rabbit hole, but for simplicity it ends here 
     parsedData.blah = "some data"; 
     processor.ProcessBlahMessage(parsedData); 
    } 
} 

class Processor 
{ 
public: 
    Processor() 
    { 
    } 

    ProcessBlahMessage(ParsedData& parsedData) 
    { 
     //Do logic 
    } 
} 

プロセスメソッドのif文の束。

process(ParsedData& parsedData) 
{ 
    if(parsedData.type == "blah") 
    { 
     ProcessBlahMessage() 
    } 
    else if(parsedData.type == "grah") 
    { 
     ProcessGrahMessage() 
    } 
    //etc... 
} 

私の質問は、私は基本的にちょうどそれを再度解析していますこれらすべてのif文を避けるどうやっているのですか？私が関数ポインタ、またはラムダをparsedDataに渡すと、パーサのプロセッサへの参照が必要になります。

Answer 1

あなたはこれを試すことができます： 1）の代わりに.blahと.grah店のあなたの解析されたデータにどのようなメッセージタイプの新しい何とかオブジェクト（または作成）とタイプを使用して（ベクターに、それへのポインタを置くと仮定しvector<void*> 2）インデックスとして）。 3）プロセッサストアのハンドラのベクトル。あなたは正しいポインタのために正しいハンドラを呼び出すことができます 4）あなたのハンドラの中でvoid *をblah *に（または実際の型は何でも）