Forum PHP.pl

Witam.

Dopiero zacząłem swoją przygodę z objektowym PHP, nie wiem czy do końca dobrze rozumię zasady w nim panujące. Byłbym wdzięczny jeżeli osoba która zna się na tym, mogła mi powiedzieć gdzie popełniłem błędy (mimo tego że skrypt działa tak jak chcę, pewnie jakieś się znajdą).

Klasa ma być odpowiedzialna za dodawanie treści do bazy danych, oraz ich późniejsze wyświetlanie. Aktualnie napisałem tylko dwie metody, ponieważ nie wiem czy dobrze realizuję założenia programowania obiektowego.

[PHP] pobierz, plaintext 
	class newsSystem {
 
		public $title;
		public $short_content;
		public $content;
		public $id;
 
		function addNews() {
			$connect = new mysqli("localhost", "root", "", "index");
			$connect->set_charset('utf8');
 
			$connect->query("INSERT INTO news(title, short_content, content) VALUES ('".$this->title."', '".$this->short_content."', '".$this->content."')");
		}
 
		function showNewsById($id) {
			$connect = new mysqli("localhost", "root", "", "index");
			$connect->set_charset('utf8');
 
			$result = $connect->query("SELECT * FROM news WHERE id=". $id ."");
 
			while($row = $result->fetch_array()) {
				echo $row['id'] . "<br />";
				echo $row['title'] . "<br />";
				echo $row['short_content'] . "<br />";
				echo $row['content'];
			}
		}
 
	}
 
	$editNews = new newsSystem();
 
	// Dodawanie newsa
	$editNews->title = "TYTUŁ <br />";
	$editNews->short_content = "KRÓTKA TREŚĆ <br />";
	$editNews->content = "TREŚĆ";
	$editNews->addNews();
 
	// Wyswietlanie newsa po id
	$editNews->showNewsById(19);
[PHP] pobierz, plaintext 

1. W klasie nie łączymy się z bazą, a już na pewno nie w każdej metocie osobno. Utwórz prywatne pole db i wrzuć do niego obiekt bazy danych przez wstrzykiwanie.
2. Klasa nie powinna niczego wyświetlać, może zwrócić dane.

Dzięki za odpowiedź.
Jeżeli chodzi o 2. punkt, myślę że teraz chyba jest dobrze. Odnośnie 1. - nie za bardzo wiem czy dobrze zrobiłem, pierwszy raz spotkałem się z terminem "wstrzykiwania", w internecie za dużo mi się na ten temat znaleźć nie udało; już nie łączę się z bazą w klasie (pewnie i ten sposób jest do poprawy).

[PHP] pobierz, plaintext 
	class newsSystem {
 
		public $title;
		public $short_content;
		public $content;
		public $id;
		public $db;
 
		function addNews($db) {
 
			$db->query("INSERT INTO news(title, short_content, content) VALUES ('".$this->title."', '".$this->short_content."', '".$this->content."')");
		}
 
		function showNewsById($id) {
 
			$result = $this->db->query("SELECT * FROM news WHERE id=". $id ."");
 
			return $result;
		}
 
	}
 
	$editNews = new newsSystem();
	$editNews->db = new mysqli("localhost", "root", "", "index");
 
	// Dodawanie newsa
	$editNews->title = "TYTUŁ <br />";
	$editNews->short_content = "KRÓTKA TREŚĆ <br />";
	$editNews->content = "TREŚĆ";
 
	// Wyswietlanie newsa po id
 
	$wynik = $editNews->showNewsById(19);
 
	while($row = $wynik->fetch_array()) {
			echo $row['id'] . "<br />";
			echo $row['title'] . "<br />";
			echo $row['short_content'] . "<br />";
			echo $row['content'];
	}
[PHP] pobierz, plaintext 

Drugi wynik z google pod zapytaniem wstrzykiwanie oop php ;p klik

możesz zwrócić tablicę danych a nie obiekt wynikający z metody query.

Co do nowego komponentu ktory tworzysz a uzycie DI polecam przeczytac:
http://docs.phalconphp.com/en/latest/reference/di.html

mysle ze swietnie Ci to wytlumaczy dependency injection w praktyce

Ten post edytował ssstrz 2.09.2013, 16:44:40

Wielkie dzięki za rady i artykuły. Poczytałem i sądze, że sposób w jaki teraz to robię jest odpowiedni (mam nadzieję ). Czy są jeszcze jakieś niedociągnięcia oraz miejsca, które dało by radę jeszcze troche zoptymalizować?

[PHP] pobierz, plaintext 
	class newsSystem {
		private $db;
		public $title;
		public $short_content;
		public $content;
		public $id;
 
		function addNews($db){
			$db->query("INSERT INTO news(title, short_content, content) VALUES ('".$this->title."', '".$this->short_content."', '".$this->content."')");
		}
 
		function showNewsById($db, $id) {
			$wiersz = $db->query("SELECT * FROM news WHERE id=". $id ."");
			//$wynik = $wiersz->fetch_array(MYSQL_ASSOC);
			$wynik = $wiersz->fetch_assoc();
 
			return $wynik;
		}
	}
 
	$connect = new mysqli("localhost", "root", "", "index");
 
	$editNews = new newsSystem();
 
	//Dodanie newsa
	$editNews->title = "Tytuł";
	$editNews->short_content = "Krótka";
	$editNews->content = "Treść";
	$editNews->addNews($connect);
 
	//Wyświetlenie pojedyńczego newsa
	$newsById = $editNews->showNewsById($connect, 23);
	foreach ($newsById as $indeks => $showNews) {
		echo "newsById[$indeks] = $showNews <br />";
	}
[PHP] pobierz, plaintext 

Tak na szybko:

Czemu wszędzie przekazujesz $connect?

Daj proteted $_db; i metodę setDb($db);
Potem w metodach używasz $this->_db->query()

Skoro klasa nazywa się news, po co w każdej metodzie zaznaczasz że operujesz na newsach?

Dzięki za cierpliwość
Nawet się już nie pytam czy teraz dobrze, tylko czy teraz lepiej jest napisany ten kod?

[PHP] pobierz, plaintext 
	class newsSystem {
 
		public $title;
		public $short_content;
		public $content;
		public $id;
		private $db;
 
		function connectDb($connect) {
			$this->db = $connect;
		}
 
		function add() {
			$this->db->query("INSERT INTO news(title, short_content, content) VALUES ('".$this->title."', '".$this->short_content."', '".$this->content."')");
		}
 
		function showById($id) {
			$wiersz = $this->db->query("SELECT * FROM news WHERE id=". $id ."");
			$wynik = $wiersz->fetch_assoc();
 
			return $wynik;
		}
 
	}
 
	$news = new newsSystem;
 
	$connection = new mysqli("localhost", "root", "", "index");
 
	$news->connectDb($connection);
 
	//Dodawanie newsa
	$news->title = "tytul";
	$news->short_content = "short";
	$news->content = "content";
 
	$news->add();
 
	//Wyswietlanie newsa po id
	$newsById = $news->showById(23);
	foreach ($newsById as $indeks => $showNews) {
		echo "newsById[$indeks] = $showNews <br />";
	}
[PHP] pobierz, plaintext 

@up

Jest on bardzo źle napisany, zawiera luki w bezpieczeństwie oraz nie ma nic wspólnego z OOP. Wymyślono już dawno temu ORM-y, które rozwiązują Twoje problemy, a do tego są napisane poprawnie - dlaczego z nich nie skorzystasz? Uprzedzając jeszcze: 1) dużo szybciej nauczysz się znaczenie więcej 2) kod będzie działał i nie rzucał kłód pod nogi na każdym kroku 3) przy okazji poznasz i zyskasz doświadczenie w pracy z daną biblioteką.

Łamiesz przede wszystkim zasadę Single Responsibility Principle. To jest ciągle kod proceduralny tylko opakowany w klasę i nie ma (prawie) nic wspólnego z obiektowością. Bez zagłębiania się w kod klas finalnie byłoby lepiej, gdyby wyglądało to tak:

[PHP] pobierz, plaintext 
// wzorzec factory tworzy nam nowy obiekt Wpisu
$post=NewsFactory::create('Post');
$post->setTitle('Tytuł wpisu');
$post->setShort('Lorem...');
$post->setBody('Lorem ipsum..');
 
// uruchamiamy silnik bazy, w tym przypadku SQL
// ale nic nie stoi na przeszkodzie, żeby podstawić inny silnik bez zmiany kodu
$sql_connection=new SqlConnection('localhost', 'user', 'haslo');
 
// albo:
// $sql_connection=new OracleConnection();
// $sql_connection=new FlatFileDatabase();
// $sql_connection=new CookieDatabase();
// itd.
 
// uruchamiamy wrapper bazy, który odpowiada za operacje
// na systemie bazodanowym (lub każdym innym)
$db_wrapper=new DbWrapper($sql_connection);
 
// zapisz post
$db_wrapper->save($post);
 
// pobierz post z bazy
$post_z_bazy=$db_wrapper->getById(1);
[PHP] pobierz, plaintext 

Klasa SqlConnection powinna implementować jakiś stabilny interfejs typu:

[PHP] pobierz, plaintext 
interface PersistentStorage
{
   /**
    * Zapisz dane do magazynu.
    * @return bool Czy zapis się powiódł.
    */
   public function save($object);
 
   /**
    * Pobierz dane przez klucz główny.
    * @return mixed Pobrany rekord
    */
   public function getById($id);
}
[PHP] pobierz, plaintext 

Ten post edytował em1X 27.09.2013, 11:55:10