Witam, właśnie (od 2 dni) zacząłem swą "przygodę z programowaniem obiektowym" i na początek postanowiłem napisać sobie class'ę wczytywania modułów. Oto kod:

[PHP] pobierz, plaintext 
<?php
class Module
 {
	public $modules_dir;
	public $module_name;
	public $file_list;
 
	 public function loadModule()
	  {
		  if(file_exists($this->modules_dir.'/'.$this->module_name))
		   {
			if(strpos($this->file_list, ';'))
			 {
				 $this->file_list = explode(';', $this->file_list);
				 $num_file_list   = count($this->file_list);
			 }
 
			for($i = 0; $i = $num_file_list; $i++)
			 {
				if(file_exists($this->modules_dir.'/'.$this->module_name.'/'.$this->file_list[$i]))
				 {
					require_once($this->modules_dir.'/'.$this->module_name.'/'.$this->file_list[$i]);
					return 'ok';
				 }
			  else
			   {
					return "nie ok";
				}
			 }
			   
		   }
		  else
		   {
			   return 'Katalog nie istnieje';
		   }
	  }
 }
 
$object = new Module();
$object->modules_dir = '../../modules';
$object->module_name = 'Article';
$object->file_list   = 'index.php;html.php';
$object->loadModule();
?>
[PHP] pobierz, plaintext 

Problem w tym, że wywala mi błąd:


Nie wiem czy błąd leży gdzieś w 'obiektowości' ($this, new Object itp.) czy może (oby) gdzieś w "zwykłym" kodzie PHP, tak czy inaczej - proszę o pomoc.
Przy okazji prosiłbym o ocenę klasy - czy waszym zdaniem dobre podejście do problemu czy to tylko "strukturalny kod objęty w klasę"

A nie powinno być:

[PHP] pobierz, plaintext 
<?php
// zamiast
for($i = 0; $i = $num_file_list; $i++)
// tak:
for($i = 0; $i < $num_file_list; $i++)
// ?
?>
[PHP] pobierz, plaintext 

count zwróci Ci liczbę elemetów ale tablica jest indeksowana od zera, błąd jest stąd że przez to próbujesz includować plik, którego nie ma

Blad podstawowy taki jak poprzednik napisal. Od siebie dodam, ze ta metoda nigdy Ci nie zadziala, jezeli bedzie w srodku wykonywania petli zwracac jakas wartosc (return) - bo return automatycznie konczy wykonywanie metody w tym przypadku przerywa takze wykonywanie petli.
Staraj sie udostepniac spojny interfejs uzytkownikom klasy i nie dawac im bezposredniego dostepu do skladowych klasy (zobacz hermetyzacja). Czyli dostep do skladowych jest mozliwy dzieki udostepnionemu interfejsowi, uzytkownika nie interesuje sposob implementacji. Osiagniesz to ustawiajac skladowe jako prawatne. W tym konkretnym przypadku powinno to sie odbywac przez metody akcesory (udostepnione publicznie): setModule(string $moduleName), setFilesPath(string $filesPath), setFilesArray(array $filesArray) itd. Moze to wydawac sie bez sensu ale oczywiscie nie jest go pozbawione.
Np. gdybys chcial uniemozliwic wpisywanie znakow ./ ../ do nazw plikow za kazdym razem musialbys uruchamiac mechanizm sprawdzajacy/usuwajacy te znaki z ciagu a pozniej metode klasy ladujace dane pliki. Doszloby do powielania kodu a taki mechanizm mozesz zaimplementowac we wnetrzu klasy i masz pewnosc, ze ktos nie przypisze na sztywno nazw plikow zawierajacych te znaki. Mechanizm znajduje sie w jednym miejscu i jest wywolywany przy kazdej probie przypisania wartosci do skladowej.

W Twojej klasie powinna byc mozliwosc dodania jednego pliku do listy (metoda dodaje do listy nazwe pliku), dodania ciagu (plik;plik - ta metoda powinna zajac sie parsowaniem ciagu do tablicy) albo samej tablicy (metoda dolacza tablice do tablicy plikow w klasie). Oddciazasz w ten sposob zadania metody loadModules. W setModule mozesz zaimplementowac od razu mechanizm sprawdzania czy plik istnieje i ewentualnie rzucac wyjatek.

Jak widzisz same plusy z takiego podejscia do sprawy, zamiast udostepniania mozliwosci przypisywania wartosci do skladowych z zewnatrz - na sztywno.

Mozesz takze pomyslec o dodaniu obslugi bledow - w PHP 5 - patrz wyjatki.

SPL i DirectoryIterator