Klasa obiektu vs. klasa kolekcji obiektow Opcje

[little-did-he-kn...]

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!

[Strzałek]

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.

[Sedziwoj]

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:

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ł (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) )

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.