Problem z klasą "Moduł", Problem w 'myśleniu obiektowym' czy w kodowaniu? Opcje

[zlw]

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:

Kod

Warning: Module::require_once(../../modules/Article/) [function.Module-require-once]: failed to open stream: No such file or directory in C:\wamp\www\reiki\includes\classes\class.Modules.php on line 33

Fatal error: Module::require_once() [function.require]: Failed opening required '../../modules/Article/' (include_path='.;C:\php5\pear') in C:\wamp\www\reiki\includes\classes\class.Modules.php on line 33

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ę"

[Foxx]

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

Ten post edytował Foxx 13.11.2007, 18:14:39

[Helios]

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.

Ten post edytował Helios 14.11.2007, 01:52:37

[Whisller]

SPL i DirectoryIterator