Witam, jestem początkującym programistą PHP (samoukiem). Ponieważ uważam że najlepiej uczy się wykonując praktyczne zadania stworzyłem sobie projekt który poniekąd jest mniej rozbudowaną wersją pewnego serwisu. Szablon strony tworzę w bootstrapie. Chciałbym przedstawić kawałek kodu który stworzyłem i poprosić o to aby znawcy programowania obiektowego na tym forum wypowiedzieli się na ten temat, czy ten kod który napisałem mogę nazwać OBIEKTOWYM oraz o ewentualne porady dotyczące modyfikacji tego co napisałem.

W tym wszystkim chodzi o to aby do bazy danych można było dodawać książki a potem na swoim profilu dodawać je do ulubionych, na półki typu przeczytane, posiadam, chcę przeczytać itd. Na stronie głównej natomiast chcę wyświetlać 5 ostatnio dodanych (przez użytkowników) książek do bazy (okładka, tytuł, autor oraz opis) tak jak widoczne jest to na zdjęciu (zmieściły się tylko 3).



Tutaj prezentuję kod klasy stworzonej do pobierania rekordów z bazy i wyświetlania ich na stronie.

[PHP] pobierz, plaintext 
class LatestBooks{
		private $tytul;
		private $autor;
		private $opis;
		private $sciezka;
		private $num;
		private $pdo;
 
		public function __construct($num, $pdo ){
			$this->num = ($num - 1);
			$this->pdo = $pdo;
		}
 
		public function getData(){
			$stmt = $this->pdo->query('SELECT * FROM `books`ORDER BY id DESC LIMIT 5 ');
			$row = $stmt->fetchAll(PDO::FETCH_ASSOC);
			$this->tytul	 = $row[$this->num]['tytul'];
			$this->autor	= $row[$this->num]['autor'];
			$this->opis  =   $row[$this->num]['opis']."...".'<button type="button" class=class="btn btn-inverse btn-xs">Więcej</button>';
			$this->sciezka = $row[$this->num]['sciezka'];
			$stmt->closeCursor();
 
		}
 
		public function write(){
		return "
			<div class=\"book col-md-12\">
			<div class=\"col-md-4\"> 
			<img src=". "{$this->sciezka}". " class=\"fotka\">  
			</div>
			<div class=\"col-md-7 \"> 
			<h4>". "{$this->tytul}". "</h4>
			<p>Autor:". "{$this->autor}". "</p>
			<h5>". "{$this->opis}" ."</h5>
			</div>
		</div>  ";
		}
 
}
[PHP] pobierz, plaintext 

A tutaj ten, odpowiedzialny za utworzenie obiektów i wywołanie metody.

[PHP] pobierz, plaintext 
require('klasa.php');
	$pdo = new MyPDO();
	$book = new LatestBooks(1, $pdo);
	$book->getData();	
	print $book->write();
	$book = new LatestBooks(2, $pdo);
	$book->getData();	
	print $book->write();
	$book = new LatestBooks(3, $pdo);
	$book->getData();	
	print $book->write();
	$book = new LatestBooks(4, $pdo);
	$book->getData();	
	print $book->write();
	$book = new LatestBooks(5, $pdo);
	$book->getData();	
	print $book->write();
[PHP] pobierz, plaintext 

Dziękuję z góry wszystkim którzy zechcą poświęcić czas na pomoc, mam nadzieję że wszystko objaśniłem jak najlepiej i podałem wszystkie niezbędne informacje. Ze swojej strony pragnę jeszcze dodać że jest to pierwszy raz kiedy napisałem coś (mam nadzieję) obiektowo.

Witamy na Forum. Zapoznaj się z panującymi tu zasadami, by uniknąć blokady wątków.
Do ocen jest dział Oceny, przenoszę.

Przepraszam i dziękuję za przeniesienie.

Może i masz rację na temat tworzenia projektów, ale wtedy za podstawę trzeba brać poprawny kod, bo to co stworzyłeś śrenio trzyma się kupy.

- wyświetlasz listę, a dla każdej pozycji tworzysz nie potrzebnie obiekt
- pobierasz za każdym razem 5 rekordów a wyświetlasz jeden
itp itd

Jako parametr powinieneś podać nie ID, tylko ilość materiałów do wyświetlenia i wywołać klasę tylko raz.

[PHP] pobierz, plaintext 
$book = new LatestBooks( $pdo );
$book->getData( 5 - liczba pozycji );	
echo $book->write();
[PHP] pobierz, plaintext 

Dlatego stworzyłem ten temat, aby dowiedzieć się czy to co zrobiłem jest poprawne.
Dziękuję za wyrażenie opinii, spróbuję to poprawić zgodnie ze wskazówkami.


@edit: Mam jeszcze pytanie. Po wpisaniu wartości dla ilości rekordów które chciałbym wyświetlić po pobraniu z bazy powinienem wewnątrz tej metody użyć pętli do wyświetlenia tych 5 różnych rekordów ? Czy może istnieje jakieś inne rozwiązanie?

Moim zdaniem nie od tego powinieneś zacząć, a od poznania struktur itp. bo już teraz łapiesz nie potrzebnie nawyki ale każdy ma swoje podejście i oczywiście nie twierdzę, że Twoje jest złe :-)

Na poczatek spróbuj wobec tego wykonać to na petli w metodzie write.

Zasięgnąłem porady również na innym forum i teraz wyżej prezentowany kod wygląda tak.



Wywołanie:



Jak to oceniasz?

Już lepiej ale nie ważne jak Ci napiszą abyś robił, ale to czy rozmiesz dlaczego w ten sposób.

W ogóle to rozbij skrypt na MVC.
FrontController - reaguje na to co w np. GET itp. Jeśli GET masz / to odpal tę stronę główną z 5 książkami (czy co tam ma być robione na stronie głównej) czyli odpala konkretną metodę kontrolera
Controller - metoda wywołana w FC wysyła zapytanie do modelu
Model - odpala zapytanie do bazki, zapisuje wynik w tablicy i returnuje do kontrolera, a kontroler odpala teraz metode widoku
View - returnuje zrenderowany widok do kontrolera, a następnie kontroler zwraca do FC i tu następuje dopiero echo