SOLID i Zasada jednej odpowiedzialności, jak można zastosować tą zasadę dla tych klas i takiego wywołania Opcje

[Nortonek]

Witam

Mam prośbę o nakierowanie mnie na temat pierwszej zasady SOILID, jak można zastosować tą zasadę dla tych klas i takiego wywołania,

w pierwszej klasie odwołuję się do drugiej, jednak znając życie będzie taka sytuacja że będę musiał przekazać więcej parametrów a nie tylko ID książki

i będę musiał modyfikować 2 klasy.

[PHP] pobierz, plaintext 
<?php
 
 
class Ksiazki {
 
  private  $id_ksiazki;
 
  function __construct($id){
     $this->id_ksiazki - $id;
  }
 
  private function zapisz_oddanie_zniszczonej_ksiazki(){
	  global $db, $table_ksiazki_oddane;
 
	  $result0 = $db -> Execute("UPDATE $table_ksiazki_oddane SET	`oddana`= '1' WHERE `id_ksiazki`='".$this->id_ksiazki."' ");
 
	  $Ksiazki_Zniszczone 	= new Ksiazki_Zniszczone();
          $Ksiazki_Zniszczone-> insert_ksiazki_zniszczone($this->id_ksiazki);
  }
 
}
 
 
class  Ksiazki_Zniszczone{
 
 public function insert_ksiazki_zniszczone( $id_ksiazki){
    global $db, $table_ksiazki_zniszczone;
 
    $result0 = $db -> Execute("UPDATE $table_ksiazki_zniszczone SET `zniszczenie_ksiazki`= '5' WHERE `id_ksiazki`='$id_ksiazki' ");
 }
 
}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował Nortonek 23.02.2014, 13:45:38

[Crozin]

Na sam początek: nie próbuj samemu robić mapowania bazy danych na model obiektowy (ORM). Jest to wbrew pozorom dosyć skomplikowane zadanie i bez odpowiedniej wiedzy ciężko Ci będzie zrobić cokolwiek co będzie miało jakieś zalety w porównaniu do "zwykłego klepania zapytań". Skorzystaj z Doctrine'a, w większości miejsc jest on całkiem dobrym przykładem zasad SOLID.

Co do Twojego kodu:
1. Powinieneś przekazywać obiekty jako argumenty, a nie ich wewnętrzne właściwości.
2. Jeżeli klasa ma za zadanie reprezentować pojedynczy obiekt (np. książkę) powinna nazywać się Książka, nie Książki.
3. Jedna klasa nie powinna zajmować się reprezentacją pojedynczej książki jak i operacjami bezpośrednio na bazie. Na dobrą sprawę powinieneś mieć tutaj kilka obiektów:
- Książka reprezentujący pojedynczą książkę,
- MenadżerKsiążek, którego metoda oddajZniszczonąKsiążkę przyjmie jako argument obiekt klasy Książka. Pojawienie się w nazwie klasy słowa "Menadżer" z reguły sugeruje jej zbyt szeroki zakres obowiązków, ale tutaj mamy tylko przykład,
- KsiążkaDAO - po prostu DAO dla typu Książka (czy jakiś bardziej generyczny), wykorzystywany przez menadżera.
4. Nie powinieneś korzystać z globali.