0

Вот выдает ошибку: Call to a member function query() on a non-object

include "DataBase.class.php";

$opts = array(
        'user'    => 'root',
        'pass'    => '',
        'db'      => 'school',
        'charset' => 'utf8'
);

$db = new SafeMySQL($opts); // with some of the default settings overwritten

class School
{
    protected $fullname;
    protected $rating;
    protected $about;

    public function __construct($fullname, $rating, $about)
    {
        $this->fullname = $fullname;

        $this->rating = $rating;

        $this->about = $about;
    }

    public function save()
    {
        $db->query("INSERT INTO 'school' ('fullname', 'rating', 'about' ) VALUES ('$this->fullname', '$this->rating', '$this->about')");
    }
};

Что делаю не так?

1
  • 1
    > Что делаю не так? Пытаетесь достучаться до переменной, лежащей вне области видимости. Область видимости для метода - это объявленные классы/функции, параметры, $this, но не внешние переменные. Поэтому $db внутри метода не существует, надо его туда поместить с помощью global, параметра, или, самое лучшее, dependency injection.
    – etki
    Commented 23 июн. 2014 в 21:49

2 ответа 2

2

Просто маленький тест для демонстрации:

$db = 'db';
class School
{
    public function test_db() { print_r( $db ); }
};
$school = new School;
$school->test_db();

Вывод:

PHP Notice:  Undefined variable: db in /home/klopp/dev/1.php on line 6
PHP Stack trace:
PHP   1. {main}() /home/klopp/dev/1.php:0
PHP   2. School->test_db() /home/klopp/dev/1.php:9

Дальше пояснять нужно?

Обновление

@Александр Клюев, потому что в PHP такой код:

$db = 1;
function test_db() { $db = 2; }
echo $db; // выведется 1

аналогичен сишному:

int db = 1;
void test_db() { int db = 2; }
printf( "%d\n", db ); // выведется 1

То есть внутри функции переменная $db объявляется как локальная, и доступ к глобальной $db нужно организовывать отдельно.

P.S. RTFM.

5
  • Да, а то до меня не дошло
    – MAKADORE
    Commented 23 июн. 2014 в 22:09
  • Чуть выше уже объяснили - переменную $db из метода класса просто не видно. Но это ещё не всё: я не знаю что внутри DataBase.class.php, но есть шанс что эта переменная и в области видимости может быть не определена. То есть может сначала понадобиться проверка $db = new SafeMySQL($opts); if( !$db ) { /* продолжать не можем */ }
    – user6550
    Commented 23 июн. 2014 в 22:15
  • Но почему? Я же объявил до того, как вызвал
    – MAKADORE
    Commented 23 июн. 2014 в 22:24
  • @klopp, насколько знаю, невозможно проинициализироват класс в null, php либо упадет, либо создаст экземпляр. @Александр Клюев, потому что там, где обращаетесь, не видно того, где объявили.
    – etki
    Commented 23 июн. 2014 в 22:30
  • Т.е нужно было перед этим объявить? $school = new School; $school->save(); ?
    – MAKADORE
    Commented 23 июн. 2014 в 22:33
1

@Александр Клюев, еще раз

У каждого метода и функции своя область видимости, куда не попадают переменные из родительской области видимости (кроме замыканий с use). Поэтому $db, являющаяся внешней переменной для метода, из метода недоступна. Ее надо так или иначе в метод передать, способ "в лоб" такой:

class Superclass
{
    protected $db;
    public function __construct($db)
    {
        $this->db = $db;
    }
    public function testDb()
    {
        $this->db->test();
    }
}

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.