[PHP]poprawność kodu, pisanie klas Opcje

[wpaski]

Prosiłbym o przyjrzenie się tej prostej klasie i ew. wskazanie mi błędów co zmienić w sposobie pisania, zastanawiałem się czy użyć dla zmiennych private czy public, uznałem, że nie ma potrzeby dostępu z poza klasy, druga moja wątpliwość to kod html zwracany w funkcji pobierzZdjecia(), czy należało by go odseparować? Np jeżeli chciałbym użyć tą klasę w stronie opartej o wzorzec MVC, a ta klasa była by w bibliotekach ( czyli tzw. helper?)

[PHP] pobierz, plaintext 
<?php
class Galeria {
 
	private $kat;
	private $katalog = 'obrazki/';
	private $katalogminiaturki = 'miniaturki/';
	private $galeria;
	private $zdjecie;
	private $odczyt;
 
	public function __construct() {
	   $this->kat = isset($_GET['kat']) ? $_GET['kat'] : 'dyrdy';
	   $this->katalog = $this->katalog.$this->kat;
	   $this->katalogminiaturki = $this->katalogminiaturki.$this->kat;   
	}
	public function pobierzZdjecia() {
		$this->galeria = opendir($this->katalog);	
		while($this->zdjecie = readdir($this->galeria)) {
			$this->odczyt = pathinfo($this->katalog.'/'.$this->zdjecie);
				if($this->odczyt['extension'] == 'jpg') {
					return '<li><a href="'.$this->katalog.'/'.$this->zdjecie.'" class="highslide" onclick="return hs.expand(this)" title="Zdjęcie: '.$this->zdjecie.'"><img width="200" height="133" src="'.$this->katalogminiaturki.'/'.$this->zdjecie.'" alt="Zdjęcie: '.$this->zdjecie.'" /></a></li>';
			}
		}
	}	
}
 
$galeria = new Galeria();
echo $galeria->pobierzZdjecia();
 
?>
[PHP] pobierz, plaintext