MVC - model, swobodne rozwazania Opcje

[.dragonfly]

Buduje aplikacje oparta o MVC i w zwiazku z tym faktem doszedlem do pewnego rozwiazania. Teraz mam pytanie na ile jest ono trafne.

Jak wszyscy wiemy model ma za zadanie tylko operacje na danych. Czy dobrym rozwiazaniem jest zbudowanie modelu ktory odwoluje sie do "pod-modelow" (modulow) w nastepujacy sposob:

[PHP] pobierz, plaintext 
<?php
 
/*
 * W tym fragmencie kodu odwolujemy sie do modelu oraz sub-modulu user
 * ktory operuje rzadaniami zwiazanymi z operacjami na uzytkowniku
 */
$this->model[user]->userProfile(1);
 
 
?>
[PHP] pobierz, plaintext 

Ladowanie modulu (metoda w klasie Model):

[PHP] pobierz, plaintext 
<?php
 
 
 
   /**
	* Wybiera odpowiedni modul, includujac plik
	* 
	* @param  $strModuleType 
	* @param  $strModuleName
	* @param  $strModuleVar
	* @return 
	*/
	function loadModule( $strModuleType, $strModuleName, $strModuleVar = false )
	{
		if ( !$strModuleVar ) $strModuleVar = $strModuleName;
		require_once ( ENVIE_DIR . &#092;"modules\" . $strModuleType . \"envie.module.\" . $strModuleName . \"\" . $strModuleName . '.module.php' );
		$this->model[$strModuleVar] = new $strModuleName;
	} 
?>
[PHP] pobierz, plaintext 

Kazdy sub-modul jest "dzieckiem" (extends) klasy Model. Co sadzicie o takim rozwiazaniu

[.dragonfly]

Jedynym niedopatrzeniem z mojej strony mogl byc byc fakt iz nie pokusilem sie o sprawdzanie czy dana klasa oraz plik istnieja, ale to w rozwazaniu na temat metody sprawa kosmetyczna.

[PHP] pobierz, plaintext 
<?php
 
/**
	* Wybiera odpowiedni modul, includujac plik
	*
	* @param  $strModuleType
	* @param  $strModuleName
	* @param  $strModuleVar
	* @return
	*/
function loadModule( $strModuleType, $strModuleName, $strModuleVar = false )
	{
		if ( !$strModuleVar ) $strModuleVar = $strModuleName;
 
		if ( file_exists( ENVIE_DIR . &#092;"modules\" . $strModuleType . \"envie.module.\" . $strModuleName . \"\" . $strModuleName . '.module.php' ) )
		{
			require_once ( ENVIE_DIR . &#092;"modules\" . $strModuleType . \"envie.module.\" . $strModuleName . \"\" . $strModuleName . '.module.php' );
			if ( class_exists( $strModuleName ) )
			{
				$this->model[$strModuleVar] = new $strModuleName;
			} 
			else
			{ 
				// brak klasy
			} 
		} 
		else
		{ 
			// brak pliku
		} 
	} 
?>
[PHP] pobierz, plaintext 

Dodatkowo w twoim rozwiazaniu nie ma mozliwosci zaladowania modulu pod inna nazwa. Tzn. nie mozesz zaladowac modulu "Foo" i odwolywac sie do niego poprzez np. $this->model[bar]->costam()

Co do dziedziczenia w modelu jest to jak najbardziej wskazane przy takowym rozwiazaniu wiec w tym aspekcie nie moge sie z toba zgodzic. Anyone else?