Forum PHP.pl

Witam,
Ostatnio nasunęło mi się pytanie odnośnie programowania obiektowego w PHP (a i chyba ogólnie).

Załóżmy że mamy plik index.php

[PHP] pobierz, plaintext 
$pages = new Pages();
$pages->funkcja1();
$pages->funckja2();
[PHP] pobierz, plaintext 

Dodatkowo jest klasa z formularzami:

[PHP] pobierz, plaintext 
class Forms{
  public function submit(){}
  public function inputText(){}
}
[PHP] pobierz, plaintext 

Teraz w funkcji1 chcemy stworzyć jeden formularz i w funkcji2 chcemy stworzyć jeden formularz.
czyli

[PHP] pobierz, plaintext 
class Pages{
  public function1(){
    $form = new Forms();
    // lecimy dalej z formularzem i treścią
  }
  public function2(){
    $form = new Forms();
    // dalej treści
  }
}
[PHP] pobierz, plaintext 

Można zrobić też w drugi sposób tzn index.php by wyglądał

[PHP] pobierz, plaintext 
$pages = new Pages();
$form = new Forms();
$pages->funkcja1();
$pages->funckja2();
[PHP] pobierz, plaintext 

a Pages:

[PHP] pobierz, plaintext 
class Pages{
  public function1(){
    global $form;
    // lecimy dalej z formularzem i treścią
  }
  public function2(){
    global $form;
    // dalej treści
  }
}
[PHP] pobierz, plaintext 

Przy czym z użytkowego punktu widzenia nie ma to żadnego znaczenia. W tej klasie nie ma żadnej zmiennej statycznej, którą można by wymieniać między poszczególnymi funkcjami. Chciałem się zapytać która z tych opcji jest właściwa i która należało by używać.

Nie lubie używać global w php, wiec w klasie Pages dodałbym pole typu Forms.

Przekazuj obiekt do klasy ktora ma go pozniej uzyc.

A mi wydaje się, że sposób pierwszy jest najlepszy. Nowy formularz = nowa instancja klasy.

A jak w drugiej funkcji chcesz użyć instancji, na której przed chwilą wykonywałeś jakieś operacje?

A no właśnie zależy od zamierzeń użycia.
1. Jeśli chce w każdej metodzie tworzyć nowy form i na nim operować.
2. Jeśli chce operować na jednym formie to wtedy przekazanie obiektu.

[PHP] pobierz, plaintext 
//1.
class Pages{
  public function1(){
    $form = new Forms();
    // lecimy dalej z formularzem i treścią
  }
  public function2(){
    $form = new Forms();
    // dalej treści
  }
}
 
//2.
class Pages{
  public function __construct(Forms $form){
    $this->form = $form;
  }
  public function1(){
     $this->form ='function1';
    // pierwsza oepracja na formularzu
  }
  public function2(){
    // druga operacja na tym samym formularzu
    if ($this->form == 'function1'){
        return 'byl edytowany w function1';
    }
  }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$pages = new Pages();
$form = new Forms();
$pages->form = $form;
$pages->funkcja1();
$pages->funckja2();
[PHP] pobierz, plaintext 

i później w klasie używasz $this->form zamiast global.

Jeżeli jesteś pewien że będziesz używać tylko jednego egzeplarza klasy Form, to skorzystaj z singleton.

[PHP] pobierz, plaintext 
class Form
{
    public static $instance;
    public function __construct()
    {
         self::$instance = $this;
    }
    public function getInstance()
    {
        if(!((object)self::$instance instanceof Form))
            new Form();
        return self::$instance;
    }
}
class Pages
{
    public function function1()
    {
        $form = Form::getInstace();
    }
    public function function2()
    {
        $form = Form::getInstance();
    }
}
[PHP] pobierz, plaintext 

I robisz:

[PHP] pobierz, plaintext 
new Form();
$page = new Pages();
$page->function1();
//itd...
 
[PHP] pobierz, plaintext 

singletonu używam do baz danych (logera itp), tutaj też jest możliwe aby go zastosować, w sumie to można by go zastosować przy każdej z klas (IMG:style_emoticons/default/smile.gif) .

Właśnie chciałem zobaczyć jak to robicie Wy i jakie są Wasze propozycje.

Jeśli mam w dwóch miejscach użyć tej samej zmiennej (są od siebie zależne) to musi być to ta sama klasa, w tym klasach o których mi się rozchodzi, nie ma takiej potrzeby są to tylko generatory tabel, formularzy, rzeczy od siebie nie zależnych.

Jeśli chodzi o przekazywanie obiektu i singleton, to która z nich jest szybsza? bo jeśli chodzi o miejsce w pamięci przy przetwarzaniu skryptu, raczej będzie takie samo.

Ten post edytował grzemach 19.08.2009, 11:03:25

(IMG:style_emoticons/default/sciana.gif) (IMG:style_emoticons/default/sciana.gif) (IMG:style_emoticons/default/sciana.gif) (IMG:style_emoticons/default/sciana.gif)

Nie potrzebujesz tu singletonu.
Rozwiazanie zalezy od tego co ty robisz, czym jest Page, czym Form (po co one sa, co robia, co reprezentuja), jak ze soba wspolpracuja itd.
Bez tej wiedzy nic dobrego nie stworzysz.

Dependency Injection (@Fafu podał to rozwiązanie) - klik

Poprostu przekazujesz referencję obiektu (Form) do innego obiektu (Pages) poprzez konstruktor, metodę dostępową (setter, np. setForm()) lub też publiczną składową (odradzane).

@deirathe - ten Twój singleton jest trochę kulawy (publiczny konstruktor, nie będe się już czapiał możliwości klonowania bo przykład miał zapewne być okrojony, ale lepiej zmień modyfikator przy konstruktorze z public na private/protected)

Dlaczego przez publiczną składową odradzane?

Żadna klasa nie powinna mieć publicznych składowych, powód - hermetyzacja danych, czyli jedna z podstawowych zasad OOP. Są również inne argumenty na to, ale nie chce mi się ich przytaczać, bo by wyszedł z tego przydługawy post, którego pisać mi się nie chce (IMG:style_emoticons/default/smile.gif)