Forum PHP.pl

Czesc.

Mam dylemat zwiazany z budowa klasy obiektow i jej klasy kolekcji.
Zalozmy, ze obiektami sa goscie (Guests)

Obecnie robie to tak:

1. Klasa pojedynczego obiektu

[PHP] pobierz, plaintext 
<?php
class Guest {
 
	private $id;
	private $name;
	private $email;
	private $password;
	private $errors = array();
 
	function __construct($id) {
		global $dbm;
 
		if (is_array($id)) {
			$this->id=( !empty($id['ProfileId']) ? $id['ProfileId'] : 0 );
			$this->name=( !empty($id['Name']) ? $id['Name'] : $id['Email']);
			$this->email=$id['Email'];
			$this->password=md5($this->email.microtime());
		}	 
	}
 
	function validate() {
 
		if ( !preg_match('/b[A-Z0-9._%+-]+@[A-Z0-9.-]+.[A-Z]{2,4}b/',strtoupper($this->email)) ) {
			$this->errors[] = "nieprawidlowy adres e-mail!";
		} 
 
		return count($this->errors) == 0;
	}
 
		function __toString() {
		 //..
		  }
	 
	function getErrors() {
		return $this->errors;
	}
 
 
	function setId($id) {
		$this->id = $id;
	}
 
	function getEmail() {
		return $this->email;
	}
 
	function getPassword() {
		return $this->password;
	}
}
?>
[PHP] pobierz, plaintext 

2. Klasa kolekcji obiektow:

[PHP] pobierz, plaintext 
<?php
class GuestList {
  private $aList = array();
 
  function add(Guest $g) {
   //...
  }
 
  function delete(Guest $g) {
   //...
  }
 
  function getOne() {
 
  }
 
  function getById($id) {
 
  }
 
  function getAll() {
 
  }
 
 
}
?>
[PHP] pobierz, plaintext 

Teraz pytania:

1. Metoda walidujaca powinna byc w klasie Goscia czy klasie kolekcji Gosci czy nie ma to znaczenia?
2. Dodanie nowego goscia powinno zostac poprzedzone utworzeniem obiektu klasy kolekcji czy tez tak jak ponizej wywolane bez kreacji obiektu?
3. Przekazanie obiektu przez referencje "&" jest zawsze prawidlowym rozwiazaniem?

[PHP] pobierz, plaintext 
<?php
$aGuest = new Guest($arr);
 
if ( $aGuest->validate() ) {
	GuestList::add(&$aGuest);
	echo $aGuest ;
 
}
?>
[PHP] pobierz, plaintext 

4. Kasujac obiekt powinienem tworzyc instancje obiektu kolekcji i samego obiektu goscia jak ponizej, czy tez wywolac ja bezposrednio?:

[PHP] pobierz, plaintext 
<?php
$aGuestList = new GuestList();
$aGuest = $aGuestList -> getOne($pId);
$aGuestList->delete(&$aGuest);
?>
[PHP] pobierz, plaintext 

Z gory dziekuje za merytoryczne odpowiedzi!

Ja to widzę trochę inaczej.

Najpierw sprawdzamy co mamy. Walidujemy to i dopiero jak się wszystko zgadza to tworzymy obiekt Guest i przechowujemy w nim Jana Kowalskiego. Następnie chcemy stworzyć kolekcje gości:

[PHP] pobierz, plaintext 
<?php
$collection = new GuestsCollection();
 
 foreach($guests as $guest){
   $collection -> add($guest);
 }
 
$collection -> remove('Kowalski');
?>
[PHP] pobierz, plaintext 

Metodę remove można zaimplementować różnie, wg. potrzeb.

ad 1. Uważam, że wszelkie metody walidujące powinny być w klasie Guest, ponieważ może przyjść w przyszłości ochota na stworzenie nowego obiektu typu Guest poza kolekcją, wtedy taką metodę już będziesz "miał pod ręką"

ad 2. Tu wedle uznania, ja osobieście nie tworzyłbym obiektu kolekcji, tylko odwoływałbym się dokładnie tak jak pokazałeś w przykładzie (przez metody statyczne)

ad 3. W PHP5 zawsze jest wszystko przekazane przez referencję, więc '&' jest niekonieczne i nie musisz tego używać. Jeśli chcesz pracować na kopi obiektu używaj odwrotnego operatora do '&', czyli clone

ad 4, Jak już masz kolekcję to lepiej użyć kolekcji i w niej zaimplementować fukcję kasującą, tam dobrze jest korzystać z unset()

Pozdrawiam

Ten post edytował Cezar708 17.03.2008, 16:25:35

Jeśli to co zapisałeś jest tak jak sądzisz, to masz z teka dziwne podejście (powiedział bym że utrudniające życie), bo to obiekt Guest powinien weryfikować to co mu się daje, zazwyczaj jest jakaś filtracja danych z formularzy, ale to też nie powinno być nic specjalnego.
Kapsułkować kod potrzebny do wykonania pewnych rzeczy, czyli po co ma to robić i wiedzieć co ma robić coś poza obiektem Guest?

(o widzę że Cezar708 to też napisał )

Co do kolekcji, to wszystko czego używamy ma czemuś służyć, nikt nie wynajmuje autobusu, aby gdzieś pojechać kawałek, tylko zadzwoni po taksówkę. Więc nie ma co tworzyć kolekcji dla jednego obiektu.