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.

[Nortonek]

Wyeliminowałem globale, mam nadzieję że prawidłowo, zmieniłem nazewnictwo klas i dodałem Dependency injection w klasie Bibliotekarki

[PHP] pobierz, plaintext 
<?php
 
$db = NewADOConnection('mysql');
$Ksiazka= new Ksiazka();
$Pani_Bibliotekarka = new Pani_Bibliotekarka($Ksiazka, $db); 
 
$Ksiazka->set_id_ksiazki(1);
$Pani_Bibliotekarka -> zapisz_oddanie_ksiazki();
 
$Ksiazka->set_id_ksiazki(2);
$Pani_Bibliotekarka -> zapisz_oddanie_zniszczonej_ksiazki();
 
 
class Tablica_SQL {
	static const tab_sql_ksiazki_oddane = 'tab_ksiazki_oddane';
	static const tab_sql_ksiazki_ozniszczone = 'tab_ksiazki_zniszczone';
}
 
class Ksiazka {
  private  $id_ksiazki;
  private $Ksiazka;
 
  function set_id_ksiazki($id){
			$this->id_ksiazki = $id;
  }
 
}	
 
 
class Pani_Bibliotekarka{
	private $Ksiazka;
	private $db
 
	function __construct(Ksiazka $Ksiazka, db $db){
			$this->Ksiazka = $Ksiazka;
			$this->db = $db;
	}
 
  private function zapisz_oddanie_ksiazki(){
	  $result0 = $this->db -> Execute("UPDATE ".Tablica_SQL::tab_sql_ksiazki_oddane." SET	`oddana`= '1' WHERE `id_ksiazki`='".$this->Ksiazka->id_ksiazki."' ");
  }
 
  private function zapisz_oddanie_zniszczonej_ksiazki(){
	  $result0 = $this->db -> Execute("UPDATE ".Tablica_SQL::tab_sql_ksiazki_oddane." SET	`oddana`= '1' WHERE `id_ksiazki`='".$this->Ksiazka->id_ksiazki."' ");
 
		$Ksiazki_Zniszczone 	= new Ksiazki_Zniszczone();
    $Ksiazki_Zniszczone-> insert_ksiazka_zniszczona($this->Ksiazka->id_ksiazki);
  }
 
}
 
 
class  Ksiazka_Zniszczona{
 
 public function insert_ksiazka_zniszczona( $id_ksiazki){
    $result0 = $this->db -> Execute("UPDATE ".Tablica_SQL::tab_sql_ksiazki_zniszczone." SET `zniszczenie_ksiazki`= '5' WHERE `id_ksiazki`='$id_ksiazki' ");
 }
 
}
 
?>
[PHP] pobierz, plaintext 

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

[Crozin]

1. Stan zniszczenia książki to właściwość samej książki, po co Ci do tego osobna klasa?
2. PaniBibliotekara to właśnie taki menadżer z mojego wcześniejszego postu. Powinna mieć ona metodę przyjmijZwracanąKsiążkę, która powinna jako argumenty przyjąć obiekty typu Książka oraz OsobaWypożyczająca. Wewnątrz tej metody mogłaby porównać stan książki sprzed wypożyczenia i w momencie jej zwrócenia. Jednak samo zapisywanie danych w bazie nie powinno już leżeć w jej zakresie obowiązków.

PS. Tak jak mówiłem, zapoznaj się z jakimś gotowym ORM-em: będziesz tam miał przykłady jak poprawnie pisać kod. Zaoszczędzisz sobie masy czasu, wyrobisz zdecydowanie więcej dobrych nawyków, a przy okazji poznasz jakieś konkretne narzędzie.

[Ormin]

Pozwolę napisać sobie jak ja bym to zrobił i why:

[PHP] pobierz, plaintext 
class Ksiazka {
    private $id;
    public function __construct($id) {
        $this->id = $id;
    }
 
    public function getID() {
         return $this->id;
    }
 
}	
 
 
class Pani_Bibliotekarka {
 
    private $db
 
    public function __construct(db $db) {
        $this->db = $db;
    }
 
    public function oddaj_ksiazke(Ksiazka $ksiazka) {
          $this->db->Execute("UPDATE tab_ksiazki_oddane  SET `oddana`= '1' WHERE `id_ksiazki`='".$ksiazka->getID()."' ");
    }
 
    public function oddaj_zniszczona_ksiazke(Ksiazka $ksiazka){ 
           $this->oddaj_ksiazke($ksiazka);
           $this->db->Execute("UPDATE tab_ksiazki_zniszczone SET `zniszczenie_ksiazki`= '5' WHERE `id_ksiazki`='".$ksiazka->getID()."' ");
    }
 
}
 
[PHP] pobierz, plaintext 

Podział odpowiedzialności:
Ksiazka - reprezentuje obiekt domenowy ( naszą książkę )
Pani Bibliotekarka - reprezentuje repozytorium domenowe dla naszych książek i udostępnia API do zarządzania nimi ( w tym wypadku tylko oddawnaie zniszczonych / niezniszczonych książek )

[em1X]

Złota definicja: "Jeśli istnieje więcej niż jeden powód, żeby zmodyfikować klasę, to łamie ona zasadę SRP." Btw. słyszałem, że za używanie słowa kluczowego global mają ucinać ręce.

Ten post edytował em1X 9.05.2014, 19:21:32