MVC验证建议

Question

我目前正在验证输入并在"胖控制器"中返回错误,如下所示:

class SomeController
{
    public function register()
    {
        // validate input
        $username = isset($_POST['username']) && strlen($_POST['username']) <= 20 ? $_POST['username'] : null;

        // proceed if validation passed
        if (isset($username)) {
            $user = $this->model->build('user');
            if ($user->insert($username)) {
                $_SESSION['success'] = 'User created!';
            } else {
                $_SESSION['error'] = 'Could not register user.';
            }
        } else {
            $_SESSION['failed']['username'] = 'Your username cannot be greater than 20 characters.';
        }

        // load appropriate view here
    }
}

class SomeModel
{
    public function insert($username)
    {
        // sql for insertion
        // ...

        return $result;
    }
}

虽然这很有效并且我很容易实现,但我知道这是不正确的,因为验证属于模型,我试图使用"胖模型"进行更正,如下所示:

class SomeController
{
    public function register()
    {
        $user = $this->model->build('user');
        $user->insert($_POST['username']);

        // load appropriate view here
        // ...
    }
}

class SomeModel
{
    public function insert($username)
    {
        // validate input
        $error = false;
        $username = trim($username) != '' && strlen($username) <= 20 ? $username : null;

        // proceed if validation passed
        if (isset($username)) {
            // sql for insertion
            // ...

            $_SESSION['success'] = 'User created!';
        } else {
            // store error in session
            $error = true;
            $_SESSION['error']['username'] = 'Your username cannot be greater than 20 characters ';
        }

        return $error ? false : true;
    }
}

我在这里看到的问题是该模型应该是可移植的,因为它永远不需要改变.但如果要求$username变化的长度,那么显然我将不得不改变我的模型.

我觉得这可能是一个非常常见的问题,但我还没有找到一个直截了当的答案.如果没有实现任何额外的"层","映射器"或其他任何令人困惑的术语,那么如何修改提供的示例伪代码以正确处理此事务？(例如,验证输入,如果验证失败则返回错误)？

Answer 1

没有实现任何额外的"层","映射器"或其他任何东西

您应该将"模型"视为应用程序层而不是单个类.术语"层"可以被认为是参考M切片MVC三明治的简单方式.因此,要实现你的愿望,你的灵活性,将需要创建它.

可以进行一些明确的分离.我会考虑有三个抽象:服务,数据映射器和实体.

服务将暴露给控制器并执行所请求的服务.

// some controller
function register() {
  $service = $this->getUserService();
  $user = $service->register($_POST['first_name'], $_POST['last_name']);
  if ($user instanceof \My\Entity\User) {
    // set user in view
  } else {
    // redirect to error
  }
}

所以任务一完成,控制器现在对寄存器内发生的事情都是愚蠢的,它只想知道如何解决相应的结果.如果有用户对象,则成功,否则会出错.

服务类本身将封装所提供的服务:

// class UserService.php
function register($firstname, $lastname) {
  // validate arguments
  if ($this->isValidUsername(....
    $userMapper = $this->getUserMapper();
    $user = new My\Entity\User();
    $user->setFirstName($firstname);
    $user->setLastName($lastname);
    return $userMapper->save($user);
  }
  return false;
}

我们处理参数的验证并创建新用户,将其传递给数据映射器,数据映射器将执行抽象数据库操作的"实际保存".

// UserMapper
function save($user) {
  // save $user to db
  $sql = 'INSERT INTO ....

 return true;
}