Forum PHP.pl

Witam,
próbuję stworzyć klasę do zarządzania bazą danych:

[PHP] pobierz, plaintext 
include './config.php';//polaczenie z baza
 
class DaneUzytkownika
{
 
 protected $dane = array();
 
 function __construct($dane = array())
 {
 if (is_array($dane))
 $this->dane = $dane;
 else if ($dane instanceof DaneUzytkownika)
 $this->dane = $dane->pobierzJakoTablica();
 }
 
 
 function __get($parametr)
 {
 if (isset($this->dane[$parametr]))
 return $this->dane[$parametr];
 return '';
 }
 
 // ustawia atrybut
 function __set($parametr, $wartosc)
 {
 $this->dane[$parametr] = $wartosc;
 }
 
 // zwraca dane w postaci listy par klucz='wartosc'
 // oddzielonych przecinkiem
 function pobierzListeDanych()
 {
 $list = '';
 foreach($this->dane as $atrybut => $wartosc)
 $list .= $atrybut.'=\''.addslashes($wartosc).'\'';
 return $list;
 }
 
 // zwraca listę atrybutow oddzielonych przecinkiem
 function pobierzListeAtrybutow()
 {
 return join(', ', array_keys($this->dane));
 }
 
 // zwraca listę wartości oddzielonych przecinkiem
 function pobierzListeWartosci()
 {
 $values = array();
 foreach($this->dane as $wartosc)
 $values[] = '\''.addslashes($wartosc).'\'';
 return join(', ', $values);
 }
 
 // zwraca dane w postaci tablicy (zwraca przez wartosc)
 function pobierzJakoTablica()
 {
 return $this->data;
 }
}
 
class Uzytkownik extends DaneUzytkownika
{
 private $dane_w_bazie;
 
 // konstruktor - tworzenie obiektu
 function __construct(&$dane, $baza)
 {
 parent::__construct();
 
 if ($this->dane)
 $this->dane_w_bazie = new DaneUzytkownika($this->dane);
 }
 
 
 
 function zapisz()
 {
 mysql_query('UPDATE users SET '.$this->pobierzListeDanych().' WHERE '.$this->dane_w_bazie->pobierzListeDanych());
 if (mysql_affected_rows())
 {
 $this->dane_w_bazie = $this->dane;
 return true;
 }
 return false;
 }
 
 function usun()
 {
 mysql_query('DELETE FROM users WHERE '.$this->pobierzListeDanych());
 return (bool) mysql_affected_rows();
 }
}
 
class BazaUzytkownikow
{
 function pobierz(DaneUzytkownika $szukamy)
 {
 $query = mysql_query('SELECT * FROM users WHERE '.$szukamy->pobierzListeDanych());
 if (mysql_num_rows($query))
 return new Uzytkownik(mysql_fetch_assoc($query));
 return false;
 }
 
 function dodaj(DaneUzytkownika $dane)
 {
 mysql_query('INSERT INTO users('.$dane->pobierzListeAtrybutow().') VALUES('.$dane->pobierzListeWartosci().')');
 if (mysql_affected_rows())
 return new Uzytkownik($dane, $this);
 return false;
 }
 
 function zapisz($dane)
 {
 if ($dane instanceof Uzytkownik)
 $dane->zapisz();
 else
 $this->dodaj($dane);
 }
 
 function usun(Uzytkownik $uzytkownik)
 {
 return $uzytkownik->usun();
 }
}
 
$baza = new BazaUzytkownikow;
 
 
$szukamy = new DaneUzytkownika;
$szukamy->user_name= 'rr';
 
 
$uzytkownik = $baza->pobierz($szukamy);
 
// zmieniam dane pobranego wcześniej użytkownika
$uzytkownik->user_from = 'nowy gościu';
//próbuje zapisać zmiany
$baza->zapisz($uzytkownik);
[PHP] pobierz, plaintext 

Otrzymuję komunikat :
Catchable fatal error: Argument 1 passed to BazaUzytkownikow::dodaj() must be an instance of DaneUzytkownika
Co robię źle ?

W deklaracji metody dodaj() masz, że przyjmuje jako argument instancję klasy DaneUzytkownika, a Ty próbujesz tam wcisnąć jakiś inny typ.

Heh, trudno się domyślić... A jak rozwiązać ten problem ?

No przeciez masz wyraznie napisane...... Masz tam zapodac obiekt klasy DaneUzytkownika.
Tutaj zdaje sie tworzysz taki obiekt
$szukamy = new DaneUzytkownika;
, wiec to jego masz przekazywac.

Więc przekazuje

[PHP] pobierz, plaintext 
$baza = new BazaUzytkownikow;
$szukamy = new DaneUzytkownika; 
$szukamy->user_name = 'rr'; 
$szukamy->user_from = 'gdziediabelmowidobranoc'; 
echo "szukamy jest klasy " , get_class($szukamy) , "\n";
$uzytkownik = $baza->pobierz($szukany);
[PHP] pobierz, plaintext 

i otrzymuje
Catchable fatal error: Argument 1 passed to BazaUzytkownikow::pobierz() must be an instance of DaneUzytkownika, null given...
bazowałem na przykładzie stad

$szukany != $szukamy
...

Znajdź 10 różnic

Poprawiłem literówki

[PHP] pobierz, plaintext 
<?php
include './config.php';
db_connect();
class DaneUzytkownika
{
 // nasza wlasciwosc, chroniona przed dostepem z zewnatrz
 public $dane = array();
 
 function __construct($dane = array()) 
 {
 if (is_array($dane))
 $this->dane = $dane;
 else if ($dane instanceof DaneUzytkownika)
 $this->dane = $dane->pobierzJakoTablica();
 }
 
 // zwraca atrybut
 function __get($parametr)
 {
 if (isset($this->dane[$parametr]))
 return $this->dane[$parametr];
 return '';
 }
 
 // ustawia atrybut
 function __set($parametr, $wartosc)
 {
 $this->dane[$parametr] = $wartosc;
 }
 
 // zwraca dane w postaci listy par klucz='wartosc'
 // oddzielonych przecinkiem
 function pobierzListeDanych()
 {
 $args= array();
 foreach($this->dane as $atrybut => $wartosc)
 $args[]= $atrybut.'=\''.addslashes($wartosc).'\'';
 return join(' AND ', $args);
 }
 
 // zwraca listę atrybutow oddzielonych przecinkiem
 function pobierzListeAtrybutow()
 {
 return join(', ', array_keys($this->dane));
 }
 
 // zwraca listę wartości oddzielonych przecinkiem
 function pobierzListeWartosci()
 {
 $values = array();
 foreach($this->dane as $wartosc)
 $values[] = '\''.addslashes($wartosc).'\'';
 return join(', ', $values);
 }
 
 // zwraca dane w postaci tablicy (zwraca przez wartosc)
 function pobierzJakoTablica() 
 {
 return $this->data;
 } 
}
 
class Uzytkownik extends DaneUzytkownika
{
 public $dane_w_bazie;
 
 // konstruktor - tworzenie obiektu
  function __construct($dane, $baza) 
 {
 parent::__construct();
 
 if ($this->dane) 
 //$this->dane_w_bazie = new DaneUzytkownika($this->dane);
 $this->dane_w_bazie = new DaneUzytkownika($dane);
 }
 
 function wyswietl()
 {
 
 }
 
 function zapisz()
 {
 mysql_query('UPDATE users SET '.$this->pobierzListeDanych().' WHERE '.$this->dane_w_bazie->pobierzListeDanych()); 
 if (mysql_affected_rows()) 
 { 
 $this->dane_w_bazie = $this->dane;
 return true;
 }
 return false;
 }
 
 function usun()
 {
 mysql_query('DELETE FROM users WHERE '.$this->pobierzListeDanych());
 return (bool) mysql_affected_rows();
 }
}
 
class BazaUzytkownikow
{
 function pobierz(DaneUzytkownika $szukamy) 
 {
 $query=mysql_query('SELECT * FROM users WHERE '.$szukamy->pobierzListeDanych()); 
 if (mysql_num_rows($query)) {
            return( new Uzytkownik (mysql_fetch_assoc($query)));
        }
        return false;
 }
 
 function dodaj(DaneUzytkownika $dane)
 {
 mysql_query('INSERT INTO users('.$dane->pobierzListeAtrybutow().') VALUES('.$dane->pobierzListeWartosci().')');
 if (mysql_affected_rows())
 return new Uzytkownik($dane, $this);
 return false;
 }
 
 function zapisz(DaneUzytkownika $dane)
 {
 if ($dane instanceof Uzytkownik)
 $dane->zapisz();
 else
 $this->dodaj($dane);
 }
 
 function usun(Uzytkownik $uzytkownik)
 {
 return $uzytkownik->usun();
 }
}
$baza = new BazaUzytkownikow;
$szukamy = new DaneUzytkownika; // tworzymy nową jakby tablicę na dane
$szukamy->user_name = 'aaa'; // podajemy login szukanego uzytkownika
$szukamy->user_from = 'gdziediabrlmowidobranoc'; // mozemy takze podac inny parametr
//echo "szukamy jest klasy " , get_class($szukamy) , "\n";
 
$uzytkownik = $baza->pobierz($szukamy); // wyszukujemy w bazie przekazując jej dane z tablicy, baza zwraca nam Użytkownika
var_dump($uzytkownik);
 
// zmieniamy dane pobranego wcześniej użytkownika
//$uzytkownik->user_from = 'Tajemniczy gościu9';
 
$uzytkownik->zapisz();
//$baza->zapisz($szukamy);
[PHP] pobierz, plaintext 

Pojawia sie błąd
Call to a member function pobierzListeDanych() on a non-object
Nie przypisuje do zmiennych obiektu wartości :object(Uzytkownik)#3 (2) { ["dane_w_bazie"]=> NULL ["dane"]=> array(0) { } }

Byloby jeszcze milo, jakbys raczyl napisac ktorej linii dotyczy blad, gdyz pobierzListeDanych to ty odpalasz w wielu roznych miejscach... WEź pod uwage fakt, ze tu wrozek nie ma.