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

[wilman]

Witam!
Napisalem pierwszą klasę w swoim życiu
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 

[Cysiaczek]

1. Nazwa listing nie odzwierciedla tego, co robi klasa.
2. Najpierw przekazujesz parametr do funkcji, a potem używasz tablicy $_GET do jego odczytania - bez sensu
3. Ten sam parametr zaburza hermetyzację
4. Używasz polskich nazw dla zmiennych - w połączeniu z angielskimi nazwami metoda jest to jakieś esperanto - używaj tylko angielskich nazw.
5. To co już zostało powiedziane wyżej - zgodność z PHP 4 - php 4 już nie ma
6. Zajrzyj na http://www.php.net/~helly/php/ext/spl/ i zobacz DirectoryIterator oraz RecursiveDirectoryIterator - są wbudowane w php i robią to, co Twoja klasa.

Pozdrawiam