Fatal error: Call to a member function error() on a non-object

Question

I'm trying to follow an OOP PHP tutorial from phpacademy and I'm stuck. I've checked through the code countless times and I just can't seem to figure it out. Others have asked similar questions, but I've not found a solution that works for me.

I'm getting the error:

Fatal error: Call to a member function error() on a non-object in /var/www/vhosts/nightra.net/httpdocs/index.php on line 5

I'm relatively new to OOP, so I'm sure it's something obvious. Can any of you experts point me in the right direction, please?

Here is my code:

index.php

    <?php
require_once 'core/init.php'; 

$user = DB::getInstance()->get('users', array('username', '=', 'alex'));
if($user->error()){
    echo 'No user';
}else{
    echo 'OK'; 
}

init.php

<?php
session_start();

$GLOBALS['config'] = array(
        'mysql'=>array(
            'host' => '127.0.0.1',
            'username' => 'myusernamehere',
            'password' => 'mypasswordhere',
            'db' => 'mydatabasename'
        ),
        'remember'=>array(
            'cookie_name' =>'hash',
            'cookie_expiry' => 604800
        ),
        'session'=>array(
            'session_name' => 'user'
        )
);

spl_autoload_register(function($class) {
    require_once 'classes/' . $class . '.php';
});

require_once 'functions/sanitize.php';

DB.php

<?php 
class DB {
    private static $_instance = null;
    private $_pdo,
            $_query,
            $_error = false,
            $_results,
            $_count = 0;

    private function __construct() {
        try{
            $this->_pdo = new PDO('mysql:host=' . Config::get('mysql/host') . ';dbname=' . Config::get('mysql/db'), Config::get('mysql/username'), Config::get('mysql/password'));

        } catch (PDOException $e) {
               die($e->getMessage());
        }
    }
    public static function getInstance() {
        if(!isset(self::$_instance)){
            self::$_instance = new DB();
        }
        return self::$_instance;

    }

    public function query($sql, $params = array()){
        $this->_error = false;
        if($this->_query = $this->_pdo->prepare($sql)){
            $x=1;
            if(count($params)){
                foreach($params as $param){
                    $this->_query->bindValue($x, $param);
                    $x++;
                }
            }
             if($this->_query->execute()){
                 $this->_results = $this->_query->fetchAll(PDO::FETCH_OBJ);
                 $this->_count = $this->_query->rowCount();
             }else{
                 $this->_error = true;                 
            }            
        }    
        return $this;
    }

    public function action($action, $table, $where = array()){
        if(count($where) === 3){
            $operators = array('=', '>', '<', '>=', '<=');

            $field     = '$where[0]';
            $operator  = '$where[1]';
            $value     = '$where[2]';

            if(in_array($operator, $operators)){
                $sql = "{$action} FROM {$table} WHERE {$field} {$operator} ?";
                if( ! $this->query($sql, array($value))->error()){
                    return $this;
                }
            }
        }
        return false;
    }
    public function get($table, $where){
        return $this->action('SELECT *', $table, $where);

    }
    public function delete($table, $where){
        return $this->action('DELETE', $table, $where);
    }

    public function error(){
        return $this->_error;
    }
 }

Config.php

<?php
class Config {
    public static function get($path = null){
        if($path){
            $config = $GLOBALS['config'];
            $path = explode('/', $path);

            foreach($path as $bit){
                if(isset($config[$bit])){
                    $config = $config[$bit];
                }
            }

            return $config;
        }

        return false;
    }
}

Answer 1

Above answer is correct on looking on the entire code array $where is accessed within single quotes actually it should be $field = $where[0]; similar for rest elements of array where

Answer 2

looking at the code of index.php and DB.php

$user has its value coming from the BD::get method which return the value from the DB::action method which could be returning false, so you would need to test if $user is false.

At this point you should also test the query by hand on the database. And also try to get the PDO errors if any.