Quiero refactorizar el siguiente método y dejar un código limpio.
El método en cuestión recibe una cadena. Según qué cadena sea se ejecuta un método. Este método a ejecutar puede ser de un padre y una madre diferente al resto.
class MiClase : public QObject
{
void metodoParaRefactorizar(QString cadena)
{
QJsonDocument jDoc;
bool instructionFound = false;
if(cadena.compare("instruccionUno") == 0)
{
jDoc = this->objClassOne->metodoPublico();
instructionFound = true;
}
else if(cadena.compare("pepito") == 0)
{
jDoc = this->objeClassTwo->getList();
instructionFound = true;
}
else if(cadena.compare("cadenaMolona") == 0)
{
jDoc = this->objeClassServer->getName();
instructionFound = true;
}
else if(cadena.compare("otraCadenaDiferente") == 0)
{
jDoc = this->objeClassHtml->ejecutaScript();
instructionFound = true;
}
....
// Muchos más 'else if'
....
else if(cadena.compare("ultima") == 0)
{
jDoc = this->objeClassLast->closeAll();
instructionFound = true;
}
if(instructionFound)
{
sendAnswer(jDoc);
}
}
}
He estado buscando por internet y he visto que la solución sería el polimorfismo. Pero en este caso los métodos a ejecutar pueden ser de clases muy diferentes.
Me huele muy mal este código. Lo veo infumable y difícil de mantener. Produce mucho acoplamiento. Se que el problema es más de diseño que otra cosa. Me gustaría aprender a hacerlo mejor.
¿Qué solución le podría dar? ¿Como convierto esto en código limpio y fácil de mantener?
switch -> case
en vez deelse if