Forum PHP.pl

Witam,

Mam pytanie o różnice między takimi dwoma rozwiązaniami, chodzi o konstruktory.

Rozwiązanie 1

[PHP] pobierz, plaintext 
class Osoba {
 
private $imie;
private $nazwisko;
private $www;
private $email;
private $telefon;
 
function __construct($imie, $nazwisko, $www, $email, $telefon){
 
	$this->imie = $imie;
	$this->nazwisko = $nazwisko;
	$this->www = $www;
	$this->email = $email;
	$this->telefon = $telefon;
 
}
 
function dodajOsobe($imie, $nazwisko, $www, $email, $telefon) {			
 
	$db = new Database();
	$db -> DBConnect(HOSTNAME, USERNAME, PASSWORD, DB);
	$db -> DBQuery('INSERT INTO osoba SET
					imie=\''.$imie.'\',
					nazwisko=\''.$nazwisko.'\',
					www=\''.$www.'\',
					email=\''.$email.'\',
					telefon=\''.$telefon.'\';');
 
	$db -> DBClose();
}
 
}
[PHP] pobierz, plaintext 

W tym wypadku jeżeli mamy konstruktor to aby utworzyć obiekt trzeba zrobić tak:

[PHP] pobierz, plaintext 
$imie = "Jan";
$nazwisko = "Testowy";
$www = "http://www.test.pl";
$email = "osoba@test.pl";
$telefon = "666777888";
 
$dodaj_osoba = new Osoba($imie, $nazwisko, $www, $email, $telefon);						
$dodaj_osoba -> dodajOsobe($imie, $nazwisko, $www, $email, $telefon);
[PHP] pobierz, plaintext 

Rozwiązanie 2

[PHP] pobierz, plaintext 
class Osoba {
 
function dodajOsobe($imie, $nazwisko, $www, $email, $telefon) {			
 
	$this->imie = $imie;
	$this->nazwisko = $nazwisko;
	$this->www = $www;
	$this->email = $email;
	$this->telefon = $telefon;
 
	$db = new Database();
	$db -> DBConnect(HOSTNAME, USERNAME, PASSWORD, DB);
	$db -> DBQuery('INSERT INTO osoba SET
					imie=\''.$imie.'\',
					nazwisko=\''.$nazwisko.'\',
					www=\''.$www.'\',
					email=\''.$email.'\',
					telefon=\''.$telefon.'\';');
 
	$db -> DBClose();
}
 
}
[PHP] pobierz, plaintext 

Tworzymy obiekt:

[PHP] pobierz, plaintext 
$imie = "Jan";
$nazwisko = "Testowy";
$www = "http://www.test.pl";
$email = "osoba@test.pl";
$telefon = "666777888";
 
$dodaj_osoba = new Osoba();						
$dodaj_osoba -> dodajOsobe($imie, $nazwisko, $www, $email, $telefon);
[PHP] pobierz, plaintext 

Moje pytania są następujące:

1. Które rozwiązanie jest bardziej poprawne?
2. Jaka jest różnica między tymi dwoma rozwiązaniami?

Będę bardzo wdzięczny jeżeli ktoś znajdzie chwilkę na wytłumaczenie mi różnic i poprawności rozwiązania.

Z góry bardzo dziękuję i pozdrawiam.
Kazik.

Oba rozwiązania są. kiepskie.
w konstruktorze zrób połączenie z bazą, a w destruktorze rozłącz się z bazą.
Krótko i na temat.

To ja chyba nie bardzo rozumiem po co jest konstruktor, jeżeli ma być w nim tylko połączenie z bazą.
Wiem, że to pytanie jest banalne, ale czy możesz mi to wytłumaczyć łopatologiczne?

Dziękuję,
Kazik.

Konstruktor jest po to abyś zadeklarował coś co będzie wykorzystywane w wszystkich/większości metodach
czyli deklarujesz w nim np. $this->db=new database();
i w metodach typu add(), delete(), get() odnosisz się do $this->db zamiast robić 3 razy $db=new database();

W sumie ma to sens (IMG:style_emoticons/default/winksmiley.jpg)

Dziękuję za pomoc.

Pozdrawiam,
Kazik.

http://en.wikipedia.org/wiki/Fluent_interface

[PHP] pobierz, plaintext 
<?php
class Osoba
{
	private $imie;
	private $nazwisko;
	private $www;
 
	public function getImie() { return $this->imie; }
	public function getNazwisko() { return $this->nazwisko; }
	public function getWww() { return $this->www; }
 
	public function setImie($imie) { $this->imie = $imie; return $this; }
	public function setNazwisko($nazwisko) { $this->nazwisko = $nazwisko; return $this; }
	public function setWww($www) { $this->www = $www; return $this; }
}
 
$osoba = new Osoba();
$osoba->setImie('Jan')->setNazwisko('Kowalski')->setWww('www.JanKowalski.pl');
 
$osoba->setImie('Jan')
             ->setNazwisko('Kowalski')
             ->setWww('www.JanKowalski.pl');
?>
[PHP] pobierz, plaintext 

oraz do tego co już napisał starach:

[PHP] pobierz, plaintext 
 
class OsobaService
{
   // metody laczace z db itp 
 
    public function save(Osoba $osoba) 
    {
        // dodaj do obiektu osoba ID 
        // sprawdzaj czy obiekt $osoba posiada ID - sprawdzaj czy obiekt istnieje w bazie. 
        // jesli obiekt istnieje to rob update, w przeciwnym wypadku insert
    }
}
 
$osoba = new Osoba();
$osoba->setImie('Jan')
             ->setNazwisko('Kowalski')
             ->setWww('www.JanKowalski.pl');
 
$osobaService = new OsobaService(); 
$osobaService->save($osoba);
 
[PHP] pobierz, plaintext 

Nie, nie od tego jest (patrz pkt 1).

1. Do konstruktorów przekazujemy tylko i wyłącznie dane potrzebne do działania obiektu. W przypadku takiego obiektu jak Osoba takie dane jak adres email czy strona www nie są potrzebne do jego działania. Ba! Imię i nazwisko nie jest potrzebne.
2. Jeden obiekt, jedno zadanie: ten obiekt ma reprezentować jakąś osobę, więc niech zajmuje się tylko tym. Takie rzeczy jak zapisywanie danych w bazie danych na podstawie tego obiektu pozostaw innym obiektom (patrz: post @plurr-a). Osobiście uważam, że ActiveRecord (czyli to co zaimplementowałeś) nie jest zbyt dobrym podejściem.