0

在 PHP MVC 应用程序中处理可能的 MySQL 错误的最常见/最佳实践是什么?最好将模型中的成功布尔值传递给控制器​​还是抛出异常?假设我正在调用存储过程,我可能遇到的可能错误是数据库连接、用户没有权限、无效数据或随机 MySQL 错误,这可能是最有效/最有效的处理方法。

例如: 方法一:

//UserController.php
private function get_user_info(){
    $user_info = $user_model->read_user_info(123);

    if($user_info[0]){
        //Do stuff with user data
    }else{
        //Check if db, permission, invalid data, or random MySQL error
    }
}

//UserModel.php
public function read_user_info($read_user_id){
    $stmt = $db->prepare("CALL read_user_info(?, ?)");

    $stmt->bindParam(1, $current_user_id);
    $stmt->bindParam(2, $read_user_id);

    if($stmt->execute()){
        $result_set = $stmt->fetchAll(PDO::FETCH_ASSOC);

        //Does the user have permission to read other user's info
        if($result_set["granted"]){
            return array(true, $result_set["user_info"]);
        }else{
            return array(false, "Permission error");
        }
    }else{
        return array(false, "MySQL error");
    }
}

方法二:

//UserController.php
private function get_user_info(){
    try{
        $user_info = $user_model->read_user_info(123);

        //Do stuff with user data
    }catch(ConnectionException $e){

    }catch(InvalidDataException $e){

    }catch(MySQLException $e){

    }
}

//UserModel.php
public function read_user_info($read_user_id){
    $stmt = $db->prepare("CALL read_user_info(?, ?)");

    $stmt->bindParam(1, $current_user_id);
    $stmt->bindParam(2, $read_user_id);

    if($stmt->execute()){
        $result_set = $stmt->fetchAll(PDO::FETCH_ASSOC);

        //Does the user have permission to read other user's info
        if($result_set["granted"]){
            return $result_set["user_info"];
        }else{
           throw new PermissionException();
        }
    }else{
        throw new MySQLException();
    }
}
4

1 回答 1

1

最好将模型中的成功布尔值传递给控制器​​还是抛出异常?

最好在模型中完成整个错误处理,因为它是整个 MVC 应用程序概念中“可重用”代码的合适位置。

什么可能是最有效/最有效的处理方法。

我说这是Method 1。首先,控制器仅接收user_info变量并在此事件上应用自定义逻辑,特别是没有捕获各种类型Exception和处理(恕我直言,这应该集中在模型中)。

无论如何,设置自定义Error处理程序可能会变得非常有用 - 集中式方法、错误显示、日志管理等)。我使用的示例Error类:

public static $error_types = array(
    E_ERROR => 'E_ERROR',
    E_WARNING => 'E_WARNING',
    E_PARSE => 'E_PARSE',
    E_NOTICE => 'E_NOTICE',
    E_CORE_ERROR => 'E_CORE_ERROR',
    E_CORE_WARNING => 'E_CORE_WARNING',
    E_COMPILE_ERROR => 'E_COMPILE_ERROR',
    E_COMPILE_WARNING => 'E_COMPILE_WARNING',
    E_USER_ERROR => 'E_USER_ERROR',
    E_USER_WARNING => 'E_USER_WARNING',
    E_USER_NOTICE => 'E_USER_NOTICE',
    E_STRICT => 'E_STRICT',
    E_RECOVERABLE_ERROR => 'E_RECOVERABLE_ERROR',
    E_DEPRECATED => 'E_DEPRECATED',
    E_USER_DEPRECATED => 'E_USER_DEPRECATED'
);

public static $throwables = array();

public static function set_throwable_handlers()
{
    error_reporting(E_ALL);
    ini_set('display_errors', FALSE);
    ini_set('log_errors', TRUE);
    ini_set('error_log', APP_DIR.DIR_SEP.'system'.DIR_SEP.'logs'.DIR_SEP.'error.log');

    set_error_handler(array('Error', 'error_handler'));
    set_exception_handler(array('Error', 'exception_handler'));

    register_shutdown_function(array('Error', 'shutdown_handler'));
}

public static function set_throwable($error_number, $error_text, $error_file, $error_line)
{
    self::$throwables[$error_number][] = array('type' => self::$error_types[$error_number], 'text' => $error_text, 'file' => $error_file, 'line' => $error_line);
}

public static function exception_handler(Exception $exception)
{
    self::set_throwable($exception->getCode(), $exception->getMessage(), $exception->getFile(), $exception->getLine());
}

public static function error_handler($error_number = '', $error_text = '', $error_file = '', $error_line = '')
{
    self::set_throwable($error_number, $error_text, $error_file, $error_line);
}

public static function shutdown_handler()
{
    $error = error_get_last();

    if ($error !== NULL)
    {   
        self::set_throwable($error['type'], $error['message'], $error['file'], $error['line']);

        View::display();
    }
}

public static function throw_error($error_text, $error_number = E_USER_NOTICE)
{
    trigger_error($error_text, $error_number);

    View::display();

    exit;
}
于 2015-04-08T11:05:33.367 回答