[PHP]Wywoływanie klasy w klasie Opcje

[sannin]

Witam,

dopiero zaczynam swoją zabawę z obiektowym php. Teraz mam pytanie do bardziej zaawansowanych koderów czy moje rozwiązanie jest dobre. Więc tak mam folder gdzie tworzę pliki z klasami dajmy 'Mysql.class.php'. W pliku library.php ładuje wszystkie klasy

[PHP] pobierz, plaintext 
<?php
require('Mysql.class.php');
     require('Main.class.php');
?>
[PHP] pobierz, plaintext 

w pliku system.php ładuje library.php i config.php Natomiast w pliku index.php mam

[PHP] pobierz, plaintext 
<?php
require('config/system.php'); 
     
     $mysql = new MySql;
     $mysql->Connect($CONFIG['dbHost'], $CONFIG['dbUser'], $CONFIG['dbPassword'], $CONFIG['dbName']);
     
 ...
 
     $index = new Main();
     $index->prace());
?>
[PHP] pobierz, plaintext 

Teraz żeby uzyskać dostęp do mysql w klasie Main, metodzie prace mam coś takiego

[PHP] pobierz, plaintext 
<?php
function prace()  {    
             global $mysql;
             
             $job = $mysql->selectToArray("tabela", "id, name", "id > 3", "");
             foreach($job['name'] as $value) {
                 echo $value.'
     ';
             }
         }
?>
[PHP] pobierz, plaintext 

Czy ogólnie jest to poprawne podejście? Czy raczej wystrzegać się global? Jak to inaczej rozwiązać? Proszę mi nie proponować Zend'a itd. bo najpier chciałbym liznąć troszkę obiektówki (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam i z góry dziękuje za pomoc.

Powód edycji: Proszę używać tagu [PHP] do umieszczania kodu PHP. Dodane także tagi do tematu.

[erix]

Czy ogólnie jest to poprawne podejście? Czy raczej wystrzegać się global?

Się wystrzegać. Do tego zadania doskonale sprawdzi się singleton.

[l0ud]

Się wystrzegać. Do tego zadania doskonale sprawdzi się singleton.

...który tylko 'ładniej' wygląda. (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Utwórz klasę łączącą wszystkie (zawierającą w sobie tablicę wszystkich obiektów) i przekazuj jej instancję do wszystkich nowo utworzonych obiektów. Musi zawierać metody, dzięki którym pozwoli na dostęp do obiektów z zewnątrz, oraz najlepiej od razu je wczytywać.

Jakiś uproszczony przykład:

index.php

[PHP] pobierz, plaintext 
<?php
include ('includes/core.php');
 
$core = new core;
 
$core->initComponents(); //tutaj można dodać obsługę ew. wyjątków
?>
[PHP] pobierz, plaintext 

core.php

core.php

[PHP] pobierz, plaintext 
<?php
class core {
   private $obj = array();
 
   public function __get($objName) {
      if (isset($this->obj[$objName])) return $this->obj[$objName];
   }
 
   public function load($className) {
      include('includes/'.$className.'.php');
      $this->obj[$className] = new $obj($this);
   }
 
  public function initComponents() {
     $this->load('mysql');
      $this->load('config'); //itd
  }
 
}
?>
[PHP] pobierz, plaintext 

przykładowe config.php

[PHP] pobierz, plaintext 
<?php
class config {
   private $db;
   //inne obiekty, których tutaj potrzebujemy
   public function __construct(core $core) {
      //wczytywanie tych obiektów do klasy
      $this->db = $core->db;
      //itd.
   }
   public function action() { //i jakaś tam akcja tej klasy
      $this->db->query('zapytanie'); 
   }
}
?>
[PHP] pobierz, plaintext 

Później jeszcze możesz dodać metodę startAction() do core, które wczyta właściwy plik, który będzie coś robił (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Plik zostanie wczytany wewnątrz klasy core, a więc do obiektów będziesz się odnosił tam przez $this.

[erix]

Uzasadnij, w jaki sposób Twoje rozwiązanie jest lepsze od singletona?

[sannin]

Już mniej lub więcej rozumiem o co chodzi ;p Teraz mam pytanie czy stosować new czy coś takiego

Kod

public function singleton()
    {
       static $instance;
       if(!isset($instance)) {
          $instance = new test5;
       }
       return $instance;
    }

i później

Kod

$single1 = test5::singleton();

[Mize]

Takie, że przykładowo w Widoku nie masz dostępu do wszystkich komponentów.

[erix]

Jakoś nigdy nie miałem tego typu problemów z singletonem.

[l0ud]

Uzasadnij, w jaki sposób Twoje rozwiązanie jest lepsze od singletona?

Tak jak chociażby Mize powiedział, nie mogę np. w widoku wykonywać zapytań do bazy danych (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) Oczywiście o ile odpowiednio go obsłużę. No i wszystko mam w jednym miejscu, co daje pewien porządek.
Równie dobrze mógłbym spytać Ciebie, w czym, poza niemożnością nadpisania lepszy jest ten singleton od globala? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[sannin]

Trochę poczytałem, nie wiem czy zrozumiałem ;d ale zrobiłem tak:

do każdej klasy dodałem na początku

Kod

private static $objInstance = null;
     private $intValue = null;
     ########################################################
     private function __construct() {
         $this->intValue = rand(1, 100);
     }
     ########################################################
     public static function create() {
         if (is_null(self::$objInstance)) {
             self::$objInstance = new self();
         }
         return self::$objInstance;
     }
     ########################################################

Później wywołuje to tak

Kod

$Mysql = MySql::Create();
$Mysql->Connect($CONFIG['dbHost'], $CONFIG['dbUser'], $CONFIG['dbPassword'], $CONFIG['dbName']);
$Mysql->metoda();

W klasach muszę natomiast robić tak

Kod

MySql::selectToArray()

Czy może mi to kotś wytłumaczyć (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ?

[erix]

Tak jak chociażby Mize powiedział, nie mogę np. w widoku wykonywać zapytań do bazy danych

Yyyyy... W widoku zapytania do bazy...? My mamy nadal na myśli MVC, czy mam w czymś braki?

No i wszystko mam w jednym miejscu, co daje pewien porządek.

No z tym się zgodzę.

poza niemożnością nadpisania lepszy jest ten singleton od globala?

Właśnie ta niemożność nadpisania. (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) Poza tym, łatwiej jest znaleźć deklarację obiektu, do którego się singleton odnosi. A w przypadku global szukaj wiatru w polu.

Osobiście korzystam z pewnego rodzaju "hybrydy"; wyznaję zasadę, że co jest wygodniejsze+wydajniejsze w użyciu. (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[sannin]

Czy jest ktoś w stanie mi pomóc zamiast się wykłócać?

[erix]

W klasach muszę natomiast robić tak

A zwykła konstrukcja create nie działa...? Nie wiem, jaką masz deklarację selectToArray...

PS. Poprosiłem już o wydzielenie wątku, jak coś.

[sannin]

Działa, tylko że wtedy muszę od nowa łączyć się z mysql. Chciałbym, zeby coś takiego w pliku system.php

Kod

require('library.php');
require('config.php');
$mysql = MySql::Create();
$mysql->connect($CONFIG['dbHost'], $CONFIG['dbUser'], $CONFIG['dbPassword'], $CONFIG['dbName']);

Załatwiało sprawę połączenia. Później daje tylko $index->prace() w której mam selectToArray

Później mam wykonywać metody MySql::metoda czy $mysql->metoda ?

Ten post edytował sannin 20.11.2008, 22:59:38

[erix]

Później $mysql->metoda. Z dwukropkiem wywołujesz tylko metody oznaczone frazą static" title="Zobacz w manualu PHP" target="_manual w deklaracji.

Troszkę mało informacji podałeś, np. jak wywołujesz selectToArray.

[sannin]

Poczytałem i wydaje mi się, że już to łapie... Dosyć pomocy był ten wpis http://athlan.pl/singleton-registry-map/ tylko autor pisze tutaj, że singleton nie jest najlepszym rozwiązaniem, ale przeszukują fora widzę, że jest to kwestia bardzo sporna które rozwiązanie jest najlepsze... Robię tak... do każdej klasy dodaje

Kod

private function __construct() {
         }
         ########################################################
         public static function instance() {
             if(!self::$_oInstance instanceof self) {
                 self::$_oInstance = new self;
             }        
             return self::$_oInstance;
         }

Później wystarczy tylko

Kod

$mysql = MySql::instance();
             $zmienna = $mysql->metoda()

Z tego co rozumiem to MySql::instance(); Utworzy obiekt, a jeśli istnieje to go zwróci, dzięki temu mogę działać tą klasą po całym "systemie" bez obaw i nie muszę się martwić nadpisaniem i mam pewność, że zawsze korzystam z tego samego wywołania obiektu. Czy dobrze to rozumiem i czy jest to dobre (nienajlepsze) rozwiązanie na używanie obiektowego php? Odchodząc od tematu mam jeszcze pytanie co do config.php w którym są przetrzymywane zmienne systemowe. Czy wstawiać tam stałe, żeby były dostępne wszędzie czy zmienne i wczytywać w funkcji jak globalne? Czy może jeszcze jakoś inaczej?

[erix]

tylko autor pisze tutaj, że singleton nie jest najlepszym rozwiązaniem, ale przeszukują fora widzę, że jest to kwestia bardzo sporna

Między innymi dlatego wynikł offtop w tym temacie. ;P

Z tego co rozumiem to MySql::instance(); Utworzy obiekt, a jeśli istnieje to go zwróci, dzięki temu mogę działać tą klasą po całym "systemie" bez obaw i nie muszę się martwić nadpisaniem i mam pewność, że zawsze korzystam z tego samego wywołania obiektu.

Czyli dobrze zrozumiałeś. (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Czy wstawiać tam stałe, żeby były dostępne wszędzie czy zmienne i wczytywać w funkcji jak globalne? Czy może jeszcze jakoś inaczej?

Hmm, to kwestia tak samo sporna, jak wyżej. Choć powstały pewne niepisane zasady, jak np. trzymanie ścieżek do katalogów aplikacji w stałych, itp. Przejrzyj sobie źródła paru skryptów (mam na myśli skryptów, a nie jakichś beznadziejnych "systemów" newsowych pisanych przez samozwańczych "programistów"), to sam wyciągniesz wnioski.

[l0ud]

Odchodząc od tematu mam jeszcze pytanie co do config.php w którym są przetrzymywane zmienne systemowe. Czy wstawiać tam stałe, żeby były dostępne wszędzie czy zmienne i wczytywać w funkcji jak globalne? Czy może jeszcze jakoś inaczej?

Jeżeli chodzi o mnie, to w takim pliku definiuję kilka stałych, które są niezbędne do zainicjowania właściwego obiektu, który zajmuje się konfiguracją i wczytuje wartości z bazy oraz które wystarczą do w miarę eleganckiego obsłużenia błędu, jeżeli coś się posypie. Z reguły są to dane do bazy oraz ścieżki do poszczególnych folderów. Czasem umieszczę jakąś wartość, której nie planuję zmieniać przez panel administracyjny, a która może się przydać przy późniejszej przebudowie skryptu. Takie rzeczy jednak, jak np. ilość elementów na stronie (do paginacji) trzymam już w bazie, z możliwością zmiany w PA (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[sannin]

Takie rzeczy jednak, jak np. ilość elementów na stronie (do paginacji) trzymam już w bazie, z możliwością zmiany w PA (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Tak robię, bardziej chodziło mi o adres email admina itd. W sumie mam taki pomysł na to, w pliku config.php trzymam tylko dane do połączenie z mysql. Tworzę sobie klasę o nazwie Config która będzie pobierała z mysql wszystkie inne wartości. Dzięki temu będę miał do nich dostęp wszędzie i możliwość edycji z PA. Czy jest to dobre rozwiązanie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ?

Ten post edytował sannin 21.11.2008, 23:57:18

[erix]

To już jak Tobie wygodniej. (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Choć dobrym sposobem jest inicjowanie połączenia bezpośrednio z pliku konfiguracyjnego, gdyż dane do SQL nie są przechowywane w zmiennych i nie można ich już tak łatwo odczytać.

[adek140]

core.php

[PHP] pobierz, plaintext 
<?php
class core {
   private $obj = array();
 
   public function __get($objName) {
      if (isset($this->obj[$objName])) return $this->obj[$objName];
   }
 
   public function load($className) {
      include('includes/'.$className.'.php');
      $this->obj[$className] = new $obj($this);
   }
 
  public function initComponents() {
     $this->load('mysql');
      $this->load('config'); //itd
  }
 
}
?>
[PHP] pobierz, plaintext 

12 linijka w pliku powyżej powoduje błąd (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)
Fatal error: Class name must be a valid object or a string in H:\Serwer\tescik\includes\core.php on line 11

jakaś wskazówka?
tutaj pliczki http://wyslijto.pl/plik/u0rcn5sg8s