Проблемы с PDO в ООП, что делать?

Question

[Banan44]

<?php

/* Не работает */

class Database {
	private $mysqli;

	public function __construct(){
		$dsn = "mysql:host=localhost;dbname=banan165_database";
		$this->mysqli = new PDO($dsn, "root", "")
	}
	public function query($sql){
		$this->mysqli->query($sql)
	}
}

$db = new Database();
$db->query("INSERT INTO categories(name) VALUES('PDO')");

/* Работает */

$dsn = "mysql:host=localhost;dbname=banan165_database";
$mysqli = new PDO($dsn, "root", "");
$mysqli->query("INSERT INTO categories(name) VALUES ('PDO')");

?>

Почему с ООП не работает? Строка где name = PDO не добавляется с использованием ООП, а без ООП работает.

Comments

[Дмитрий]

$this->mysqli = new PDO улыбнуло

[Дмитрий]

VicTHOR, ты походу не увидел прикола)

[Дмитрий]

VicTHOR, у него свойство называется mysqli а в нем объект PDO =)

[alex-1917]

а почему банан? в школьном дневнике одни бананы?)))

[Banan44]

Дмитрий, ))))

Answer 1

[Максим]

Решение:

class Database {
  private $mysqli;

  public function __construct(){
    $dsn = "mysql:host=localhost;dbname=banan165_database";
    $this->mysqli = new PDO($dsn, "root", "");
  }
  public function query($sql){
   return $this->mysqli->query($sql);
  }
}

$db = new Database();
$db->query("INSERT INTO categories(name) VALUES('PDO')");

Но если правильно - так не делается.

Comments

[Banan44]

а что поменялось?

[Максим]

Banan44, return и точка с запятой.

[Adamos]

Максим, а зачем return-то? ;)

[Banan44]

Максим, а зачем возвращать что то?

[Banan44]

Спасибо за ; , а работает и без return

[Максим]

Banan44, да) Это я поставил return, а вам решать))

[Максим]

VicTHOR, да ставят)) Просто они в этом случае и так работают. По правильному, конечно, ставить.

Answer 2

[FanatPHP]

Потому что это бессмысленный "ООП". PDO - это уже класс, и добавлять к нему бессмысленный Database нет ни малейшей причины. Тем более зачем-то называя переменную mysqli, делая код не только бессмысмленным, но и запутанным.

Свой класс надо писать, когда он что-то добавляет к существующему.

Comments

[Максим]

Согласен, хотя бы так... )) Бессмысленно вообще пытаться делать обёртку когда мы живем в 21 веке. Полно открытого исходного кода с решением подобных задач. Некоторые куда получше и гибче.

[FanatPHP]

Максим, непонятно, зачем у тебя в run() вдруг throw new Exception, непонятно, зачем статика, непонятно, зачем переписывать исходный код каждый раз когда надо подключиться к новой базе данных.
Плюс функции get*сделаны криво. Явно делалось не для практического приенения и никогда не использовалось.

[Максим]

FanatPHP, да это не мой код. Вы путаете)) Я просто нашел первый попавшийся пример. Такими вещами я не страдаю))

[FanatPHP]

Максим, ну вот и не надо постить ссылки на код, который ты не понимаешь

[Максим]

FanatPHP, хорошо, босс)

[alex-1917]

Максим, да тут дело не в хорошо босс, а в том,что босс тебя между строк предупреждает, что недолго и до бани, создавая вопросы высосал-из-пальца-помогите.

[Максим]

alex-1917, я вопрос и не задавал.

Спорить с босом не собираюсь) Пример, действительно, не совсем удачный. Сунул первый попавшийся из поисковика даже не смотрев. И мой пример лиш в комментарии, а не в ответе, который претендует на решение. Откуда такие страсти?