Jak inicjować obiekt, gdy nie wiemy czy będzie łączył się z bazą Opcje

[Staszek27]

Cześć!
Problem jest następujący:

[PHP] pobierz, plaintext 
class Wataha {
 private $zbior = array();
 private $PDO;
 
 public function ustawPolaczenie(PDO $PDO) {
  $this->PDO = $PDO;
 }
 
 public function stworz($rasa) {
  //pobieramy psy z bazy danych razem z ich opisami, wpierw sprawdzając czy mamy psy danej rasy. załóżmy, że mamy
  foreach($wynikZapytania as $wiersz) {
    $Pies = new Pies($wiersz['imie'], TRUE); //jedynką informujemy psa, że on naprawdę istnieje
    $Pies->ustawUmaszczenie($wiersz['umaszczenie']);
    $this->zbior[] = $Pies;
   }
 }
}
[PHP] pobierz, plaintext 

No i teraz bez problemu w kontrolerze:

[PHP] pobierz, plaintext 
// laczymy sie z baza itp.
$Model = new Wataha();
$Model->ustawPolaczenie($this->PDO);
[PHP] pobierz, plaintext 

przesyłamy model do widoku, ten go obrabia i wszystko śmiga. Problem powstaje gdy nakażemy kontrolerowi pobrać jednego, konkretnego psa. Normalnie, gdy tworzymy sobie kolekcję psów, konkretny pies nie musi sprawdzać czy to konkretnie on istnieje (po co pies miałby pytać bazę czy rzeczywiście istnieje, skoro przywołujemy go do życia danymi pobranymi właśnie z tej bazy?). Natomiast gdy chcemy powołać do życia jednego psa musimy sprawdzić czy on rzeczywiście sobie tam siedzi (np. użytkownik dopuszcza się literówki)!

[PHP] pobierz, plaintext 
class Pies {
 private $imie;
 private $umaszczenie;
 
 public function __construct($imie, $sprawdzony = FALSE) {
  if($sprawdzony) {
   $this->imie = $imie;
   return TRUE;
  }
   //pytamy bazę czy taki pies się w niej znajduje
 }
 public function ustawUmaszczenie($umaszczenie) {
   //ustawiamy umaszczenie
 } 
}
[PHP] pobierz, plaintext 

i teraz gdzieś w kontrolerze:

[PHP] pobierz, plaintext 
$Model = new Pies($imie); // drugi parametr jest równy zero, więc pies upewni się czy może istnieć
[PHP] pobierz, plaintext 

No i widzimy, że ten kod nie ma racji bytu, ponieważ pies nie ma dostępu do połączenia z bazą danych. Kontroler musi mu je jakoś przesłać. Oto garść moich pomysłów, każdy mi nie odpowiada:
1) Na konstruktor zrzucamy jedynie odpowiedzialność za dowiedzenie się czy trzeba będzie sprawdzać swoje istnienie i ew. kontroler mu połączenie. Jeśli zajdzie potrzeba już konkretną metodą proponujemy mu imię a on albo przyjmuje je bezdyskusyjnie albo je weryfikuje.
2) Pozostawiamy konstruktor pusty i w zależności od potrzeb wykonujemy wszystkie zadania ?ręcznie?.
W obu rozwiązaniach pies nie wie od razu jak się nazywa (dopiero potem proponujemy mu imię), a to chyba nie jest zbyt dobry pomysł (a może jest? tego nie wiem)? Można także:
3) Pies jest bardzo nieufny i niezależnie od wszystkiego sprawdza swoje imię. Jeśli tworzymy kolekcję 100. psów, to 100 na 101 zapytań jest całkowicie niepotrzebnych (sprawdza poprawne dane).
4) Konstruktor psa odwołuje się do prywatnej metody, która pobiera z kontrolera połączenie z bazą. Wtedy każda klasa mogłaby sobie pytać bazę, wystarczyłoby że pobierze z kontrolera dane o połączeniu. Równie dobrze mógłbym użyć singletona lub zmiennej globalnej, a tego chciałbym uniknąć?
W jaki sposób wybrnąć z tego problemu? Chodzi mi o rozwiązanie, które ma ?ręce i nogi? (tutaj raczej: cztery łapy), a o takie chyba nietrudno, bo problem wydaje się dość częsty, tylko trzeba go wpierw choć raz rozwiązać a z tym mam problem?

Ten post edytował Staszek27 25.07.2010, 20:43:12

[jarexx]

A ja jeśli nie używam frameworka korzystam z metod statycznych.

Do połączenia z bazą klasa Connection ze statyczną metodą getConnection()

[PHP] pobierz, plaintext 
class Connection
{
  private static $DbConnect;
  private function __construct() {}
 
  public static function getConnection()
  {
      if(empty(self::$DbConnect))
      {
          self::$DbConnect = mysql_connect("host","username","haselko");
          mysql_select_db("nazwabazy", self::$DbConnect) or die ("Brak bazy<BR>\n" . mysql_error());
      }
      return self::$DbConnect;
  }
}
[PHP] pobierz, plaintext 

Teraz przykladowa klasa Dog. Konstruktor jako parametr potrzebuje id_dog, a nastepnie statyczna metoda getDog() pobiera sobie z bazy dane psa i przypisujemy je potem do własciwości

[PHP] pobierz, plaintext 
class Dog
{
	protected $id_dog;
	protected $name;
 
    public function __construct($id_dog)
    {
     $data = Database::getDog($id_dog); 
	 $this->id_dog = $id_dog;
	 $this->name = $data['NAME'];
    } // end func
 
	//tu klasycznie -settetery i gettery
 
} // end class
[PHP] pobierz, plaintext 

Dla wygody zawsze używam sobie oddzielnego pliku do obsługi zapytań do bazy.

[PHP] pobierz, plaintext 
class Database
{
    public static function getDog($id_dog)
    {
        $m = "SELECT `ID`, `NAME` FROM DOGS WHERE ID=".$id_dog."";
        $result = mysql_query($m, Connection::getConnection()) or die ("$m<BR>\n" . mysql_error());
        return mysql_fetch_assoc($result);
 
    } // end func
 
	public static function getDogs()
	{
	    $m = "SELECT ID FROM DOGS";
	    $result = mysql_query($m, Connection::getConnection()) or die ("$m<BR>\n" . mysql_error());
	    while($aw = mysql_fetch_assoc($result)) {
	    $tab[] =new Dog($aw['ID']);
	    }
	    return $tab;
	} // end func
 
} // end class
[PHP] pobierz, plaintext 

I teraz gdy np. chce pobrac dane konkretnego psa to robie po prostu

[PHP] pobierz, plaintext 
$dog = new Dog($_POST['id_dog']);
echo $dog->getName();
[PHP] pobierz, plaintext 

Mogę też w prosty sposób pobrać np. wszystkie psy w formie tablicy obiektów typu Dog

[PHP] pobierz, plaintext 
$dogs = Database::getDogs();
if ( is_array($dogs) && sizeof($dogs) ) {
foreach ( $dogs as $obj ) {
echo $obj->getName().'<br />';
}
}
[PHP] pobierz, plaintext 

Dla mnie to bardzo wygodne rozwiązanie, choć pewnie ma bardzo wiele minusów.