关于架构,从模型到控制器抛出异常时,下面两种方法中哪一种是好的实践?
结构 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仍然有catch语句的原因是模型应该执行回滚操作。如果我在这里从模型中删除try-catch,让控制器直接捕获异常,那么回滚将在控制器上处理,我认为这会使控制器的功能变得混乱。
请告诉我你的想法。 谢谢!
1xx:请稍等; 2xx:好的,给你; 3xx:别来烦我; 4xx:你出错了; 5xx:我出错了。
- Jeremy Gibersonthen()
和catch()
)。 - cre8