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]

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.