web-dev-qa-db-fra.com

Appel à une fonction membre sur null?

J'ai fait toutes sortes de choses pour comprendre ce qui ne va pas avec le code, mais je n'arrivais pas à comprendre ce qui causait cette erreur.

Ci-dessous se trouve la classe appelée 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;
        }
    }

Quand je crée un objet de classe Topic,

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

il y a toujours une erreur comme celle-ci:

Erreur fatale: Appel à une requête de fonction membre () sur null dans /Applications/XAMPP/xamppfiles/htdocs/forumproject/libraries/Topic.php sur la ligne 25, c'est-à-dire $ this-> db-> query ( ...)

La classe Database fonctionne parfaitement car j'ai exécuté des tests indépendamment pour cette classe, et cela ne pose aucun problème.

Voici la classe 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();
        }


    }

?>
6
manjula

Vous avez une faute de frappe dans public function __contruct(), elle devrait être public function __construct() { (il vous manque un s)

16
gmc