[skrypt][PHP]Kawałek kodu do oceny Opcje

[Ojcov]

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).

(IMG:http://imageshack.com/a/img673/3348/hhMjX0.png)

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.

Ten post edytował Ojcov 1.05.2015, 11:32:31

Powód edycji: [Kshyhoo]: bb-code

[Ojcov]

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

Kod

class LatestBooks{
        private $pdo;
        
        public function __construct(MyPDO $pdo ){
            $this->pdo = $pdo;
        }
        
        public function getData($num){
            $stmt = $this->pdo->query('SELECT * FROM `books`ORDER BY id DESC LIMIT '.$num.' ');  
            return $stmt->fetchAll(PDO::FETCH_ASSOC);                                         
        }    
}

Wywołanie:

Kod

require('inc/klasa.php');
    $pdo = new MyPDO();
    $book = new LatestBooks($pdo);
    $data = $book->getData(5);
foreach($data as $row) {
?>
    <div class="book col-md-12">
            <div class="col-md-4">
            <img src="<?=$row['sciezka']?>" class="fotka">  
            </div>
            <div class="col-md-7 ">
            <h4>"<?=$row['tytul']?>"</h4>
            <p>Autor:"<?=$row['autor']?>"</p>
            <h5>"<?=$row['opis']?>"</h5>
              </div>
        </div>  
        <?php
}

Jak to oceniasz?