W jaki sposób mam przekazać uchwyt połączenia stworzonego w PDO do klasy. Tak abym w całej klasie mógł z niej korzystać nie tworząc cały czas nowego.

Przykład
plik: db_connect.php

[PHP] pobierz, plaintext 
<?php
// ...
try {
    $connect = new PDO("mysql:host=localhost;dbname=php", $username, $password);
}catch(PDOException $e) {
 
    echo "Error: " . $e->getMessage() . "<br/>";
 
}
?>
[PHP] pobierz, plaintext 

i TODOList.php

[PHP] pobierz, plaintext 
<?php
include("db_connect.php");
 
class TODOListModel {
    private $dbh; // zmienna w której chcę przechowywać uchwyt połączenia
 
    public function _construct($connection){
 
        $this->dbh = $connection ;
 
    }
    public function add_note($text, $author){
 
        $this->dbh->query("INSERT INTO todolist SET date=NOW(),
                           content='$text', author='$author'");
 
    }
    public function get_note(){
 
        //...
 
    }
    public function delete_note(){
 
        //...
 
    }
 
}
 
class TODOListController  {
 
}
 
$obj = new TODOListModel($connect);
$obj->add_note("Zrob pranie", "Krzysiek") or die("błąd");
?>
[PHP] pobierz, plaintext 

I w funkcji add_note wyskakuje mi błąd
Fatal error: Call to a member function query() on a non-object in C:\xampp\htdocs\todolist\TODOList.php on line 25

Wybaczcie ale uczę się dopiero programować obiektowo ta klasa to tak jakby ćwiczenie.

Zmień _construct na __construct (2x_)

Przekształciłem kod

[PHP] pobierz, plaintext 
<?php
 
class iPDO extends PDO {
 
    public function __construct($host, $login, $pass){
 
        new PDO("mysql:host=$host;dbname=php", $login, $pass);
 
    }
 
}
class TODOListModel  {
    private $dbh;
    public function __construct(iPDO $db){
 
        $this->dbh = $db ;
 
    }
    public function add_note($text, $author){
 
        $this->dbh->query("INSERT INTO todolist SET date=NOW(),    // Linia w której jest błąd
                           content='$text', author='$author'");
 
    }
     // ...
 
}
 
class TODOList  {
 
}
$dbh = new iPDO("localhost", "root", "");
$obj = new TODOListModel($dbh) ;
$obj->add_note("Zrob pranie", "Krzysiek");
 
?>
[PHP] pobierz, plaintext 

Niestety nadal nie mogę nic zrobić. Teraz pokazuje mi błąd: Warning: PDO::query() [pdo.query]: SQLSTATE[00000]: No error: PDO constructor was not called in C:\xampp\htdocs\todolist\TODOList.php on line 30

Vielta, dzięki ale to nie wina tego. U siebie w edytorze miałem dobrze, przypadkiem musiałem skasować jedną dolna spacje.

[PHP] pobierz, plaintext 
<?php
new PDO("mysql:host=$host;dbname=php", $login, $pass);
?>
[PHP] pobierz, plaintext 

Skoro to instancjujesz, to może by się przydało ten uchwyt zachować?


Dodaj konstrukcję rodzica w klasie rozszerzającej:

[PHP] pobierz, plaintext 
<?php
parent::__construct();
?>
[PHP] pobierz, plaintext 

erix, wkońcu zaczeło działać. Działa tylko bez tego parent::__construct(), gdy to dodaje pojawia się błąd:


Tu akurat jest bez argumentów, ale próbowałem dać w parent::__construct("mysql:host=$host;dbname=php",$login, $pass); i wtedy żadnego błędu już nie pokazuje lecz niestety nie dodaje rekordu do bazy.

Tak czy inaczej WIELKIE DZIĘKI za pomoc

Kod który działa:

[PHP] pobierz, plaintext 
<?php
 
class iPDO extends PDO {
    public $cdbh;
    public function __construct($host, $login, $pass){
 
        //parent::__construct();
        $this->cdbh = new PDO("mysql:host=$host;dbname=php",
                               $login, $pass);
       
    }
 
}
class TODOListModel  {
    private $dbh;
    public function __construct(iPDO $db){
 
        $this->dbh = $db->cdbh;
 
    }
    public function add_note($text, $author){
 
        $this->dbh->query("INSERT INTO todolist SET date=NOW(),
                           content='$text', author='$author'");
 
    }
}
 
class TODOList  {
 
}
$dbh = new iPDO("localhost", "root", "");
$obj = new TODOListModel($dbh);
$obj->add_note("Zrob pranie", "Krzysiek");
 
?>
[PHP] pobierz, plaintext 

Po co tworzysz klasę iPDO która nie robi kompletnie nic?

To może powiedz mi jak mam to zrobić inaczej bo próbowałem również zrobić to na inne sposoby między innymi w taki:

[PHP] pobierz, plaintext 
<?php
class TODOListModel  {
    private $dbh;
    public function __construct(PDO $db){
 
        $this->dbh = $db;
 
    }
    public function add_note($text, $author){
        $this->dbh->query("INSERT INTO todolist SET date=NOW(),
                                      content=$text, author=$author");
 
    }
}
 
class TODOList  {
 
}
$dbh = new PDO("mysql:host=localhost;dbname=php", "root", "");
$obj = new TODOListModel($dbh);
$obj->add_note("Wynieś śmieci", "Krzysiek");
?>
[PHP] pobierz, plaintext 

Niestety ten kod nie działa. Nie pokazuje się żaden błąd ale nie dodaje danych do tabeli.

1) PDO rzuca wyjątkami, gdzie masz bloki try i catch?
2) Twoje zapytanie jest niepoprawne, wygląda ono tak:

[SQL] pobierz, plaintext 
INSERT INTO todolist SET date=NOW(), content=Wynieś śmieci, author=Krzysiek
[SQL] pobierz, plaintext 

Tekst wstawiamy w cudzysłowie.
3) Włącz raportowanie błędów bo pewnie teraz wywala Ci FATALa, że nie obsłużyłeś wyjątku.

Crozin, w normalnym kodzie obsługuje wyjątki z PDOEXpection(kod umieszczony w pierwszym poście to dowodzi). Nie używam również metody query lecz prepare, bindParam i execute więc cudzysłowia nie są mi potrzebne przez co w tym przykładzie zwyczajnie o nich zapomniałem.

No to pokaż kod w takiej postaci w jakiej go masz i w jakiej niby nie działa.

No ok, ale po co?

[PHP] pobierz, plaintext 
<?php
 
/**
 * Description of TODOList
 *
 * TODOList is simple class for generate short note about future work
 *
 * @author Kri normal
 */
 
class iPDO extends PDO {
    public $cdbh;
    public function __construct($host, $login, $pass){
        try{
            $this->cdbh = new PDO("mysql:host=$host;dbname=php",
                                   $login, $pass);
        } catch(PDOException $e) {
 
            echo "Error!: " . $e->getMessage() . "<br/>";
 
        }
       
    }
 
}
 
/*
 * 
 * var @peq - prepare and execute query
 * 
 */
class TODOListModel  {
    private $dbh;
    public function __construct(iPDO $db){
 
        $this->dbh = $db->cdbh;
 
    }
   
    public function add_note($text, $author){
        try {
 
            $sql = "INSERT INTO todolist SET date=NOW(),
                    content=:content, author=:author";
            $peq = $this->dbh->prepare($sql);
            $peq->bindParam(':content', $text, PDO::PARAM_STR);
            $peq->bindParam(':author', $author, PDO::PARAM_STR);
            $peq->execute();
 
        } catch(PDOException $e) {
 
            echo "Error!: " . $e->getMessage() . "<br/>";
 
        }
        $this->dbh->query("INSERT INTO todolist SET date=NOW(),
                    content=$text, author=$author");
 
    }
    public function get_note(){
 
        $sql = "SELECT date, content, author FROM todolist";
        $peq = $this->dbh->prepare($sql);
        $peq->execute();
        
    }
    public function delete_note(){
 
        //mysql_delete();
 
    }
 
}
 
class TODOList  {
 
}
$dbh = new iPDO("localhost", "root", "");
$obj = new TODOListModel($dbh);
$obj->add_note("Wynieś śmieci", "Krzysiek");
 
?>
[PHP] pobierz, plaintext 

Mógłbyś mi teraz powiedzieć co wg. ciebie jest nie tak z klasą iPDO. Czemu ona nic nie robi? Jak już wcześniej napisałem jestem "świeży" w OOP nie jeszcze wielu rzeczy...

Zdecyduj się co do iPDO - dziedziczenie czy kompozycja.

No a zastanów się... co takiego niby robi ta klasa?

Crozin, ok masz pełną rację. Ostatecznie zrezygnowałem z klasy iPDO i po prostu w konstruktorze klasy TODOListModel jako parametr pobieram obiekt PDO(czyli uchwyt do bazy). Wcześniej już tak robiłem niestety popełniłem wielki błąd mianowicie nie ustawiłem id tabeli auto_increment co spowodowało te wszystkie problemy hehe Ale przynajmniej trochę się nauczyłem.

antyqjon, ostatecznie i definitywnie kompozycja ale iPDO już nie ma

Wielkie dzięki za pomoc.