Как успешно отделить эти классы

Question

Я не был уверен в лучшем имени для этой темы, но я пытаюсь реорганизовать некоторый код и удалить некоторую развязку. Я вынужден использовать Visual Studio 2005 C++ без boost, не спрашиваю ... (Так что не C++ 11 или новее)

У меня есть протокол, который получает сообщения, а протокол содержит синтаксический анализатор и процессор , Parser извлекает информацию из сообщения и заполняет структуру. Затем структура передается в процессор для принятия дальнейших мер.

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 
    } 
} 

Я надеялся лишить процессор с Parser поэтому он стал больше, как это ...

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 

    } 
} 

Единственная проблема, которая мешает мне делать это у меня будет иметь связка операторов if в методе процесса.

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

Мой вопрос в том, как я могу избежать всех тех утверждений if, где я по существу просто разбираю его снова? Если я дам parsedData указатель на функцию или lambda, то мне все равно понадобится ссылка на процессор в синтаксическом анализаторе.

Answer 1

вы можете попробовать это: 1) в ваших проанализированных данных вместо .blah и .grah магазина vector<void*> 2) при условии создания нового объекта л (или любой другой тип сообщения) и поместить указатель на него в вектор (с использованием типа как индекс). 3) в вашем векторе хранения процессора обработчиков. перейдите в тип процессора вместе с анализируемыми данными, поэтому вы можете вызвать правильный обработчик для правильного указателя 4) в вашем листе обработчика void * to blah * (или какой-либо реальный тип)