Laravel控制器 - 模型异常处理结构与数据库事务

Question

关于体系结构,当从模型到控制器抛出异常时,哪两个是一个很好的做法？

结构A:

UserController.php

public function updateUserInfo(UserInfoRequest $request, UserModel $userModel)
{
    $isError = false;
    $message = 'Success';

    try {
        $message = $userModel->updateUserInfo($request->only(['username', 'password']));
    } catch (SomeCustomException $e) {
        $isError = true;
        $message = $e->getMessage();
    }

    return json_encode([
        'isError' => $isError,
        'message' => $message
    ]);
}

UserModel.php

public function updateUserInfo($request)
{
    $isError = false;
    $message = 'Success';

    $username = $request['username'];
    $password = $request['password'];

    try {
        $this->connect()->beginTransaction();

        $this->connect()->table('users')->where('username', $username)->update(['password' => $password]);

        $this->connect()->commit();
    } catch (\Exception $e) {
        $this->connect()->rollback();
        $isError = true;
        $message = $e->getMessage();        
    }

    return [
        'isError' => $isError,
        'message' => $message
    ];
}

结构B:

UserController.php

public function updateUserInfo(UserInfoRequest $request, UserModel $userModel)
{
    $isError = false;
    $message = 'Success';

    try {
        $userModel->updateUserInfo($request->only(['username', 'password']));
    } catch (SomeCustomException $e) {
        $isError = true;
        $message = $e->getMessage();
    } catch (QueryException $e) {
        $isError = true;
        $message = $e->getMessage();    
    }

    return json_encode([
        'isError' => $isError,
        'message' => $message
    ]);
}

UserModel.php

public function updateUserInfo($request)
{
    $username = $request['username'];
    $password = $request['password'];

    try {
        $this->connect()->beginTransaction();

        $this->connect()->table('users')->where('username', $username)->update(['password' => $password]);

        $this->connect()->commit();
    } catch (\Exception $e) {
        $this->connect()->rollback();
        throw new QueryException();
    }
}

在结构A中,模型捕获任何异常,回滚事务并在控制器发生错误或没有错误时返回.然后控制器返回从模型返回的任何内容.

在结构B中,模型捕获任何异常,回滚事务,然后在发生异常时抛出QueryException.然后,控制器从模型中捕获抛出的QueryException,如果它有错误或没有,则返回返回.

结构B仍然有一个问题的原因是该模型应该是进行回滚的模型.如果我要删除模型上的try-catch和控制器以直接捕获异常,那么回滚将在控制器上处理,我认为这会使控制器的功能变得混乱.

让我知道你的想法.谢谢!

Answer 1

为什么我认为B的方法更好:

1. 您的模型应该只包括逻辑部分:这包括与数据库的通信(事务和回滚),而不是您要打印给用户的错误消息的格式.

2. 保持模型清洁:这是MVC结构中最重要的部分.如果搞砸了,很难找到任何错误.

3. 外包错误处理:如果你把它放在控制器中你可以选择在那里处理它(也许你想要一些特殊的格式化输出用于这个方法,或者你需要一些其他函数来调用)或者你在处理它App\Exceptions\Handler.在这种情况下,您可以在此处呈现此错误消息,而不必在控制器中执行此操作.

因此,如果你不需要任何特殊的函数调用,并希望使用Laravel的全部功能,我建议你使用Structure C.

UserController.php

public function updateUserInfo(UserInfoRequest $request, UserModel $userModel)
{
    $userModel->updateUserInfo($request->only(['username', 'password']));
    return response()->json(['message' => 'updated user.']); 
}

UserModel.php

public function updateUserInfo($request)
{
    $username = $request['username'];
    $password = $request['password'];
    try {
        $this->connect()->beginTransaction();

        $this->connect()->table('users')->where('username', $username)->update(['password' => $password]);

        $this->connect()->commit();
    } catch (\Exception $e) {
        $this->connect()->rollback();
        throw new QueryException();
    }
}

应用\ \例外处理程序

public function render($request, Exception $exception)
{
    //catch everything what you want
    if ($exception instanceof CustomException) {
        return response()->json([
          'message' => $exception->getMessage()
        ], 422);
    }

    return parent::render($request, $exception);
}

您可以清楚地分离数据库内容(模型),表示内容(Controller)和错误处理(Handler).结构C允许您在其他控制器函数中具有相同情况的其他函数中重用错误处理.

这是我的观点,但我愿意讨论您认为这种方法不是最佳解决方案的任何情况.