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

[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