[klasa]Modules Opcje

[Joachim Peters]

Chciałbym się Wam spytać, jak podoba się wam moja klasa do obsługi modułów, co w niej zmienić itd.
Zapewne nie ma się czym chwalić, ale głównie chodzi mi o sugestie, propozycję i opinię!

[PHP] pobierz, plaintext 
<?php
 
class Modules {
	 private $moduleType;
	 private $defaultUserModule = 'news';
	private $defaultAdminModule = 'stats';
 
	/**
	 * Sprawdza czy moduł istnieje
	 */
	private function checkModuleExists($moduleName) {
		if(file_exists('./modules/'.$this->moduleType.$moduleName.'.php')) {
			return true;
		}
 
		return false;
	}
 
	/**
	 * Uruchamia poszczególne moduły. 
	 */
	public function run($moduleName = null, $admin = false) {
		 $moduleName = strtolower($moduleName);
		 
		if(strlen($moduleName) < 4 || strlen($moduleName) > 15) {
			throw new Exception('Nazwa modułu jest za krótka lub za długa.');
		}
 
		$this->moduleType = ($admin) ? 'admin/' : 'user/';
 
		if($moduleName) {
			if($this->checkModuleExists($moduleName)) {
				include_once('./modules/'.$this->moduleType.basename($moduleName.'.php'));	
 
				return;
			}	
		} 
 
		if($this->checkModuleExists(($admin) ? $this->defaultAdminModule : $this->defaultUserModule)) {
			include_once('./modules/'.$this->moduleType.basename(($admin) ? $this->defaultAdminModule : $this->defaultUserModule.'.php'));	
		} else {
			throw new Exception('Nie można załadować żadnego modułu');
		}
	}
}
 
?>
[PHP] pobierz, plaintext 

Główne założenie klasy to bezpieczne wczytywanie plików (modułów). Nazwa modułu jest pobierana z adresu.
Chętnie przyjmę krytykę, lecz bez przesady bo to dopiero moje początki (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam

Ten post edytował Joachim Peters 24.08.2007, 14:25:19

[menic]

No wybacz, ale nie wiele tu oceniac. Raptem pare instrukcji... I po co ta "blokada" na dlugosc nazwy?

[Joachim Peters]

Tylko ten dział pasuje aby spytać się Was o sugestie, wiem, że nie ma tu co oceniać ;P

[bełdzio]

w checkModuleExists zamiast warunku możesz dać po prostu return file_exists

[Speedy]

w checkModuleExists zamiast warunku możesz dać po prostu return file_exists

Ja bym w ogóle wywalił tę metodę, bo nie robi ona niczego poza wywołaniem funkcji file_exsits(). Jest to dodawanie niepotrzebnego kodu.

[Joachim Peters]

Zrobiłem to dla przejrzystości, żebym nie musiał ciągle ścieżki pisać (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował Joachim Peters 25.08.2007, 16:44:18

[Moli]

No ale mogłeś dać tylko

[PHP] pobierz, plaintext 
<?php
return file_exists('./modules/'.$this->moduleType.$moduleName.'.php') ;
?>
[PHP] pobierz, plaintext