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

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

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

...który tylko 'ładniej' wygląda.

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ł Plik zostanie wczytany wewnątrz klasy core, a więc do obiektów będziesz się odnosił tam przez $this.

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

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

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

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

Tak jak chociażby Mize powiedział, nie mogę np. w widoku wykonywać zapytań do bazy danych 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?

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

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



Później wywołuje to tak

W klasach muszę natomiast robić tak

Czy może mi to kotś wytłumaczyć ?

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


No z tym się zgodzę.


Właśnie ta niemożność nadpisania. 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.

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

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

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

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



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 ?

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.

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

Później wystarczy tylko 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?

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


Czyli dobrze zrozumiałeś.


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.

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

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 ?

To już jak Tobie wygodniej.

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ć.

12 linijka w pliku powyżej powoduje błąd
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

No tak. A gdzie deklarujesz zmienną $obj?

Jeżeli Ci to pomoże, mam coś podobnego w swoim prototypowym widoku:

[PHP] pobierz, plaintext 
<?php
public function __get($name) {
            if ($this->modules[$name]) {
                return $this->modules[$name];
            }else{
                return false;
            }
        }
 
        public function _load($name) {
            if ($this->$name) {
                return $this->$name;
            }else{
                $temp = __CLASS__ .'_'. $name; 
                if (file_exists('modules/'.$temp.'.php')) {
                    include_once('modules/'.$temp.'.php');
                    return $this->$name = new $temp;
                }else{
                    return false;
                }
            }
        }
?>
[PHP] pobierz, plaintext 

Oczywiście to wycięty fragment.
Uznałem, że podzielenie widoku na moduły odpowiadające np. za szablony, bbcode, formularze będzie dobrym rozwiązaniem. (mam wszystko ładnie posegregowane i wiem co do czego jest)

Sam plik z tą klasą nazywa się view.php
A moduły przykładowo: view_template.php (który zawiera klasę o takiej samej nazwie)

Działa i powiem, że dość fajne rozwiązanie na które po części naprowadził mnie Erix (mój pro ) .

niestety wciąż nie potrafie sobie poradzić

przedstawie wszystko od początku

index.php

[PHP] pobierz, plaintext 
<?php
 
   include ('includes/core.php');
 
   $core = new core;
   $core->initComponents();
 
   $core->config->test();
 
 
?>
[PHP] pobierz, plaintext 

includes/core.php

[PHP] pobierz, plaintext 
<?php
 
class core
   {
 
      public function _load($name) 
      {
        include_once('includes/'.$name.'.php');
        $this->$name = new $name($core);
      }
 
      public function initComponents()
      {
        $this->_load('config');
        $this->_load('maile');
      }
   }
 
?>
[PHP] pobierz, plaintext 

includes/config.php

[PHP] pobierz, plaintext 
<?php
class config {
 
   private $maile;
 
   public function __construct($core) {
      $this->maile = $core->maile;
   }
 
   public function test() {
     echo 'krok 1 z klasy config <br>';
     $this->maile->test2();
   }
}
?>
[PHP] pobierz, plaintext 

includes/maile.php

[PHP] pobierz, plaintext 
<?php
class maile
{
   private $config;
 
   public function __construct($core)
   {
       $this->config = $core->config;
   }
 
   public function test2()
   {
        echo 'krok 2 z klasy maile';
   }
}
?>
[PHP] pobierz, plaintext 

efekt



mógłby mi ktoś podpowiedzieć gdzie popełniam błąd? domyślam się, że w którymś miejscu przy przekazywaniu obiektu core, ale, jak pisałem, jestem początkujący i nie potrafię sam sobie z tym poradzić, mimo wielu prób metodą 'prób i błędów'

pozdrawiam