Klasa w klasie? Opcje

[Tarcil]

Witam serdecznie!

Od jakiegoś czasu próbuję opanować OOP w PHP i idzie mi, nie przymierzając, słabo. Mój problem polega na tym, że nie potrafię z jednej klasy odwołać się do metod innej. Nie wiem, czy to nie jest przypadkiem durnota jakaś i mój błąd w próbie myślenia obiektowo... proszę o pomoc w każdym bądź razie.

Mam klasę MySqlEng(). Konstruktor tej klasy tworzy połączenie z bazą danych i zapisuje uchwyt połączenia w prywatnej zmiennej klasy. Dalej mam 4 metody: selectQuery, updateQuery, insertQuery i deleteQuery, które przygotowują w prywatnej zmiennej $query string, który metodzie doQuery() służy do wykonania odpowiedniego zapytania w bazie danych. Dochodzi jeszcze metoda fetchResults(), która pozwala na przewijanie rekordów uzyskanych w odpowiedzi na zapytanie.

Standardowo używam tej klasy tak:

[PHP] pobierz, plaintext 
<?php
$sql = new MySqlEng();
 
$sql -> selectQuery('nazwa, ilosc', 'produkty', 'id > 93');
while($res = $sql->fetchResults())
 {
 echo $res['nazwa'].' pozostała ilość: '.$res['ilosc'].'<br />';
 }
?>
[PHP] pobierz, plaintext 

Próbuję teraz złożyć klasę pagesSupport(), której zadaniem będzie: dodawanie stron do bazy danych, ich edycja i zmiana, oraz wyciągnie z bazy danych o odpowiedniej podstronie i wyswietlanie jej na ekranie. Zwykłe użycie wewnątrz metody tej klasy metody $sql->selectQuery() wywołuje błąd...

Jak skorzystać z klasy MySqlEng() wewnątrz klasy pagesSupport()(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

Podaję też kod klasy MySqlEng():

[PHP] pobierz, plaintext 
<?php
//config
    //this data should be inserted in another file, which is not available  by http://
    define('DATABASE_HOST', 'localhost');
    define('DATABASE_USER', 'tarcil_gsms');
    define('DATABASE_PASS', 'gsms123');
    define('DATABASE_NAME', 'tarcil_gsms');
    
    class MySqlEng 
        {
        private $handler;
        public $query;
        private $results;
        private $affected_rows;
        private $last_row_id;
        private $selected_rows;
        public $records = array();
        
        
        public function __construct()
            {
            if(@!$sql_handler = mysql_connect(DATABASE_HOST, DATABASE_USER, DATABASE_PASS))
                {
                throw new Exception('Problem z połączeniem do bazy danych.<br />Nr błędu: <b>'.mysql_errno().'</b><br /><b>Komunikat: </b>'.mysql_error());
                }
            else
                {
                if(!mysql_select_db(DATABASE_NAME))
                    throw new Exception('Problem z wybraniem bazy danych.<br />Nr błędu: <b>'.mysql_errno().'</b><br /><b>Komunikat: </b>'.mysql_error()); 
                else
                    {
                    //connection successfull:
                    $this->handler = $sql_handler;
                    unset($sql_handler);
                    }
                }
            }
            
        //next four methods create sql query for doQuery method;
        public function selectQuery($fields, $table, $where = "", $order = "", $limit = "")
            {            
            //if is another query saved:
            if($this->query) unset($this->query);
            
            //creating query for doQuery() method
            $this->query = 'SELECT '.$fields.' FROM '.$table;
            if($where)
                $this->query .= ' WHERE '.$where;
            if($order)
                $this->query .= ' ORDER BY '.$order;
            if($limit)
                $this->query .= ' LIMIT '.$limit;
                
            $this->doQuery();
            $this->selected_rows = mysql_num_rows($this->results);
            }
            
        TUTAJ SĄ JESZCZE TRZY METODY DO USTAWIANIA ZMIENNEJ QUERY... (updateQuery, deleteQuery, insertQuery)
 
            
        //here is private method which will really make operations in db. it gets no arguments - all are written in object properties.
        
        private function doQuery()
            {
            if(!$this->results = mysql_query($this->query, $this->handler)) 
                {
                throw new Exception('Problem z wywołaniem zapytania do bazy danych.<br />Nr błędu: <b>'.mysql_errno().'</b><br /><b>Komunikat: </b>'.mysql_error());
                }
            }
        
        public function fetchResults()
            {
            if($this->results)
                {
                $this->records = mysql_fetch_array($this->results, MYSQL_ASSOC);
                
                if(is_array($this->records))
                    {
                    return $this->records; 
                    }
                }
            }
            
        public function getAffectedRows()
            {
            return $this->affected_rows;
            }
        public function getSelectedRows()
            {
            return $this->selected_rows;
            }
        public function getLastRowId()
            {
            return $this->last_row_id;
            }
        
        public function __destruct()
            {
            mysql_close($this->handler);
            }
        }
?>
[PHP] pobierz, plaintext 

[Crozin]

Albo przekazujesz obiekt klasy MysqlEng do obiektu klasy pagesSupport poprzez argument jakiejś metody/konstruktora:

[PHP] pobierz, plaintext 
<?php
 
require_once 'mysqleng.class.php';
require_once 'pagessupport.class.php';
 
$db = new MysqlEng();
$ps = new pagesSupport('jakies', 'arg.', $db);
?>
[PHP] pobierz, plaintext 

A w konstruktorze bądź innej metodzie:

[PHP] pobierz, plaintext 
<?php
public function __construct($abc, $def, MysqlEng $db){
  //...
  $this->db = $db; //$this->db to jakies pole prywatne
}
?>
[PHP] pobierz, plaintext 

Albo korzystając z Singletonu, Rejestru czy jeszcze czegos innego, byle nie globali.


EDIT:
I to o co pytasz to obiekt wewnątrz obiektu. PHP nie umożliwia zapiania klasy w klasie (nested classes).

Ten post edytował Crozin 28.12.2008, 13:08:43

[plurr]

takie rzeczy to tylko w javie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) A szkoda.

[LBO]

To o czym pisze kolega Crozin to tak zwane inversion of control.
TDzieki temu możesz uczynić kod bardziej uniwersalnym.

[PHP] pobierz, plaintext 
<?php
/*
 * Ogólny interfejs dla abstrakcji zapytań.
 *
 * Interfejs wymusza na klasie go implementującej posiadanie odpowiednich metod.
 *
 */
interface DatabaseQueries
{
    public function __construct(array $config);
 
    public function selectQuery();
 
    public function updateQuery();
 
    public function deleteQuery();
 
    public function  insertQuery();
}
 
// omijam z lenistwa argumenty dla metod :)
class MySqlQueries implements DatabaseQueries
{
    public function __construct(array $config)
    {
        // implementujesz łączenie się z bazą na podstawie tablicy $config dla MySQL
    }
 
    public function selectQuery()
    {
        // implementujesz zapytanie dla MySQL
    }
 
    public function updateQuery()
    {
        // implementujesz zapytanie dla MySQL
    }
 
    public function deleteQuery()
    {
        // implementujesz zapytanie dla MySQL
    }
 
    public function  insertQuery()
    {
        // implementujesz zapytanie dla MySQL
    }
}
 
// omijam z lenistwa argumenty dla metod :)
class SQLiteQueries implements DatabaseQueries
{
    public function __construct(array $config)
    {
        // implementujesz łączenie się z bazą na podstawie tablicy $config dla SQLite
    }
 
    public function selectQuery()
    {
        // implementujesz zapytanie dla SQLite
    }
 
    public function updateQuery()
    {
        // implementujesz zapytanie dla SQLite
    }
 
    public function deleteQuery()
    {
        // implementujesz zapytanie dla SQLite
    }
 
    public function  insertQuery()
    {
        // implementujesz zapytanie dla SQLite
    }
}
?>
[PHP] pobierz, plaintext 

a potem:

[PHP] pobierz, plaintext 
<?php
$config = array(
'host' => 'localhost',
'user' => 'username',
// etc
);
 
$db = new MysqlQueries($config);
$ps = new pagesSupport();
$ps->setDatabase($db);
?>
[PHP] pobierz, plaintext 

lub

[PHP] pobierz, plaintext 
<?php
$config = array(
'filename' => '/usr/data/database.sqlite3'
);
 
$db = new SQLiteQueries($config);
$ps = new pagesSupport();
$ps->setDatabase($db);
?>
[PHP] pobierz, plaintext 

I dzięki temu możesz w pewnym ograniczonym stopniu zmieniać bazę bez ingerencji w kod.

Jak widzisz użyłem metody (tzw setter), żeby przekazać klasę odpowiedzialną za zapytania na bazie danych. Jest to wygodniejsze od parametru w konstruktorze.

Albo korzystając z Singletonu, Rejestru czy jeszcze czegos innego, byle nie globali.

A powiedz czym się różni w użytkowaniu taki Rejestr od globali, prócz ujęcia w OOP?

[wlamywacz]

A powiedz czym się różni w użytkowaniu taki Rejestr od globali, prócz ujęcia w OOP?

Różni się tym że nie możesz go nadpisać, raz ustalony i tak zostaje do końca.

[LBO]

Różni się tym że nie możesz go nadpisać, raz ustalony i tak zostaje do końca.

Nie rozumiem. Chodzi Tobie o to, że nie można nadpisać obiektów, które trzyma? Bo jak tak to muszę Ciebie zmartwić - powszechne implementacje nie biorą Tego pod uwagę (patrz Zend_Registry) i szczerze mówiąc takie podejście nie wynika jakoś jasno z założen tego wzorca.

[wlamywacz]

Każdy dostosowuje wzorzec do swoich potrzeb, nie są one sztywno ustalone.

[LBO]

Każdy dostosowuje wzorzec do swoich potrzeb, nie są one sztywno ustalone.

1. Tym bardziej nie możesz tego używać jako argumentu.
2. Taki sposób nie jest zbyt wygodny wbrew pozorom. Zakładam, że nie robisz testów jednostkowych.
3.

[...] szczerze mówiąc takie podejście nie wynika jakoś jasno z założen tego wzorca.

[pinochet]

... nie potrafię z jednej klasy odwołać się do metod innej.

No niestety tak się czasem zdarza, że w schemacie klas są pomiędzy nimi strzałki :] I wtedy tak jak koledzy pokazali przekazujemy referencję do obiektu w konstruktorze lub innej meodzie importującej ( nie jest to oczywiście jedyne rozwiązanie ale jest najlepsze)

Co do rejestru ... chciałem tylko powiedzieć, że w zasadzie nie różni się od globali ale czy BD różni się od globali ? Nie - no właśnie dlatego tworzymy warstwę danych aby wszystko co z nimi związane było "gdzieś" a nie wszędzie :]

@plurr oprócz Javy na świecie jest jeszcze jeden język: C# (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) BTW PHP nie jest OO :[

[wrzasq]

@pinochet: jest wiele języków, a nested classes to nawet w C++ istnieją (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . taki offtopic mały jak już chcemy uściślać (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) .

[phpion]

BTW PHP nie jest OO :[

OO? Object Oriented? Jeśli tak to cię zmartwię: PHP, podobnie jak np. C++, jest językiem zorientowanym obiektowo, natomiast nie jest (jak np. Java) językiem obiektowym.

[LBO]

językiem zorientowanym obiektowo / językiem obiektowym.

To ja też Ciebie zmartwię - nie ma czegoś takiego jak język zorientowany obiektowo (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[phpion]

To ja też Ciebie zmartwię - nie ma czegoś takiego jak język zorientowany obiektowo (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

(IMG:http://forum.php.pl/style_emoticons/default/worriedsmiley.gif) jak to w takim razie jest? Mnie na uczelni uczono właśnie takiego rozgraniczenia.

// Edit:
Język zorientowany obiektowo - umożliwia pisanie kodu zarówno stukturalnie jak i obiektowo.
Język obiektowy - umożliwia pisanie kodu tylko obiektowo.
Tak mnie uczono...

Ten post edytował phpion 3.01.2009, 18:37:49

[LBO]

OO to jest paradygmat... natomiast języki mogą mieć typy np. obiektowy język albo język z elementami tegoż (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

edit:

Chcę tylko udowodnić, że nie warto się czepiać takich rzeczy - bo to normalne, że w naszym środowisku z czasem zaczynamy operować skrótami myślowymi.

Co złego to nie ja (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Alan

Ten post edytował LBO 3.01.2009, 18:42:58

[Tarcil]

Wielki dzięki (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Przepraszam, że tu nie zaglądałem, ciężki okres ostatnio w pracy miałem. Serdecznie pozdrawiam (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)