[OOP] listowanie zawartości katalogu, ... Opcje

[wilman]

Witam!
Napisalem pierwszą klasę w swoim życiu (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)
Jej zadaniem jest listowanie zawartosci katalogu i przekazywanie zawartosci do szablonów Smarty, ale tutaj jest nieco przerobiona wersja.
Problem polega na tym, że jak wylistuję sobie katalog './' i wejdę w jakiś folder to adres w pasku przeglądarki powinien wyglądać tak "index.php?katalog=./folder/" i tak wygląda, ale jak otworzę katalog, który znajduje się w katalogu 'folder', to adres wygląda tak "index.php?katalog=./to coś z katalogu folder" i wywala błąd, bo taki plik nie istnieje (powinno być "index.php?katalog=./folder/to coś")

ps. proszę o ogólne uwagi / zalecenia co do tej klasy, gdyż jak wspomnialem, jest to mój pierwszy kod OOP.
Pozdrawiam :-)

[PHP] pobierz, plaintext 
<?php
 
class listing {
 
 //deklaracje zmiennych
 var $elementy = array();
 var $foldery = array();
 var $katalog = '';
 
//sprawdzanie czy url nie zawiera ../
 function sprawdzUrl($katalog) {
	if( (eregi('../', $this -> katalog)) OR (empty($_GET['katalog']) ) ) {
		return false;
	} else {
		return true;
	}
 }
 
 //funkcja wczytująca zawartośc katalogu do tablicy $elementy
 function listing ($katalog) {
	$this -> katalog = $_GET['katalog'];
	if( listing::sprawdzUrl($katalog) AND is_dir($katalog)) {
		$d = opendir($katalog) or die ('Nie można otworzyć katalogu');
		while(false !==  ($plik = readdir($d))) {
			if($plik != '.' && $plik != '..') {
			$this -> elementy[] = $plik;
			}
		}
		closedir($d);
	} else {
		die('Podaj poprawny adres!');
	}
 }
 
}
 
//wywołanie
$klasa = new listing($_GET['katalog']);
 
foreach ($klasa -> elementy as $klucz => $folder) {
			if(is_dir($katalog.'/'.$folder)) {
			echo "<a href=\"listing2.php?katalog=$katalog$folder\">$folder</a> <br />";
			//$this -> foldery[] = $folder; echo folder
		} else {	echo $folder."<br />"; }
		}
?>
[PHP] pobierz, plaintext 

[l0ud]

Podstawowe błędy, które się rzucają w oczy:

- jak kolega wspomnial, nie wykorzystujesz mozliwosci PHP5, klasa jest wyraznie napisana na PHP4, od ktorego sie juz odchodzi.
- w przypadku bledu skrypt jest przerywany, a powinien byc zwracany blad/wyjatek.
- metody i zmienne nazywasz po polsku, a skrypty powinno sie pisac po angielsku (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

btw. trochę niekonsekwentnie wyglada wykorzystanie tej klasy, przy warunku is_dir() do dajesz slash (/) a przy echo już nie.

[Fantazyn]

- w przypadku bledu skrypt jest przerywany, a powinien byc zwracany blad/wyjatek.

Mam pytanie, jak powinno to wyglądać? Try, catch? Takie komunikaty o błędach w die nie wystarczą? Można prosić o krótki przykład?
Czy chodzi o to, by metody zwracały zawsze wartość (return)?
Pozdrawiam.