我在PHP中编写一个类和方法,它使用switch来调用依赖于特定url的其他方法。示例如下:
switch($this->clean([$_POST['task']))
{
case "edit":
$this->editssomething();
break;
case "save":
$this->savesomethingelse();
break;
default:
$this->dodefault();
break;
}
这是否被归类为代码气味,我应该重构它还是可以归类为工厂类。
谢谢
答案 0 :(得分:2)
闻起来很好,但默认不需要"打破",也一定要用驼峰或" _"
命名你的功能switch( $this->clean($_POST['task']) )
{
case "edit":
$this->editssomething();
break;
case "save":
$this->savesomethingelse();
break;
default:
$this->dodefault();
}