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

Jest ono calkiem pozyteczne, bo ladujesz klasy np tylko w kernelu, aczkolwiek pociaga za soba pewne konsekwencje. Moim zdaniem ta zmienna powinna byc "prywatna" (jezeli php5) i dostep do niej powinien byc ograniczony przez zdefiniowane metody.

@seaquest: Dokladnie tak jak powiedziales jest to juz rozwiazane. IMO przy takim rozwiazaniu aplkacja zyskuje nie tylko na przejrzystosci, ale przedewszyskim na wydajnosci.

Rozwiazanie jakie zaproponowalem jest moim zdaniem przydatne przy rozbudowanych aplikacjach i daje ciekawe mozliwosci rozbudowy.

Uwaga czytanie ponizszego posta jest na wlasne ryzyko oglupienie spowodowane jego zawartosci merytoryczna(doslowna jak i ukryta) nie ponosze odpowiedzialnosc ;-) czytasz to na wlasna reke

A wiec... umnie ladowanie wyglada tak

[PHP] pobierz, plaintext 
<?php
//...,
	  private function LoadModule( $strModule )
	  {
		 //
		 $strModule = strtolower( $strModule );
 
		 if( file_exists( DIR . 'core' . $strModule .'.class.php' ) )
		 {
			include( DIR . 'core' . $strModule .'.class.php' );
 
			if( class_exists( $strModule ) )
			{
			   Core::get( $strModule, new $strModule );
			   return TRUE;
			}
			else
			{
			   throw new Exception( 'Core::LoadModule - Nie istnieje klasa ' '. $strModule .' ' .' );
			   return FALSE;
			}
		 }
		 else
		 {
			throw new Exception( 'Core::LoadModule - Plik ' '. $strModule .'.class.php' nie istnieje.' );
			return FALSE;
		 }
 
	  }
//...
?>
[PHP] pobierz, plaintext 

Ale to moduly core... co do modolow typu menu news itp. bedzie do tego dodatkowa klasa ktora bedzie je odpowiednio ladowac tzn.
Wpierw sa pobierane dane z bazy danych nastepnie sa interpretowane, poczym zostaja zalaczone i tam odpowiednia czesc kodu nimi steruje, wysylaja dane i takowe o ile sa sa wysylane posrednio do widoku ktory je interpretuje...

Co do dziedziczenia w modelu nie wiem czy to napewno trafne. Osobiscie wolal bym obiekt wrzucic do protected i tam na nim operowac (w zaleznosc od tego co che uzywskac laduje sie odpowiednia klasa).

Ogulnie to malo co zrozumialem z pytania wiac odpowiedz tez moze byc malo zrozumiala i nie trafna

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?

mi obecne dzialanie kodu odpowiada

sorry za OT: ale czemu macie takie same avatary ? trudno was rozronic

Dlatego ze nalezymy do tej samej agencji (nie towarzyskiej )
Reszte tego typu pytan wysylaj raczej na PW. Odpowiedzialem tutaj zeby juz nikt nie spamowal na ten temat.