1. ホーム
  2. php

[解決済み】メンバ関数をnullで呼び出す?

2022-02-03 13:05:58

質問

実際に何が問題なのかを理解するためにいろいろとやってみたのですが、どうしてもこのエラーの原因がわかりませんでした。

以下は Topic :

class Topic {

        // Init DB variables
        private $db;

        /*
        *   Constructor
        */
        public function __contruct() {
            $this->db = new Database();

        }

        /*
        *   Get All Topics
        */
        public function getAllTopics() {

            $this->db->query("SELECT topics.*, users.username, 
                              users.avatar, categories.name 
                              FROM topics 
                              INNER JOIN users 
                              ON topics.user_id = users.id 
                              INNER JOIN categories 
                              ON topics.category_id = categories.id 
                              ORDER BY create_date DESC");


            // Assign the results
            $results = $this->db->resultset();

            return $results;
        }
    }

クラスのオブジェクトを作成すると Topic ,

$topic = new Topic();
$results = $topic->getAllTopics();

常にこのようなエラーが発生します。

致命的なエラー : メンバ関数query()のnullに対する呼び出し。 /Applications/XAMPP/xamppfiles/htdocs/forumproject/libraries/Topic.php つまり、$this->db->query(...)である。

Database クラスは、私がそのクラスに対して独自にテストを実行したところ、何の問題もなく動作しています。

以下はDatabaseクラスです。

<?php 

    /*
    * Using PDO for the first time in my life.
    */

    class Database  {

        private $host = DB_HOST;
        private $user = DB_USER;
        private $pass = DB_PASS;
        private $dbname = DB_NAME;

        private $dbh;
        private $error;
        private $stmt;

        public function __construct()   {
            // Set DSN
            $dsn = 'mysql:host=' . $this->host . ';dbname=' . $this->dbname;

            // Set options
            $options = array(
                PDO::ATTR_PERSISTENT => true, 
                PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION
            );

            // Create a new PDO instance
            try { 
                $this->dbh = new PDO($dsn, $this->user, $this->pass, $options);
            }
            // Catch the errors using exceptions handler
            catch(PDOException $e){
                $this->error = $e->getMessage();
            }
        }

        public function query($query){
            $this->stmt = $this->dbh->prepare($query);
        }

        public function bind($param, $value, $type = null){
            if(isnull($type)){
                switch(true){
                    case is_int($value):
                        $type = PDO::PARAM_INT;
                        break;

                    case is_bool($value):
                        $type = PDO::PARAM_BOOL;
                        break;

                    case is_null($value):
                        $type = PDO::PARAM_NULL;
                        break;

                    default : 
                        $type = PDO::PARAM_STR;
                }
            }

            $this->stmt->bindValue($param, $value, $type);
        }


        public function execute(){
            return $this->stmt->execute();
        }

        public function resultset(){
            $this->execute();
            return $this->stmt->fetchAll(PDO::FETCH_OBJ);
        }

        public function single(){
            $this->execute();
            return $this->stmt->fetch(PDO::FETCH_OBJ);
        }

        public function rowCount(){
            return $this->stmt->rowCount();
        }

        public function lastInsertId(){
            return $this->dbh->lastInsertId();
        }

        public function beginTransaction(){
            return $this->dbh->beginTransaction();
        }

        public function endTransaction(){
            return $this->bbh->commit();
        }

        public function cancelTransaction(){
            return $this->dbh->rollBack();
        }


    }

?>

解決方法は?

の中にタイプミスがあります。 public function __contruct() であるべきです。 public function __construct() { (sが抜けています)