用php切换代码的​​味道与否

时间:2017-08-29 14:38:09

标签: php switch-statement

我在PHP中编写一个类和方法,它使用switch来调用依赖于特定url的其他方法。示例如下:

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

这是否被归类为代码气味,我应该重构它还是可以归类为工厂类。

谢谢

1 个答案:

答案 0 :(得分:2)

闻起来很好,但默认不需要"打破",也一定要用驼峰或" _"

命名你的功能
switch( $this->clean($_POST['task']) )
{
    case "edit":
        $this->editssomething();
    break;

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

    default:
        $this->dodefault();
}