en utilisant l'odeur de code de commutateur PHP ou pas

Question

J'écris une classe et une méthode en PHP qui utilise switch pour appeler d'autres méthodes dépendantes de l'URL spécifique. Exemple ci-dessous:

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

Est-ce Classée odeur de code, et devrais-je être refactoring ou pourrait-il être classé comme une classe d'usine.

Merci

Answer 1

qui sent bon, mais par défaut n'a pas besoin de « pause », aussi être sûr de nommer votre fonction avec chameau ou « _ »

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

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

    default: 
     $this->dodefault(); 
}