MVC服务层内的限制

Question

我最近研究了OOP和PHP MVC应用程序设计。 目前，我学到了很多东西，但目前有件事困扰着我。

我阅读并现在了解了为什么将HTTP重定向放在服务层中是不明智的。 我们不知道一旦服务完成，控制器将需要做什么，等等。我还读到，该服务不应超出其用途进行任何操作。 示例：用户注册只能使用控制器传递的输入来创建新用户，但是我想知道在服务层中设置Flash消息是否还可以。

我的应用程序session based notifications for users显示了很多基于Flash session based notifications for users 。 它们全部基于与服务相关的输入验证检查，并产生类似于以下内容的警报

用户名xxxxxx已被使用

用户名应大于5个字符

应该/可以在服务类中定义/设置它，还是有什么问题？ 我有一个Alert助手功能，可以处理警报设置。 我可以轻松地使用依赖项注入器使其可用，我只是想知道这样做是否存在问题。

我在服务中实现所有重定向时犯了一个错误，而我刚刚删除了所有重定向并将它们放置在控制器中，所以我不想犯同样的耗时错误，因此我在这里寻求建议。

预先感谢您的帮助。

编辑-代码示例

<?php

/**
 *-----------------------------------------------------------------
 * 
 *  LOGIN CONTROLLER
 * 
 */
namespace Controller\www;
use \Helper\Controller;

class Login extends Controller {
    public $dependencies = ['arena', 'login', 'site'];

    /**
     *  Login
     * 
     *  Login Runs Through Various Checks Including If User is Banned, Account is Locked, 
     *  or Forgot Password Request Is Active. Then the Entered Password is Matched & if Valid
     *  User is Logged In
     */
    public function index() {

        // Define Default
        $username = '';                                                                     


        /** 
         *  User Login
         * 
         *  If      Successful, Login User, Redirect Home
         *  Else    Set Error Alerts 
         */
        if ($this->form->post('login')) { 

            // Define and Sanitize Post Data
            $username = $this->input->get('username');
            $password = $this->input->get('password');


            // Login Service Layer
            $login = $this->factory->make('user/login');

            // If Successful Redirect Home - Else Set Errors
            if ($login->user($username, $password) === true) {
                $this->redirect->home();
            }
            $this->alert->error($login->get('errors'));
        }


        /**
         *  Define Site Title & Display Page
         */
        $this->view->sitetitle('login');
        $this->view->display('www/login', [
            'video'     => $this->arena->video(),
            'username'  => $this->input->set($username)
        ], ['notifications' => 'user/forgotpassword']);
    }
}

服务层

/**
 *-----------------------------------------------------------------
 * 
 *  USER LOGIN SERVICE LAYER
 * 
 */
namespace Service\User;
use \Helper\Service;

class Login extends Service {
    public $dependencies = ['login', 'mail', 'time', 'user', 'vars'];

    /**
     *  Handles Entire Login Process For Site Users
     *  
     *  @params all         User Submitted Form Data
     */
    public function user($username = '', $password = '') {

        // Validate $_POST Form Data
        $this->validateInput($username, $password);


        /**
         *  No Errors Produced - Complete Form Submission
         * 
         *  We Are Not Using `elseif` Between Forgot Password & Normal Login
         *  After a Forgot Password Code is Generated User May Remember Old Passwords
         *  We Need to Ensure Users Can Still Login Using Account Password As Well
         */
        if (!$this->errors()) {


            /** 
             *  User Input Password Matches Account Password
             */
            if ($this->input->verifyhash($password, $this->user->get('info.password'))) {
                $this->login->user();
                return true;                                                                    
            }


            /** 
             *  If We Have Not Been Redirected Login Was Unsuccessful
             */
            $message = $forgotPW ? 'Forgot Password Code Invalid - Login Lost Incorrect' : 'Login Unsuccessful - Incorrect Username or Password';
            $this->log->error($message, ['Username' => $username, 'Password' => $password]);

            $this->error('Incorrect Username or Password');
        }

        /** 
         *  If We Have Made It This Far Login Was Unsuccessful - Log Unsuccessful Attempt
         */
        $this->login->logAttempt();


        return false;
    }


    /**
     *  Validate $_POST Data
     * 
     *  @params all         User Submitted Form Data
     */
    private function validateInput($username = '', $password = '') {

        // Display Error if Username is Empty 
        if (!$username) {                                                                                           
            $this->error('Please enter a username');                                        
        }
        // Display Error if Password is Empty
        elseif (!$password) {                                                                                       
            $this->error('Please enter a password');                                        
        } 
        // Search DB For User With Matching Username - If User Not Found Display/Log Error, Else Set User
        else {
            $user = $this->user->info($username, 'username', '', '`userid`');
            if (!$user) {
                $this->error('The username ' . $username . ' does not exist'); 
                $this->log->error('User Not Found When Attempting to Login', ['username' => $username]);   
            } else {
                $this->user->set('user', $user['userid']);
            } 
        }
    }
}

Answer 1

为了回答您的问题，我认为最好将MVC的概念分解为非常基本的形式及其各个部分。 如果事态有些屈尊，我事先表示歉意。

视图
应用程序的视图显示所有内容。 如果要显示某些内容，则应在此层中完成

控制者
控制器是视图和模型之间的中介。 它从视图中获取输入，对其应用逻辑/规则（在需要时），并与模型进行交互，然后获取数据以传递回视图。

模型
在此完成数据的加载和保存。 大部分验证应已作为控制器规则的一部分完成，并且仅应在加载或保存时将所有错误的详细信息传递回控制器。 如果没有错误，则应将相关数据或成功状态返回给控制器。

考虑到这些要点，模型不应将闪存消息设置为会话，而应根据模型的结果在控制器内完成。

Answer 2

将重定向和警报视为特定于一种特定形式的UI的，显然在模型中没有它们的位置。 只需总是尝试为您的应用程序描绘一个替代接口 ； 例如，用于管理任务的命令行界面或REST API。 重定向显然在这两种选择中都没有位置。 警报是值得商...的……至少，警报的形式会大不相同。 您的模型将需要能够将一些状态代码传回您的Controller或View，然后Controller的工作是对“负面”事件做出反应，而View的工作则是可视化任何警报（如有必要）。

例如，您的模型可能会执行以下操作：

public function registerUser(User $user) {
    ...

    if (!$successful) {
        throw new EmailAlreadyRegisteredException;
    }
    return true;
}

控制器可能看起来像这样：

public function userRegistration(Request $request) {
    try {
        $user = User::fromRequest($request);
        $this->services->get('Users')->registerUser($user);
        $this->view->render('registration_successful', $user);
    } catch (InvalidUserData $e) {
        $this->view->render('registration_form', $request, $e);
    } catch (EmailAlreadyRegisteredException $e) {
        $this->view->render('registration_failed', $user, $e);
    }
}

“警报”作为异常传递。 这只是Model向其调用者发出信号的一种方法。 然后由调用者决定如何对这些事件做出反应并可视化。 您当然不应该期望模型中有任何特定类型的可视化。 因此，您不想对特定的HTML编码的消息等进行硬编码。 您甚至根本不想接触人类语言，这就是View的全部工作。