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 

mi jako początkujecemu programiscie oop rzuca sie w oczy tylko jedna sprawa czemu uzywasz "var" zamista public, proceted, private w koncu jest juz php5

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

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

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

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.

->po co zadeklarowałeś zmienną foldery skoro jej nie używasz?
->no i jak już wyżej było nie powinno się używać tablic globalnych ($_GET, $_POST) w klasach po to są parametry w metodach

W sumie, to ta klasa jest niezbyt przydatna... Sa przeciez Iteratory...

Nie stosuje się die() w OOP - po to masz wyjątki. Łapiesz wyjątek i możesz go obsłużyć albo nie. Dla mnie wyjątek to coś krytycznego. Jeśli się zdarzy to zapisuje w logach a user widzi na ekranie komunikat błędu.


Poczytaj http://blog.whisnet.pl/tag/php5

pzdr

Trzy grosze odemnie. W PHP5 mamy scandir(), co ułatwia znacznie życie (lepsza tablica niż obiekt w tym przypadku).

hmm... no cóż... dzięki za wszystkie uwagi / wskazówki, na przyszłość będę wiedział, póki co - biorę się za poprawienie tego
Jak wiadomo - zawsze początki bywają ciężkie

Pozdrawiam