php Schalter Code Geruch verwenden oder nicht

Question

ich eine Klasse und Methode in PHP schreibe, die Switch verwendet andere Methoden, die auf die spezifische URL abhängig zu nennen. Beispiel unten:

switch($this->clean([$_POST['task'])) 
{ 
    case "edit": 
     $this->editssomething(); 
    break; 
    case "save": 
     $this->savesomethingelse(); 
    break; 
    default: 
     $this->dodefault(); 
    break; 
} 

Ist das klassiert als Codegeruch, und soll ich es sein Refactoring oder könnte es als eine Factory-Klasse eingestuft werden.

Danke

Answer 1

Das riecht gut, aber standardmäßig nicht braucht „break“, auch sollten Sie Ihre Funktion mit Kamel nennen oder „_“

switch($this->clean($_POST['task'])) 
{ 
    case "edit": 
     $this->editssomething(); 
    break; 

    case "save": 
     $this->savesomethingelse(); 
    break; 

    default: 
     $this->dodefault(); 
}