PHP 未在 Class 中抛出指定的错误消息

Question

我有三个 classes，每个在连接到我的数据库时处理不同的功能。
Class 1，名称为 ConnectDB()，处理 MySqli() 连接； Class 2 with name MessageOut() 通过 $_SESSION['message'] 键向用户提供错误和成功消息 UI and Class 3 with name WebApp() uses MySqli 连接从数据库中检索数据并将其传递给 UI.

这和我平时做数据库连接、收集和展示的方式不一样。通常我只是在三个文件中创建了很多函数，并在需要时调用它们。但是我的网站越来越大，越来越复杂，所以我正在重新组织一切。
我当前的问题是当我在第 3 个 class 中创建数据库连接时，它没有抛出我编程的错误。

class ConnectDB {
    //Connecting to database
    public function connect() {
        //connecting to mysql
        @$conn = new mysqli(DB_HOST, DB_USERRW, DB_PASSWRW, DB_DBASE);
        // check validity of database connection
        if (mysqli_connect_errno()) {
            return false;
            exit();
        }
        // select the database to use
        return $conn;
    }
}

ini_set('display_errors', 1); 
    ini_set('log_errors',1); 
    error_reporting(E_ALL); 
    mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
class WebApp {
    protected $db;
    function __construct(){
        $this->connector = new ConnectDB();
        $this->messager = new MessageOut();
        try {
            $this->db = $this->connector->connect();
            if(!$this->db) {
                throw new Exception(' Database connection is currently unavailable.');
            }
        } catch (Exception $e) {
            mysqli_report(MYSQLI_REPORT_OFF);
            $errors = array();
            $message = $e->getMessage();
            //populate array with message and divert output to error class functions
            array_push($errors, $message);
            $this->messager->erroutput($errors);
        }
    }
    public function selectIdata() {
        //select data
        try {
            $query = "SELECT *
                        FROM thetable";
            $stmt = $this->db->prepare($query);
            $stmt->execute();
            $stmt->store_result();

            $stmt->bind_result($idata);
            $result = [];
            if ($stmt->num_rows > 0) {
                while ($stmt->fetch()) {
                    $result[] = $idata;
                }
                return $result;
            } else {
                return false;
            }
        } catch (Exception $e) {
            mysqli_report(MYSQLI_REPORT_OFF);
            $errors = array();
            $message = $e->getMessage();
            //populate array with message and divert output to error class functions
            array_push($errors, $message);
            $this->messager->erroutput($errors);
        }
    }
}

我将本地主机定义的密码更改为错误的密码并加载了文件，但在我的第一个 class 中的第 10 行给出了错误，即使错误已被抑制。这个想法是在使用之前检查连接。
我如何在第 3 个 class 中得到抛出的错误给我 'Database connection is currently unavailable.' 消息？

编辑
所以我重新评估了我所做的并在我的第 3 个 class 的 constructor 中设置了一个 try-catch 块，但我知道我得到一个错误：Fatal error: Uncaught Error: Call to a member function prepare() on null in 第 3 个 [=40] =] 在第 41 行，其中使用了数据库连接。

Answer 1

看来你对错误报告的概念有误解。我将尝试向您阐明一般概念。

错误报告是为了通知开发人员错误的编码、错误和其他需要修复的潜在问题。错误报告不适用于该产品的用户。当你在开发时，你可以为自己启用 display_errors，但你不应该将它留在代码中。但是，您应该始终记录错误。 PHP 有一个非常好的错误记录器，但我可以理解它对您来说可能不够用，并且您想记录更多信息和错误消息。

您可以编写通用错误处理程序并捕获应用程序抛出的所有错误和异常，并使用您自己的记录器软件将其记录到服务器上的安全位置。不要在代码中间捕获异常。这样的记录器需要是您的应用程序的核心，并且在一个单独的文件中，以便能够捕获所有错误。

出于这个原因，您的 try-catch 不是很有用，因为您将它放在多个位置，并且它与您的应用程序代码交织在一起。此外，您只捕获异常并忽略错误。你应该抓住两者。使用 catch(\Throwable $e) 之类的东西来捕获两者。

@ 是错误抑制运算符。应该不惜一切代价避免它。如果你不能避免它，那么你需要重写代码来避免它。你在那里用 mysqli 连接做什么你实际上忽略了错误两次。首先，您使用 @ 然后终止您的脚本。不要杀死脚本。不要沉默错误。让你的错误冒出来，让它们被你的错误处理程序捕获。

如果你仔细想想，你的classConnectDB是相当没用的。要连接到数据库，您始终需要相同的 3 行。 mysqli 已经是一个 class 所以在另一个 class 中包装 3 行是没有意义的。正确的代码应该无非是：

mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
$conn = new mysqli(DB_HOST, DB_USERRW, DB_PASSWRW, DB_DBASE);
$conn->set_charset('utf8mb4');

当 mysqli 无法连接时，您当前的脚本将退出，但即使您不消除错误并且打开了错误报告，变量也不可能为空。 if(!$this->db) 成为另一个谬误。此外，当您刚刚将一个异常静音时，为什么要抛出异常？这让我想到了另一点。 为什么一抓到异常就抛出？当然，整个逻辑无非就是一个简单的if语句：

if(!$this->db) {
    $this->messager->erroutput([' Database connection is currently unavailable.']);
}

我看到您已将 class MessageOut 命名为 MessageOut，我真的希望您不要将错误消息暴露给用户。这不仅是糟糕的用户体验，而且还有安全风险。您应该改为实现一个漂亮的 HTTP 500 错误页面，或者如果您的应用程序足够复杂，您自己的错误页面将在您的错误处理程序捕获到错误时显示。

一旦发现错误就关闭 mysqli 错误报告是没有用的。只需从代码中删除 mysqli_report(MYSQLI_REPORT_OFF); 即可。

要形象化我所描述的内容，请考虑以下代码：

<?php

// ini_set('display_errors', 1);
ini_set('log_errors', 1);
error_reporting(E_ALL);

class WebApp {
    protected $db;

    function __construct() {
        $this->messager = new MessageOut();

        mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT); // it can also be at the top of the script, but it makes more sense to put it together with the rest of mysqli code
        $this->db = new \mysqli(DB_HOST, DB_USERRW, DB_PASSWRW, DB_DBASE);
        $this->db->set_charset('utf8mb4');
    }

    public function selectIdata() {
        //select data
        $query = "SELECT *
                    FROM thetable";
        $stmt = $this->db->prepare($query);
        $stmt->execute();
        $stmt->store_result();

        $stmt->bind_result($idata);
        $result = [];
        while ($stmt->fetch()) {
            $result[] = $idata;
        }
        return $result;
    }
}

try {
    $app = new \WebApp();
} catch (\Throwable $e) {
    // log your errors here.
}

这不是完美的代码，因为错误处理程序不在那里，它也应该在一个单独的文件中，但总体思路是避免应用程序逻辑中不必要的代码。不要尝试捕捉。不要消除错误，也不要添加无用的代码。把事情简单化。

PHP 未在 Class 中抛出指定的错误消息

PHP not throwing specified error message in Class

php

oop

mysqli

try-catch

throw