Włączanie plików + autoloader, chętnie bym posłuchał ciekawych pomysłów Opcje

[hawk]

Problem stary jak świat: system wymaga włączenia sporej liczby plików i zarządzanie tym jest upierdliwe. Do tego nie należy włączać więcej kodu niż potrzeba, a najlepiej zrobić jakieś lazy load.

Oczywiście, technik jest wiele:

1) require_once rozsiane po plikach, najlepiej poprzedzone jakąś stałą, np. require_once ROOT_DIR . '/foo/Foo.class.php';

2) Prado: deklarujemy namespacy - np. za pomocą funkcji using(), co dodaje nam ścieżki do include_path, a potem niech php znajdzie klasę.

3) Autoloader + mapa (nazwa klasy => ścieżka do pliku); autoloader wczytuje mapę i na jej podstawie jest w stanie znaleźć każdą klasę

Są jeszcze jakieś inteligentne sposoby? Dobry mechanizm powinien być odporny na "przemeblowanie" struktury plików (np. chcemy połączyć kilka klas w jeden plik).

BTW, włączanie plików bez klas (tylko funkcje i kod) jest gorsze, bo nie ma tego czegoś, czego można szukać po plikach... kolejna zaleta OOP?

[DeyV]

Aby sprawę jeszcze bardziej skomplikować - warto pamietać o tym - że require_once i inlude_once jest w rzeczywistości bardzo wolne = każde wywołanie tych funkcji, nawet jeśli tym razem NIE MUSIAŁA ona ładować tego pliku, bo już był załadowany wcześniej, trwa niemal tyle samo, co require()
proste porównanie:
http://www.phpinsider.com/smarty-forum/viewtopic.php?t=4323
I choć, jak ktoś to zauwazył w tym wątku, fizyczny zysk czasu jest niewielki, to jednak procentowo - różnica jest ogromna.

[Imperior]

Myślę, że moje rozwiązanie pomoże Ci, jeśli wpadniesz na jakiś dobry pomysł, to daj znać:
(nawiasem mówiąc każda klasa przekłada się na lokalizację w ten sposób:
Projekt_Podgrupa_Costam -> /packages/projekt/podgrupa/projekt_podgrupa_costam.php
zdecydowałem się na to, aby każdy plik miał pełną nazwę klasy, a instalator tworzył pakiety klas.)
Plik poza_drzewkiem/packages/noname/noname_package.php

[PHP] pobierz, plaintext 
<?php
define('NN_SEP', DIRECTORY_SEPARATOR);
define('NN_LIB', dirname(dirname(__FILE__)).NN_SEP);
define('NN_PRIVATE', dirname(NN_LIB).NN_SEP);
 
function __autoload($sName) {
	if (!(CORE :: Import($sName) || CORE :: ImportEx($sName))) {
		echo 'Kaboooom! Reason? '.$sName.' not found!!!';
	}
}
 
class CORE {
	public static function Import($sName, $sFind = null, $bSearch = true) {
		if (!isset($sFind)) {
			$sFind = $sName;
		}
		if ($bSearch && (class_exists($sFind, false) || interface_exists($sFind, false))) {
			return true;
		}
		$sName = strtolower($sName);
		$aArr = explode('_', $sName);
		array_pop($aArr);
		$sPath = NN_LIB.implode(NN_SEP, $aArr).NN_SEP.$sName.'.php';
		if (is_readable($sPath)) {
			include_once ($sPath);
			if ($bSearch && !(class_exists($sFind, false) || interface_exists($sFind, false)) ) {
				return false;
			}
			return true;
		}
		return false;
	}
 
	public static function ImportEx($sName) {
		$aArr = explode('_', $sName);
		array_pop($aArr);
		array_push($aArr, 'package');
		if (CORE :: Import(implode('_', $aArr), $sName) || CORE :: Import($sName)) {
			return true;
		}
		return false;
	}
}
?>
[PHP] pobierz, plaintext 

(zamiast kaboom w __autoload() będzie komunikat o tym, że strona nieczynna i próba poinformowania admina)

Pliki *_package.php zawierają zlepek klas najczęściej wykorzystywanych w danym pakiecie (noname_package.php zawiera noname_(core,timer,exception itp).php . )
ImportEx różni się tym, że najpierw spróbuje władować pakiet (czyli np. jak władowywyję jakąś usługę, dajmy SQL, to automatycznie mam wyjątki, klase do łączenia, klase do zapytań itp.)

Jeśli jestem pewien, że jakaś klasa/interfejs powinna być w systemie to poprostu z niej korzystam - jeśli nie jest władowana, to zrobi to autoload. (Klasa z pakietu nie musi sprawdzać innych z tego samego pakietu, najczęściej stosowana przeze mnie metoda)
Jeśli wgrywam usługę (auth, sql, cache...) to daję CORE::ImportEx, żeby wgrać pakiet.
Jeśli potrzebuję pojedynczą klasę to poprostu CORE::Import.

Obie funkcje Import i ImportEx zwracają true/false, czyli ustrzegają mnie przed wykorzystaniem nieistniejącej klasy, a co za tym idzie przed Fatal Error.
Przed próbą odczytania pliku oczywiście jest sprawdzenie, czy nie jest obecna już dana klasa/interfejs.

[hawk]

@Imperior:

Dwie słabe strony tu widzę:

1) Wszystko musi siedzieć w 2 miejscach: NN_LIB i NN_PRIVATE, co utrudnia model działania typu "user ściąga pakiet, wrzuca gdzie mu się podoba i chce żeby to działało".

2) Nie da się tego połączyć z zewnętrznymi bibliotekami, tzn takimi, gdzie nie ty decydujesz o nazwach klas.

Za to podobają mi się dwie inne rzeczy:

1) Importowanie całych "pakietów".

2) Nazwa_Klasy niejako emuluje pakiety, których php nie ma (nie możesz zrobić $foo = new com.example.Foo).

A mi się coraz bardziej podoba mapa:

[PHP] pobierz, plaintext 
<?php
$map = array(
  'MojaKlasa' => MY_DIR . 'foo/MojaKlasa.class.php',
  'MojaInnaKlasa' => MY_DIR . 'bar/KilkaKlas.inc.php',
  'MojaInnaKlasa2' => MY_DIR . 'bar/KilkaKlas.inc.php',
  'Smarty' => SMARTY_DIR . 'Smarty.class.php',
}
?>
[PHP] pobierz, plaintext 

Autoloader jest w tym momencie banalny. Takie mapy można nawet generować automatycznie przeszukując skryptem katalogi i parsując (tokenizując) pliki php. Można automatycznie łączyć mapy dostarczane z pakietami w jedną wspólną. Można dowolnie łączyć pliki w pakiety, wrzucać do jednego pliku, dzielić itd., a potem przegenerować mapę i dalej wszystko działa. Szczególnie podoba mi się to w kontekście trzymania "normalnej" hierarchii plików u siebie i dystrybuowania wszystkiego w jednym pliku po wycięci komentarzy i whitespaces.

Wada - trzeba ten głupi plik wczytać zanim autoloader cokolwiek zrobi . I ciężko byłoby automatycznie wstawić stałe jak w przykładzie powyżej.

[bela]

Ja mysle, zeby rozwiazac to na podobnej zasadzie jak w javie. Mianowicie :

Mamy metode import, ta zaś wczytuje plik/pliki jakie chcemy. Czyli robimy:

[PHP] pobierz, plaintext 
<?php
import('pl.bela.foo.bar.Bar');
import('pl.bela.foo.*');
?>
[PHP] pobierz, plaintext 

W pierwszym przypadku wczyta klase Bar ktora jest w katalogu pl/bela/foo/bar/, a w drugim wczyta wszystkie klasy znajdujące się w katalogu pl/bela/foo, z wyjatkiem pl.bela.foo.bar.Bar bo informacja o tym ze taka klasa jest załadowa jest przetrzymywane w tablicy ( 2 posty wyzej jest wyjasnienie czemu ).

Co do położenia katalogow pl/bela, net/php/smarty, nie przejmuje sie tym za bardzo, ponieważ są classpathy

Aha, i nie definiuje package(); no bo po co

[Imperior]

Dwie słabe strony tu widzę:

1) Wszystko musi siedzieć w 2 miejscach: NN_LIB i NN_PRIVATE, co utrudnia model działania typu "user ściąga pakiet, wrzuca gdzie mu się podoba i chce żeby to działało".

2) Nie da się tego połączyć z zewnętrznymi bibliotekami, tzn takimi, gdzie nie ty decydujesz o nazwach klas.

ad 1. NN_LIB jest podrzędne do NN_PRIVATE. W NN_LIB są trzymane właśnie pakiety, dla mnie nie do pomyślenia jest coś takiego, że user wrzuca jakiś pakiet gdzie mu się żywnie podoba, zamiast tego jest Instalator, przez którego może ściągnąć pakiet, uploadować, wskazać na serwerze i na podstawie plików konfiguracyjnych w pakiecie wszystko jest pięknie i ładnie instalowane razem z weryfikacją zależności.

ad 2. Z pozoru... Jeśli pliki będą w NN_LIB to zostaną załadowane, jedynie musi być spełniony warunek, żeby nie miały '_' w nazwach... (a jeśli tak, to żeby były prawidłowo umieszczone, ale wtedy probelmu nie ma )

Ciekawe z tymi mapami... a gdyby rozszerzyć je o jakieś wyrażenia regularne?

[hawk]

@bela_666: Takie rozwiązanie na pewno pasuje do javy, ale czy pasuje do php? Wszystko rozbija się o wydajność. Trzeba te importy przeczytać, pozamieniać na slashe, znaleźć katalogi, wczytać pliki... Owszem, mamy classpath (tzn. include_path ), ale w php jest to kosztowna (IMHO, nie robiłem testów) opcja, bo szuka po dysku przy każdym requeście. A przecież nie pozbędziemy się bibliotek, które nie stosują się do naszego standardu (smarty, adodb, ...). Czyli classpath się rozrasta.

Do tego, w php opłaca się wczytywać tylko te pliki, które są niezbędne. W javie nie ma tego problemu, bo wczytujesz raz i masz spokój. W php starasz się zminimalizować ilość operacji dyskowych na jedno żądanie.

W javie importy i w ogóle pakiety są potrzebne, bo 2 klasy w różnych pakietach mogą mieć tą samą nazwę i nie jest to problem. W php taka sytuacja jest niedozwolona, więc i pakiety są niepotrzebne (tak naprawdę jest to zabronione właśnie dlatego, że nie ma pakietów, ale skutek jest ten sam).

W javie z importami jest jeden problem: w pewnym momencie robi się ich zbyt dużo i upraszczamy sobie życie pisząc import java.util.*. W php jest to nie do przyjęcia - za dużo niepotrzebnie włączanego kodu. Więc nie muszę się martwić o unikalność com.example.hawk, bo to i tak nic mi nie daje.

@Imperior: sens regexpów zależy od tego, czy będą one szybsze niż wymienienie wszystkich klas. Czyli od wydajności.

Podsumowując, idealnie chciałbym mieć:
- nie muszę w swoich plikach php pisać require_once, przypominać sobie gdzie są potrzebne klasy, oraz upewniać się że nie pominąłem potrzebnego interfejsu
- mogę obsługiwać zewnętrzne (nie moje) biblioteki tak samo (tzn też bez pamiętania o require_once smarty)
- mogę w trakcie developmentu zmienić strukturę moich plików z klasami, przenieść do innych katalogów, itd. bez poprawiania mojej aplikacji
- mogę zmienić coś w n-tej dystrybucji pakietu bez obawy, że kod użytkowników nie znajdzie mojej klasy
- włączane są tylko te klasy, które są aktualnie potrzebne, i narzut czasy wykonania jest minimalny

[hawk]

@serafin: Możesz napisać więcej? Mi się zawsze wydawało że php nie pozwala na kropki w nazwie klasy. Tak bardzo mi się wydawało, że nigdy nie sprawdziłem. I czy to oznacza, że pełna nazwa klasy rzeczywiście brzmi System.Console.Out? Żeby autoloader mógł znaleźć klasę.

Bo problem z brakiem namespaces jest taki, że jak napiszemy $foo = new System.Console.Out, to php nie wie, że ma szukać klasy Out w System.Console, tylko szuka System.Console.Out.

[Imperior]

Bo problem z brakiem namespaces jest taki, że jak napiszemy $foo = new System.Console.Out, to php nie wie, że ma szukać klasy Out w System.Console, tylko szuka System.Console.Out.

A ściślej mówiąc to stara się odczytać stałe i połączyć je w jeden string.

Ja to zrozumiałem w ten sposób, że pisze kod, który zawiera błędy z punktu widzenia parsera php, a który jest parsowany, aby poprawić te błędy i dopisać jakieś includy itp.

Alternatywnie poprostu w kodzie jest coś jak:
uses 'System.Console.Out';
i to jest odczytywane, żeby do pliku wynikowego dodać includy.

serafin: czy coś w tym stylu miałeś na myśli?

[Vengeance]

A co Wam się stanie gdy zamiast System.Out będzie System_Out ?
Czy warto stosować te wszystkie parsery/tokenizery czy co tam jeszcze (opóźniające czas wykonywania) aby wygrać "walke" z kropką ?

[hawk]

Nic się nie stanie, poza utratą estetyki . Chociaż kwestia "kropka czy kreska" nie jest tutaj podstawowa. Problem jest raczej w tym, że autoloader będzie szukał klasy System_Out, a nie klasy Out w pakiecie System. Czyli na braku pakietów. I na tym, że na pewno znajdą się biblioteki, gdzie kreska jest używana do czegoś innego, co może wprowadzić system w błąd.

[Vengeance]

ale to jest php a nie java

Co do autoloadera... co za problem zrobić jakiś explode po znaku _
i odpowiednio sprawdzić?

[hawk]

Co za problem? Sprawdzanie po zrobieniu explode jest kosztowne. Odwołanie do dysku, przemnożone przez ileś tam włączanych klas. A ja zakładałem sobie idealnie, że autoloader, jako "jądro jądra", musi być super szybki.

Żeby nie było że się czepiam: to nie jest zła metoda, na razie nie zakodowałem samemu nic mądrego (stąd ten wątek). Zawsze są zady i walety, ja tylko wymieniam słabe strony.

[Seth]

Troche to stary kod ale moze sie przyda:

[PHP] pobierz, plaintext 
<?php
define( 'INCLUDE_PATH', 'include/' );
 
/**
 * Use this function to include classes in given namespace pattern
 *
 * @param $nsPattern string
 * @return true if succed, false otherwise
 */
function Import( $nsPattern )
{
  // Filter pattern namespace
  if ( preg_match( '/^[a-z0-9_]+(.[a-z0-9_]+){0,}(.*)?$/i', $nsPattern ) )
  {
	$nsPattern = str_replace( '.', '/', $nsPattern );
 
	if ( preg_match( '/^(.*)/*$/', $nsPattern, $match ) )
	{
	  $patch = INCLUDE_PATH.$match[1];
 
	  if ( $handle = opendir( $patch ) )
	  {
		while ( false !== ( $file = readdir( $handle ) ) )
		{
		  $i = 0;
 
		  if ( $file != '.' && $file != '..' && preg_match( '/^(.*).class.php$/', $file, $match ) )
		  {
			$filePath = $patch.&#092;"/\".$file;
 
			if ( file_exists( $filePath ) && is_file( $filePath ) && !class_exists( $match[1] ) )
			{
			  require_once( $filePath );
 
			  $i++;
			}
		  }
		}
 
		closedir($handle);
	  }
 
	  return ( $i ? 1 : 0 );
	}
	else
	{
	  $className = substr( strrchr( $nsPattern, '/'), 1 );
	  $filePath = INCLUDE_PATH.$nsPattern.&#092;".class.php\";
 
	  if ( file_exists( $filePath ) && is_file( $filePath ) && !class_exists( $className ) )
	  {
		require_once( $filePath );
 
		return 1;
	  }
 
	  return 0;
	}
  }
 
  // Wrong pattern
  return 0;
}
?>
[PHP] pobierz, plaintext 

Wywolania:
Import( 'db.*' );
includuje wszystkie klasy z katalogu include_path.'db/'

Import( 'db.jakas_klasa' );
includuje klasy o nazwie jakas_klasa z katalogu db

Pliki z klasami musza miec w nazwie .class.php

Aha... pisany pod php 4, wiec bez autoloadera :/

Ten post edytował Seth 25.02.2005, 12:53:16

[bela]

Seth, a czy te pregi nie spowalniają bardzo przy dużej ilości plików ?

[Seth]

Jakies spowolnienie napewno jest ale za to mozna latwo zaincludowac kilka klas

[bela]

Podsumowując, idealnie chciałbym mieć:
- nie muszę w swoich plikach php pisać require_once, przypominać sobie gdzie są potrzebne klasy, oraz upewniać się że nie pominąłem potrzebnego interfejsu
- mogę obsługiwać zewnętrzne (nie moje) biblioteki tak samo (tzn też bez pamiętania o require_once smarty)
- mogę w trakcie developmentu zmienić strukturę moich plików z klasami, przenieść do innych katalogów, itd. bez poprawiania mojej aplikacji
- mogę zmienić coś w n-tej dystrybucji pakietu bez obawy, że kod użytkowników nie znajdzie mojej klasy
- włączane są tylko te klasy, które są aktualnie potrzebne, i narzut czasy wykonania jest minimalny

No chyba udało mi się to osiągnąć, mianowicie: tokenizer przejezdza po wszystkich katalogach i załączą pliki .php, a przy okazji pomija katalogi .svn ( coś się zawieszało ), wyciąga nazwy klas/interfejsów i to gdzie one się znajdują, wciska to do tablicy i generuje piękny pliczek autoload.php, wraz z funckją __autoload.

No, to by było na tyle

[Nievinny]

Jeśli mogę wtrącić trzy grosze, ja myślę, że można przejechać po katalogach czymś takim:

[PHP] pobierz, plaintext 
<?php
 
$Dir = new RecursiveIteratorIterator( new RecursiveDirectoryIterator( ROOT_PATH ), true );
	foreach( $Dir as $File ) {
		if( $File->getFilename() === $sFile ) {
			try {
				CORE::loadClass( $File->getPathname() );
			}
			catch( CoreExceptions $Error ) {
				print Debuger::errorDisplay( $Error );
				exit;
			}
			break;
		}
	}
 
?>
[PHP] pobierz, plaintext 

Tylko trzeba zdefiniować stała ROOT_PATH i dodać do nazwy rozszerzenie i zdefiniować klasę CORE.
Warunki:

• nazwy klas musza byc takie same ja nazwy plikow (bez rozszerzenia) lub odwrotnie
• Może przy dużej ilości katalogów spowoli trochę, ale można dodać funkcje ładowania kilku klas i odrzucić katalogi

Za jakiś czas dołącze dokładniejszy kod, może trochę wolnawe, ale samo znajdzie ścieżkę dostępu do plików

[hawk]

@bela_666: pokaż pokaż pokaż

[Vengeance]

Nievinny: jakbyś zrobił cache jakiś to ok. Teraz uważam, że to jest najgorsze z najgorszych rozwiązań

[bela]

@bela_666: pokaż pokaż pokaż

Heh, ale to dopiero brzydko wyglądający prototyp, ale ok Pisałem, aby w ogóle działało

[PHP] pobierz, plaintext 
<?php
error_reporting(E_ALL|E_STRICT);
function get_microtime() {
	list($usec, $sec) = explode(&#092;" \",microtime());
	return ((float)$usec + (float)$sec);
}
 
global $fileContext;
$fileContext = array();
$start = get_microtime();
iterateDir('.'/*dirname(__FILE__)*/);
 
function iterateDir($path) {
	$di = new DirectoryIterator($path);
	foreach ($di as $k => $v) {
		// get extension
		$ext = end(explode('.', $v->getPathname()));
		// is svn dir
		$name = $v->getFilename();
		$first = substr($name, 0, 1);
		$first == '.' ? $dupa = false : $dupa = true;
 
		// if is dir and not svn dir, iterate this dir;]
		if($v->isDir() && $dupa) {
			iterateDir($v->getPath() . '/' . $v->getFilename());
		}
		// print filename with path, with php extenstion
		if($dupa && $v->isFile() && $ext == 'php' && $v->getFilename() != 'tokenizer.php' && $v->getFilename() != 'test.php' && $v->getFilename() != 'config.php' && $v->getFilename() != 'dupa.php') {
			$GLOBALS['fileContext'][] = array($v->getPathname(), token_get_all(file_get_contents($v->getPathname())));
			//print $v->getPathname() . \"<br />n\";
		}
	}
 
}
 
//dump($fileContext);
$classes = array();
$key = 0;
foreach($fileContext as $file) {
	static $i;
	foreach ($file as $c) {
		if(is_array($c)) {
			$a = false;
			foreach ($c as $k => $v) {
				if(is_array($v)) {
					if($v[0] == T_CLASS || $v[0] == T_INTERFACE) {
						$a = true;
					} elseif ($a && $v[0] == T_STRING) {
						$classes[] = array($v[1], substr($file[0], 2));
						$a = false;
					}
				}
				else
				{
					//print(\"$v<br/>n\");
				}
			}
		}
 
	}
 
}
 
//dump($classes);
 
$phpCode  = '<?php function __autoload($name) {' . &#092;"n\";
$phpCode .= 'static $map = array(' . &#092;"n\";
foreach ($classes as $k => $v) {
	$phpCode .= ''' . $v[0] . '' => '' . $v[1] . '','. &#092;"n\";
}
$phpCode .= ');' . &#092;"n\";
$phpCode .= 'require_once($map[$name]);' . &#092;"n}n\";
$phpCode .= '?>';
file_put_contents('autoload.php', $phpCode);
//dump($fileContext);
function dump($dump) {
	print('<pre>');
	var_dump($dump);
	print('</pre>');
}
print ($end = get_microtime() - $start . '<br />');
?>
[PHP] pobierz, plaintext 

[Vengeance]

Połączyłem kody: bela_666, Nievinny oraz Imperiora i osiągnołem taki efekt:

[PHP] pobierz, plaintext 
<?php
      public function getClassesMap($sDirName)
      {
         $oRecursiveDirectory = new RecursiveIteratorIterator( new RecursiveDirectoryIterator( $sDirName ), true );
         foreach ($oRecursiveDirectory as $oFile)
         {
            if(!is_file($oFile->getPathname()))
               continue;
            $aTokenAll = token_get_all(file_get_contents($oFile->getPathname()));
            $bIsClass = false;
            foreach ($aTokenAll as $iToken)
            {
               if (!is_string($iToken))
               {
                  list($iTokenID, $sTokenValue) = $iToken;
                  switch ($iTokenID)
                  {
                     case T_CLASS:
                        $bIsClass = true;
                        break;
                     case T_INTERFACE:
                        $bIsClass = true;
                        break;
                     case T_STRING:
                        if ($bIsClass)
                        {
                           $aClassesMap[$sTokenValue] = $oFile->getPathname();
                           $bIsClass = false;
                        }
                        break;
                     case T_WHITESPACE:
                        break;
                     default:
                        $bIsClass = false;
                        break;
                  }
               }
            }
         }
         return $aClassesMap;
      }
?>
[PHP] pobierz, plaintext 

Ten post edytował Vengeance 26.02.2005, 12:37:36

[Vengeance]

Linia nr 9: zmieniasz ją na:

Kod

           if(!is_file($oFile->getPathname()) OR $oFile->getFileName == '.svn')

Powinno zadziałać

[Nievinny]

@Vengeance -> Czy twoim zdaniem ładowanie z mapy powinno być zaimplementowane od razu w __autoload() czy dać jakąś funkcję pośrednią?

OK, a ja myślę, że można to jeszcze przerobić i jeszcze jedno: kiedy należy a kiedy nie robić mapę plików?

I czy pozwolisz wykorzystam to w swoim CORE?

Zobaczcie coś takiego i oceńcie

[PHP] pobierz, plaintext 
<?php
 
define( 'IN_CORE', true );
 
// ---- Definicja stałych
define( 'ROOT_PATH', './' );
define( 'PHPEX', '.php' );
 
// ---- Załadowanie systemów
include_once( ROOT_PATH . 'lib/debuger/Debuger' . PHPEX );
include_once( ROOT_PATH . 'lib/CORE' . PHPEX );
 
// ---- Mapa plików
$aMap = CORE::getClassesMap( ROOT_PATH );
// ---- Funkcja autoładowania
function __autoload( $sFile ) {
	global $aMap;
	$bLoad = false;
	foreach( $aMap as $sName => $sValue ) {
		if( $sName === $sFile ) {
			try {
				CORE::loadClass( $sValue, $sFile );
			}
			catch( CoreExceptions $Error ) {
				print Debuger::errorDisplay( $Error );
				exit;
			}
			$bLoad = true;
			break;
		}
	}
	if( $bLoad === false ) {
		print Debuger::errorLiteDisplay( 'Nie można odnaleźć podanej klasy' );
		exit;
	}	
}
 
?>
[PHP] pobierz, plaintext 

Mapa jest tworzona tylko raz, a potem porównywana w pentli foreach? Czy lepiej inaczej, np:

[PHP] pobierz, plaintext 
<?php
 
function __autoload( $sFile ) {
	global $aMap;
	if( isset($aMap[$sFile] ) ) {
		try {
			CORE::loadClass( $aMap[$sFile], $sFile );
		}
		catch( CoreExceptions $Error ) {
			print Debuger::errorDisplay( $Error );
			exit;
		}
 
	}
	else {
		print Debuger::errorLiteDisplay( 'Nie można odnaleźć podanej klasy' );
		exit;
	}	
}
 
?>
[PHP] pobierz, plaintext 

Chyba tes sposób lepszy, czy nie?

Ten post edytował Nievinny 26.02.2005, 17:25:44

[Vengeance]

Nievinny: chodzi oto, aby mape generować tylko RAZ!! Potem jak coś dodasz/zmienisz dopiero robisz ją ponownie. W moim przypadku robienie mapy zabiera 95% czasu wykonywnia skryptu! Dlatego tak ważnym jest aby wynik był objęty jakimś cache. A czy zrobisz w __autoload() czy w czymkolwiek innym to już nie jest takie ważne

[Nievinny]

Nievinny: chodzi oto, aby mape generować tylko RAZ!! Potem jak coś dodasz/zmienisz dopiero robisz ją ponownie. W moim przypadku robienie mapy zabiera 95% czasu wykonywnia skryptu! Dlatego tak ważnym jest aby wynik był objęty jakimś cache. A czy zrobisz w __autoload() czy w czymkolwiek innym to już nie jest takie ważne

Źle mnie zrozumiałeś, to, że mapa powinna być robiona tylko raz to ja doskonale wiem....
Chodzi mi o samą funkcję __autoload() czy lepsza jest z foreach czy ta druga...

Ten post edytował Nievinny 26.02.2005, 17:44:05

[Vengeance]

A po co foreach? Jeszcze opozniac ? To $map[$className] jest takie zle? W sumie to i tak kazdy robi jak mu wygodniej ;]

[hawk]

@Vengeance et al: No, bardzo ładny kod . Ja bym dodał parametryzację/konfigurację generatora mapy. Coś takiego jak ma phpdocumentor - jakie katalogi ma przeglądać, jakie pliki ma czytać. Wtedy można sobie zignorować .svn i nie trzeba tego zakodowywać na stałe w kodzie.

A potem tylko zrzucić mapę do jakiegoś pliku php i voila!

I na koniec coś takiego:

[PHP] pobierz, plaintext 
<?php
function __autoload($sClass) {
  ClassLoader::load($sClass);
}
 
class ClassLoader {
  private static $aMap;
 
  public static function addMap($aMap) {
	self::$aMap = self::$aMap == null ? $aMap : array_merge(self::$aMap, $aMap);
  }
 
  public static function load($sClass) {
	if (array_key_exists($sClass, self::$aMap)) {
	  require self::$aMap[$sClass];
	}
  }
}
?>
[PHP] pobierz, plaintext 

[Vengeance]

Ok userzy forum.php.pl "wymyslili" juz idee "routera" (a raczej po prostu to nazwali ladnie). Teraz czas nazwac to co tu razem stworzyliśmy

[Nievinny]

Nazwa: Autoload Class System...

może być, a klasa niezła, tylko, że robi zrzuty przy każdym wywołaniu skryptu a nie ciągnie z cache

Pozwolę sobie jeszcze dopisać, cos o systemie cache:
1) Jest absolutnie niezbędny:

• Czas gen bez cache: 0.0617 sec
• Z cache: 0.0111 sec
• Czyli o 0.0516 sec szybciej
• Czyli ok 5,5 raza szybciej

Jak ktoś jest zainteresowany klasą cache to na PW

Ten post edytował Nievinny 3.03.2005, 14:44:37

[chmolu]

Ja to rozwiązałem podobnie, jak w nowym Mojavi:

[PHP] pobierz, plaintext 
<?php
  function __autoload($class_name)
  {
	  $filename = ConfigManager::getConfig('autoload', 'autoload', $class_name);
	  if (!empty($filename))
	  {
		  /* include a file with the class definition */
		  require_once $filename;
	  }  
	  else 
	  {
		  /* unknown class */
		  throw new Exception(sprintf('Autoloading of class \"%s\" failed', $class_name));
	  }
  }
?>
[PHP] pobierz, plaintext 

Lista plików jest przechowywana w pliku konfiguracyjnym (ini). Bardzo elastyczne rozwiązanie. Trzeba tylko włączyć klasę ConfigManager - reszta jest ładowana automatycznie.

[Nievinny]

A ja generujesz listę klas? Tu masz od razu generator

[chmolu]

Lista klas jest w pliku konfiguracyjnym autoload.ini, który wygląda tak:

[autoload]
Class1 = sciezkadopliku
Class2 = sciezkadopliku
Classn = sciezkadopliku

Plik ten jest parsowany do tablicy i następnie chache'owany jako plik .php

Muszę przemyśleć Twój sposób ze skanowaniem katalogów... ciekawe

Ten post edytował chmolu 4.03.2005, 16:37:08

[hawk]

@chmolu:
1) Z tego co pamietam, w __autoload nie mozna rzucac wlasnych wyjatkow, bo i tak nie beda dzialac. Czyzby bug w Mojavi?

2) Pliki ini sa w tym przypadku zdecydowanie gorsze od tablic php. W pliku ini nie mozna zrobic czegos takiego:

[PHP] pobierz, plaintext 
<?php
$map = array(
  'FooClass' => FOO_DIR . '/Foo.class.php',
);
?>
[PHP] pobierz, plaintext 

Kazdy, kto bedzie uzywal systemu z plikiem ini, bedzie musial zmienic wszystkie sciezki w tym pliku, bo domyslne nie beda u niego dzialaly.

3) Jaki jest sens kompilowac plik ini do pliku php? Jezeli pliki php sa zdecydowanie szybsze, to dlaczego nie zaczac wlasnie od nich? Dla mnie to jest komplikowanie sobie zycia.

[chmolu]

1) Z tego co pamietam, w __autoload nie mozna rzucac wlasnych wyjatkow, bo i tak nie beda dzialac. Czyzby bug w Mojavi?

Wyjątki można wyrzucać, tylko nie mozna ich łapać w catch. Wtedy są uznawane jako fatal error.

Pliki ini sa w tym przypadku zdecydowanie gorsze od tablic php. W pliku ini nie mozna zrobic czegos takiego:
...

kazdy, kto bedzie uzywal systemu z plikiem ini, bedzie musial zmienic wszystkie sciezki w tym pliku, bo domyslne nie beda u niego dzialaly.

Można zrobić coś takiego. Po prostu trzeba dodawać do każdego wpisu pseudo-stałą, np. Action = %APP_DIR%/modules/actions/action.php. ConfigManager automatycznie zamienia takie 'stałe' na właściwą scieżkę.

3) Jaki jest sens kompilowac plik ini do pliku php? Jezeli pliki php sa zdecydowanie szybsze, to dlaczego nie zaczac wlasnie od nich? Dla mnie to jest komplikowanie sobie zycia.

Wiesz, to jest indywidualna sprawa. Ja przyjąłem, że taka rzecz należy do konfiguracji, a nie do kodu aplikacji. Konfigurację zapisuję w plikach ini. Mogę oczywiście zapisywac od razu w plikach php. Dla mnie jest wygodniej to oddzielić na właściwy kod aplikacji i dane konfiguracji. Wcale nie jest to komplikowanie sobie życia. Całość ogranicza się do funkcji parse_ini_file(), var_export() i fputs().

Zależy, jak kto lubi

Ten post edytował chmolu 4.03.2005, 17:53:19

[Nievinny]

W sumie, skrypt po raz pierwszy zaczyna działanie więc generuje mapę, która jest potem cachowana. Gdy odpalimy skrypt po raz drugi odczyta mapę z cache (zserializowany ciąg + <?php die(); ?> ), usunie te ostatnie i przyśpieszenie jest widoczne
to sprowadza się ogólnie do:

[PHP] pobierz, plaintext 
<?php
 
if( !CORE::addMap( $Cache->loadCacheArray( 'aMap' ) ) ) {
	$aMap = CORE::getMap( './' );
	CORE::addMap( $aMap );
	$cache->saveCacheArray( 'aMap', $aMap );
}
// 'aMap' - uchwyt do pliku cache .php w ./cache/arrays/
 
?>
[PHP] pobierz, plaintext 

I wg mnie to najprostszy sposób

[Imperior]

1) Z tego co pamietam, w __autoload nie mozna rzucac wlasnych wyjatkow, bo i tak nie beda dzialac. Czyzby bug w Mojavi?

Zgadza się, w __autoload nie rzuca się wyjątków, ale mojavi3 nie rzuca go, tylko tworzy (bo tam ma zaimplementowane metody dot. informowania o bledach).

PS. Na początku też myślałem, że mają błąd, ale tam throw nie ma

[hawk]

@chmolu: Faktycznie, obsługa plików ini jest bardzo prosta. Co ciekawe, na początku próbowałem właśnie podejścia z "%APP_DIR%/...". Pliki ini wyglądają lepiej, prościej się je edytuje, trudniej popełnić błąd lub wrzucić niedozwolony kod. Z drugiej strony, trzeba wygenerować z nich pliki php, co samo w sobie jest banalne, ale wymaga sprawdzenia daty modyfikacji, czyli dodatkowego kroku. Wpływ na wydajność jest minimalny i to już chyba kwestia gustu.

[Bora]

Pliki ini są wczytywane szybciej niż php gdyż mają specjalny prostszy parser.
Porównajcie soebie a zobaczycie że czasami nawet cechowanie w php trwa dłużej.

[Vengeance]

A czy są szybsze od zserializowanej tablicy, zapisanej w pliku ?

[Imperior]

A czy są szybsze od zserializowanej tablicy, zapisanej w pliku ?

Jeśli to nie jest pytanie retoryczne, to: Nie.

[hawk]

@Bora: Pliki ini moze i sa wczytywane szybciej, ale zamiana %APP_DIR% na odpowiednia sciezke niestety zepsuje tutaj wydajnosc.

[orson]

witam ...

a moze w ini zrobic 2 sekcje ...

Kod

[path]
app_dir = "./jakis/dir/"
trans_dir = "./jakis/dir"
jakis_dir = "./jakis/dir"
[clases]
class = file_name
class = file_name
class = file_name

i potem parse ini ze znacznikiem true i mamy sklejanie stringow ...

[PHP] pobierz, plaintext 
<?php
$array['path']['app_dir'].$array['path']['class']
?>
[PHP] pobierz, plaintext 

i nie trzeba nic zamieniac ... i da sie latwo konfigurowac w tym samym ini ...

pozdrawiam

[hawk]

A w jaki sposob z gory okreslisz, ktora sciezke masz dokleic do konkretnej klasy? Nie zrobisz tego, chyba ze narzucisz sobie jakies bardzo scisle reguly, a tego wole uniknac.

[Nievinny]

A ja mam jeszcze pytanie dot. __autoload(). Czy jeśli wywołam w niej statyczną metodę która w razie niepowodzenia wywala wyjątek, to trzeb go obsłużyć w __autoload() czy przejdzie poziom wyżej?
Przykład:

[PHP] pobierz, plaintext 
<?php
 
function __autoload( $sClassName ) {
Application::loadClass( $sClassName );
}
 
//czy
 
function __autoload( $sClassName ) {
try {
Application::loadClass( $sClassName );
}
catch( NotFoundClassException $e ) {
//coscos
}
}
 
?>
[PHP] pobierz, plaintext 

Czy 1 sposób też może być? Czy wyjątek zostanie obsłużony?

[orson]

A w jaki sposob z gory okreslisz, ktora sciezke masz dokleic do konkretnej klasy? Nie zrobisz tego, chyba ze narzucisz sobie jakies bardzo scisle reguly, a tego wole uniknac.

witam ..

no przeciez normalnie ...

Kod

[path]
mainAppPath = '/home/user/public_hmtl/site/'
mainImagesPath = '/home/user/public_hmtl/site/images/'
cos tam
cos tam
default = 'clases/'
[clases]
dbclass = 'clases/db.class.php'
session = 'clases/session.class.php'
mail = '3rdpart/clases/mail/mail.class.php'
;z defaultem
dbclass = 'db.class.php'
session = 'session.class.php'

i potem:

[PHP] pobierz, plaintext 
<?php
 
print $iniFile['path']['mainAppPath'].$iniFile['clases']['dbclass'];
print $iniFile['path']['mainAppPath'].$iniFile['clases']['session'];
//defaultem
print $iniFile['path']['mainAppPath'].$iniFile['path']['default'].$iniFile['clases']['dbclass'];
 
?>
[PHP] pobierz, plaintext 

w sekcji clases tez moga byc sciezki podane, przeciez to lokalizacja pliku a nie nazwa klasy ... czyli dajesz najpierw najbardziej ogolna a potem w poszczegolnych klasach dodajesz, podkatalogi ... mozna na sztywno dodac default do kodu strony a jak cos sie zmieni [ale znowu bez przesady ... tak czesto sie nie zmienia i klasy i tak sa w 1 miejscu] to zmienic albo nawet wyzerowac default

pozdrawiam

[hawk]

@Orson: nie o to mi chodzi. Masz np. (a raczej użytkownik systemu ma) Smarty, kawałek WACT i w ogóle przynajmniej kilka bibliotek, których używa. Przecież nie będzie tak, że jeden super framework robi wszystko najlepiej, więc autoloader nie powinien być ograniczony do tego, co sami sobie wymyślimy, tylko po prostu działać tak jak chce użytkownik.

Nie powiesz teraz użytkownikowi, że ma to wszystko umieścić w innym katalogu. Będzie miał Smarty w jakimś niestandardowym miejscu, będzie chciał użyć razem z super-autoloaderem i będzie oczekiwał, że autoloader załaduje mu Smarty - od tego przecież jest.

I co wtedy? Skąd autoloader ma wiedzieć, że Smarty akurat nie jest pod mainAppPath?

PS

...mozna na sztywno dodac default do kodu strony a jak cos sie zmieni...

Nie można. Autoloader jest po to, żeby tego kodu w ogóle nie było. Nie ma gdzie dodawać defaulta.

[orson]

witam ...

hmm ... no to w pliku ini w sekcji clases podajesz

Kod

[clases]
smarty = /full/path/to/smarty/lib/smarty.php

a zeby bylo latwiej mozna np. wprowadzic ifa przy ladowaniu i np. jak nazwa klasy zaczyna sie od 2 // a nie od 1 to laduj bez patha .... czyli jak dopiszemy // przed smarty to oleje sekcje path i biblioteka moze byc gdziekolwiek ...
rozwiazanie z plikiem ini coraz bardziej mi sie podoba ... chyba zaczne go stosowac ...

pozdrawiam

[dr_bonzo]

@Nievinny: wyrzuc wyjatek w tej metodzie i sprawdz. Wyjatek wedruje w gore do ekranu (error) dopuki go nie zlapiesz gdzies po drodze.

[Nievinny]

Owszem wywala wyjątek, ale nie obsługuje go w zwykły sposób, a skrypt wychwytuje wyjątek "po drodze", tylko wywala fatal errora.

@dr_bonzo: To co mówisz byłoby dobre, gdyby funkcję wywoływać w ten sposób:

[PHP] pobierz, plaintext 
<?php
 
__autoload( $zmienna );
 
?>
[PHP] pobierz, plaintext 

A nie jeśli zostaje ona wykonana automatycznie... :/
Zostaje obsługa błędów na poziomie funkcji. Jeśli obsługujemy przez funkcję to pojawiają się problemy:
1) Jeśli klasą którą ładuje system jest klasa obsługi błędów?

[PHP] pobierz, plaintext 
<?php
 
//$zmienna = 'Debug';
function __autoload( $zmienna ) {
try {
CORE::zaladuj( $zmienna )
} catch( Exception $e ) {
print Debug::display( $e );
}
}
 
?>
[PHP] pobierz, plaintext 

Znajdą się też inne problemy... :/
Macie jakieś pomysły jak ładnie obsłużyć ten wyjątek?

[chmolu]

Rozwiązanie z generowaniem mapy przez skanowanie katalogów wydaje się najlepsze. Jednak sposób Vengeance'a nie jest zbyt efektywny. Wymaga wczytania zawartości każdego pliku z osobna i sprawdzania, czy zawiera on jakieś klasy. Może jest to bardziej elastyczne, gdy mamy po kilka klas w jednym pliku, ale ja to zrobiłem inaczej.

U mnie każda klasa jest w osobnym pliku. Nazwa pliku zawierającego definicję klasy lub interfejsu kończy się na .class.php lub interface.php. I tylko takie pliki są ładowany do mapy.

Wycinki kodu z mojego frameworka:

[PHP] pobierz, plaintext 
<?
  /**
   * Checks whether the specified path exists.
   * If the directory is not found, it'll try to create one.
   *
   * @param  string $dirname 
   * @return void
   */
  function ensure_directory_exists($dirname)
  {
	  $dirname = substr($dirname, strlen(APP_DIR));
 
	  $sub_dirs = explode('/', $dirname);
	  $path = '';
	  foreach ($sub_dirs as $dir)
	  {
		  $path .= $dir . '/';
		  if (!file_exists($path))
		  {
			  if (!@mkdir($path, 0777))
			  {
				  throw new DaeronException('Could not create a directory \"' . $path . '\"');
			  }   
		  }
	  }
  }  
 
  /**
   * Writes the supplied content to a file.
   *
   * @param  string $filename Name of a file to write to.
   * @param  mixed  $content  Data to write to a file.
   * @return void
   */  
  function file_write($filename, $content)
  {
	  ensure_directory_exists(dirname($filename));
	  $fp = @fopen($filename, 'w');
	  if($fp)
	  {
		  @fputs($fp, $content, strlen($content));
	  }
	  @fclose($fp);	  
  }  
 
  class Autoloader
  {
	  /**
	   * Map of class names and their localization. 
	   *
	   * @var array
	   * @static 
	   */
	  private static $map = null;
 
	  /**
	   * Loads specified class.
	   *
	   * @param  string $class_name
	   * @return void 
	   */	  
	  public static function load($class_name)
	  {
		  $cached_map = APP_DIR . '/var/cache/autoload.php';
		  if (!is_readable($cached_map))
		  {
			  self::generateMap();
		  }
 
		  if (!isset(self::$map)) require_once $cached_map;
 
		  $class_name = strtolower($class_name);
		  if (!isset(self::$map[$class_name]))
		  {
			  throw new Exception('Unknown class ' . $class_name);
		  }
 
		  require_once self::$map[$class_name];
	  }
 
	  /**
	   * 
	   * 
	   *	 
	   */
	  private static function generateMap()
	  {
		  self::$map = array();
 
		  $cached_map = APP_DIR . '/var/cache/autoload.php';
 
		  self::scan(DAERON_DIR);
		  self::scan(APP_DIR);
 
		  $compiled = &#092;"<?phpn\";
		  $compiled .= 'self::$map = ' . var_export(self::$map, true) . &#092;"n?>\";
 
		  file_write($cached_map, $compiled);		   
	  }	  
 
	  /**
	   * 
	   * 
	   *	 
	   */
	  private static function scan($dir)
	  {
		  $match = array();
		 
		  $rd = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($dir), true);
		  foreach ($rd as $file)
		  {
			  if(!is_file($file->getPathname())) continue;
 
			  if (preg_match('/([a-z0-9_]+).(class|interface).php$/i', $file->getPathName(), $match))
			  {
				  self::$map[strtolower($match[1])] = $file->getPathName();	
			  }
		  }		 
	  }
  }
   
  function __autoload($class_name)
  {
	  Autoloader::load($class_name);	
  }
?>
[PHP] pobierz, plaintext 

Porównanie szybkości (dla stosunkowo małej liczby plików):
- czytanie mapy z generowaniem: 35,67 ms
- czytanie z cache'owanej mapy: 00,57 ms

Kod Vengeance'a:
- czytanie mapy z generowaniem: 129.35 ms

A czytanie z cache wygląda identycznie

Ten post edytował chmolu 13.03.2005, 12:41:45

[Vengeance]

chmolu: Zauważ, że to Ty trzymasz każdą klasę w innym pliku!

A jak byś chciał dodać do frameworka np. AdoDB, Smarty czy inne rzeczy? One nie stosują twojego nazewnictwa.

Po to właśnie ten mój* skrypt przeszukuje wszystkie pliki wczytując je!

Skoro twoje nazewnictwo jest tak ustandaryzowane to w jakim celu w ogóle tworzyć mape ? przecież wystarczy zrobić

require $className . '.class.php'; hehe

* przypominam, iż nie czuje się właścicielem skryptu posklejałem kod z wielu wypowiedzi innych osób i wrzuciłem

[chmolu]

@Vengeance: to prawda, nie każdy musi trzymać każdą klasę w innym pliku. Dla mnie takie rozwiązanie jest najwygodniejsze.

A jak byś chciał dodać do frameworka np. AdoDB, Smarty czy inne rzeczy? One nie stosują twojego nazewnictwa.

Autoloader u mnie ma służyć tylko do ładowania klas należących do jądra i aplikacji. Biblioteki typu Smarty i AdoDB zaliczam do "zewnętrznych" części aplikacji, którymi zająć musi się odpowiednia akcja/widok. Poza tym zauważ, że w przypadku tych bibliotek musimy włączyć tylko jeden główny plik, który już sam zajmuje się potrzebnymi mu plikami.
(btw: główny plik Smarty ma nazwę Smarty.class.php, więc pasuje do mojego nazewnictwa )

Skoro twoje nazewnictwo jest tak ustandaryzowane to w jakim celu w ogóle tworzyć mape ? przecież wystarczy zrobić

require $className . '.class.php'; hehe

Ale dzięki mapie struktura katalogów jest dowolna. Mogę sobie w dowolny sposób podzielić na różne katalogi i podkatalogi, a gdy zajdzie potrzeba zmiany struktury wystarczy tylko wygenerowac mapę i nie trzeba grzebać w kodzie i poprawiać wszystkich ścieżek.

Ten post edytował chmolu 13.03.2005, 09:52:28

[bela]

@chmolu a jak zmienisz ścieżke do smarty ?

[chmolu]

Nie używam Smarty więc nie ma żadnego problemu

// btw: poprawiłem troszkę ten kod

Ten post edytował chmolu 13.03.2005, 12:43:29

[Vengeance]

@chmolu: ty nie. Ale z tego co wynika z twoich wypowiedzi to co piszesz
to framework. Pewnie kiedys ktos inny z niego bedzie korzystac i moze chciec takze korzystac ze smarty

[chmolu]

Zapewniam cię, że nie będzie to żaden problem
Ale mam nadzieję, że zamiast Smarty ludzie będą używali systemu wbudowanego w mój framework (wzorowany na WACT/ASP/JSP).
Jak skończę, to zarzucę tu kod do oceny.

Ten post edytował chmolu 13.03.2005, 13:57:45

[Ociu]

Pozwole sobie odkopać temat

Zastanawiam się czy nie użyć pliku xml ładowania klas. Coś na wzór:

[XML] pobierz, plaintext 
<Classes>
 
<dirname>
<class name='jakas tam nazwa'>file.php</class>
<class name='jakas tam nazwa2'>file2.php</class>
</dirname>
 
<dirname2>
<class name='jakas tam nazwa3'>file3.php</class>
<class name='jakas tam nazwa4'>file3.php</class>
</dirname2>
 
</Classes>
[XML] pobierz, plaintext 

Potem bardzo prosto: simplexml, loader instacji (w moim przypadku) i tyle

Iść dalej tym tropem, czy zacząć myśleć nad czymś innym ?

[davidD]

Tylko właściwie... po co XML do zwykłego przypisania nazwy pliku do nazwy klasy? IMO jest to lekkie komplikowanie sobie całej sprawy - no chyba, że chcemy przechowywać jakieś dodatkowe informacje.

Mnie osobiście podoba się pomysł z iteratorem, szukającym klas po wszystkich plikach o określonym wzorcu, w podanej ścieżce (ścieżkach). Można z tego zrobić ładną tablicę, zserializować ją i do pliku... Co prawda trzeba się troszkę pobawić samym pisaniem tego generatora mapy, ale za to później, przy zmianie położenia/nazw plików zaoszczędzi się duużo czasu.

[matid]

Ja to widzę tak:
1. Crowler, szuka po określonych folderach w poszukiwaniu plików php zawierających klasy. Będzie miał możliwość wyszukiwania z filtrowaniem nazw plików, nazwach klas itp. Być może po dodaniu opcji filtracji czas generowania mapy się zwiększy nieco, ale na jakoże jest to czynność wykonywana rzadko, można sobie na to pozwolić.
2. Dodatkowy plik XML z klasami zewnętrznymi. W tym pliku będą zapisane w postaci XML dodatkowe klasy, które autoloader ma obsługiwać. Plik byłby parsowany i razem z wynikami crowlera zapisywany w pliku php w postaci tablicy.

[Ociu]

2. Dodatkowy plik XML z klasami zewnętrznymi. W tym pliku będą zapisane w postaci XML dodatkowe klasy, które autoloader ma obsługiwać.

O to mi chodziło

Śpieszyłem się i zapomaniałem dopisać.

[chmolu]

IMO pliki XML w tym wypadku są niepotrzebne. Szybciej i wygodniej będzie to zrobić przy pomocy plików ini

[Nievinny]

XML to przeładowanie, a nam zależy na czasie, nie? Chociaż wszystko da się zrobić ;P

[matid]

XML to przeładowanie, a nam zależy na czasie, nie? Chociaż wszystko da się zrobić ;P

Przecież parsowanie pliku XML będzie się odbywać tylko w wypadku rekonstruowania mapy, co za często się dziać nie powinno, czyż nie? Zyskujemy natomiast na przejrzystości i strukturze pliku konfiguracyjnego.

[bigZbig]

@matid -> zauwazylem u Ciebie duze przywiazanie do technologi xml. Wiesz mnie sie zawsze wydawalo, ze ten jezyk zostal powolany do zycia po to, aby zapewnic przenosnosc danych pomiedzy roznymi platformami. Nie wiem czy w tym przypadku zapewnienie tej cechy jest pozadane. Uwazam ze próbujesz robic cos na sile utrudniajac sobie zywot.

[Nievinny]

@matid -> ale tą mapę musisz za każdym uruchomnienie skryptu załadować i to będzie się dziać często...

[matid]

@matid -> ale tą mapę musisz za każdym uruchomnienie skryptu załadować i to będzie się dziać często...

Przecież nie chodzi mi o trzymanie mapy w XMLu, bo jest to totalną pomyłką. Nie jestem aż tak przywiązany do XMLa żeby tracić na wydajności. Chodzi mi o trzymanie w pliku XML dodatkowych ścieżek do przeszukiwania, czyli parsowane będzie to tylko przy regenerowaniu mapy.

[Nievinny]

Czyli trzymasz tylko katalogi do parsowania? To nie ma sensu, ponieważ powinno się parsować wszystkie katalogi i uzyskać pełną mapę klas.

[Ociu]

Chodzi mi o trzymanie w pliku XML dodatkowych ścieżek do przeszukiwania, czyli parsowane będzie to tylko przy regenerowaniu mapy.

Przemyślałem sposób z XML i stwierdziłem, ze pomysł jest chybiony. Tak czy inaczej trzeba za każdym razem parsowac plik, traci się na tym cenny czas... A gdy tych dodatkowych ścieżek jest dużo ? wtedy czas wydłuża się do sek, a nie tysiącznych sekund.

[matid]

Przemyślałem sposób z XML i stwierdziłem, ze pomysł jest chybiony. Tak czy inaczej trzeba za każdym razem parsowac plik, traci się na tym cenny czas... A gdy tych dodatkowych ścieżek jest dużo ? wtedy czas wydłuża się do sek, a nie tysiącznych sekund.

Nie wiem jak sobie wyobrażasz wygenerowanie mapy w tysięczne sekund. W średniej wielkości projekcie (jeszcze bez czytania dodatkowych ścieżek, ok. 90 klas w różnych plikach) trwa to ok. 0,4 s. Czytanie z mapy to ok 0.0008s. Więc nie wiem jakbyś chciał kombinować, ale IMO niemożliwym jest generowanie mapy w tysięczne sekundy. Oczywiście mówię tutaj o mapie nie wymuszającej żadnego przystosowania nazwy pliku itp.

[Vengeance]

" trwa to ok. 0,4 s"

lol :] Mój kod (a ogolnie ten co podałem powyżej gdzieś), który rekurencyjnie schodzi katalogami w dół i zapisuje znalezione klasy już teraz wykonuje się czasem więcej niż 20 sekund :]
Oczywiście robione jest to tylko raz, przy pierwszym uruchomieniu.

[matid]

" trwa to ok. 0,4 s"

lol :] Mój kod (a ogolnie ten co podałem powyżej gdzieś), który rekurencyjnie schodzi katalogami w dół i zapisuje znalezione klasy już teraz wykonuje się czasem więcej niż 20 sekund :]
Oczywiście robione jest to tylko raz, przy pierwszym uruchomieniu.

Moja klasa autoloadera jest w znaczej części oparta na wypowiedziach i nawet fragmentach kodów z tego tematu, więc nie zdziwiłbym się, gdyby funkcja skanująca foldery (lub jej założenia) była już przez ciebie cytowana.

Prędkości w dostępie do plików mogą wynikać z różnego systemu operacyjnego (ja testuję lokalnie na linuksie, który ma świetny system buforowania dysków), procesora, pamięci, obciążenia, itp.
U mnie przeskanowanie ok 100 plików (ok. 1 MB) i stworzenie mapy zawierającej 90 klas trwa ok. 0,4s.

Oprócz tego mam napisany bardzo fajny system automatycznego regenerowania mapy, cachowania wyników, itp.

Ten post edytował matid 10.05.2005, 21:49:35

[Ociu]

Czepiasz się szczegółów, liczba była podana dla przykładu.

[Vengeance]

To nie było czepianie. Chodziło mi o sposób w jaki @matid osiągnął tak dobry jak dla mnie wynik. U mnie już powoli php robi time out przy generowaniu :] (WinXP, 950 MHz, 256mb RAM)

[matid]

Dla zainteresowanych - moja klasa Autoloader: http://matid.emmatrade.pl/files/Autoloader.class.phps

I jeszcze raz moje testy:
Sprzęt: Pentium 4 2,4 GHz, 1024 MB RAM, dysk twardy 7200 RPM (system plików: ReiserFS)
System: Ubuntu Linux 5.04 Hoary Hedgehog

Dane testowe:
Rozmiar: 913.6 KB
Liczba plików: 120
Liczba klas: 90

Czas generowania mapy po raz pierwszy po włączeniu komputera: 0.872271060944s
Czas generowania mapy po raz kolejny (część plików mogła być w buforze dysków): 0.427920103073

Ten post edytował matid 11.05.2005, 21:23:57

[hawk]

Skoro temat zrobił się popularny i każdy chce mieć swój autoloader oparty o mapy (co uważam za słuszne), to warto załatwić jedną prostą rzecz. Kompatybilność. Taka mapa to na tyle prosta struktura, że pewnie wszędzie wygląda tak samo:

[PHP] pobierz, plaintext 
<?php
$map=array(
  'nazwaklasy' => 'ścieżkadoklasy',
);
?>
[PHP] pobierz, plaintext 

Wtedy każdy może sobie generować mapę jak mu się podoba, szybciej lub wolniej. I każdą mapę można wrzucić do każdego autoloadera. Dobrze myślę?

Kompatybilność jest akurat bardzo ważna w przypadku autoloadera. Bo jeżeli napiszę jakąś bibliotekę, to wykorzystam swój pomysł. I ta biblioteka musi działać z biblioteką autorstwa kogoś innego, kto też oparł ją o swój autoloader. A autoloader może działać tylko jeden, więc jeżeli jeden do drugiego nie pasuje, to całość wychodzi do dupy.

[bigZbig]

@hawk -> Amen

[dr_bonzo]

...mam i ja.

Stworzylem swojego autoloadera:
- w konfigu podaje katalogi do rekursywnego przejrzenia
- podaje rozszerzenia plikow ktore maja byc tokenizowane

- mapa jest tworzona w std. formacie: nazwa_klasy => sciezka_do_pliku
- i zapisywana w formacie .ini (moze dorobie wybor ini | php)

I co najwazniejsze -- cachuje wyniki tokenizowania plikow (to nie to samo co 'mapa') -- tokenizowane sa tylko te co zostaly zmodyfikowane (okreslane przez filemtime()), co daje znaczna oszczednosc czasu. Zapisuje tablice:

[PHP] pobierz, plaintext 
<?php
 
array (
[ 'nazwa_pliku' ] ->
(
    [ 'classes' ] ->
    (
        [ 0 ] -> 'klasa_1';
        [ 1] - > itd...
    )
    [ 'modification_time' ] = 111123423423423; // czas modyfikacji
    { ['valid' ] = TRUE }
)
)
?>
[PHP] pobierz, plaintext 

Jak to wyglada w praktyce:
- wczytuje ww. cache
- przegladam katalogi i robie liste znajdujacych sie tam plikow
- loop po tych wszystkich plikach: porownuje czas modyfikacji pliku z czasem zapisanym w cache
- jesli plik zostal zmieniony to go tokenizuje i aktualizuje cache (+ ustawienie znacznika 'valid' w cache)
- jesli nie zostal zmodyfikowany to tylko ustawienie znacznika 'valid' w cache
- po przejrzeniu wszystkich plikow, usuwam te wpisy w cache, ktore nie maja znacznika 'valid', i usuwam znacznik 'valid' z tych, ktore go maja (tylko po to zeby go (znacznika) nie zapisac w cache)
- uzyskuje tablice z wpisami
nazwa_pliku --> klasy/interfejsy w min sie znajdujace + czas modyfikacji pliku
- zapisuje to cache
- na podstawie tego cache tworze mape

Wlasciwosci
- nowe pliki ktorych nie bylo w cache sa tokenizowane i dodawane do cache i mapy
- pliki ktore byly w cache a teraz nie istnieja (nie ma ich w podanych w configu katalogach) sa usuwane z cache i mapy
- pliki ktore byly w cache i nie zostaly zmienione nie sa tokenizowane (duuuza oszczednosc czasowa)
- pliki ktore byly w cache i zostaly zmodyfikowane sa ponowanie tokenizowane -> aktualizacja cache

Czasy wykonania:
6 klas / 5 plikow:
0.1137sec // bez cache
0.0041 sec // z cache

dodanie dodatkowych plikow i klas, w sumie
355 plikow / 138 klas
4.3358 sec // bez cache
0.1672 z cache// z cache

modyfikacja 3 z tych plikow (glowne pliki Smartyego -- 161kB) i utworznie mapy, z cache
0.7571 sec




Kodu jest sporo: > 470 linii, jak dokoncze to dam linka.
Pozostaje jeszcze fopisac __autoload()'a.

Ten post edytował dr_bonzo 26.05.2005, 15:16:32

[Vengeance]

Z twojego opisu wynika mi, że za każdym uruchomieniem klasy następuje rekurencyjne przejrzenie katalogów. Jeśli tak jest, to jest to tragiczne rozwiązanie :]

Po co te wszystkie "czasy edycji plikow" itp. Na co ci oszczednosc czasowa przy generowaniu mapy, skoro zalozeniem glownym jest ze robisz to tylko raz! Ewentualnie potem gdy cos zmieniasz w strukturze ponawiasz proces generowania.

Twoje "Jak to wyglada w praktyce" wg mnie powinno ograniczac sie tylko do:
1. przejrzenie rekurencyjne wskazanych katalogów
2. tokenizacja każdego pliku celem wyłapania zawartych w nim klas
3. zapisanie wszystkich powiązań klasa=>plik do .ini czy tam .php

A sam główny skrypt powinien tylko ów .ini lub .php wczytywać, zaś gdy go nie ma powinien uruchamiać generowanie ów pliku :]

Taka moja wersja

[dr_bonzo]

Z twojego opisu wynika mi, że za każdym uruchomieniem klasy następuje rekurencyjne przejrzenie katalogów. Jeśli tak jest, to jest to tragiczne rozwiązanie :]

Jesli mapa istnieje to ten skrypt nie jest uruchamiany.

Na co ci oszczednosc czasowa przy generowaniu mapy, skoro zalozeniem glownym jest ze robisz to tylko raz!

Przy rozbudowie aplikacji i sprawdzaniu poprawnosci jej dzialania nie bedzie mi sie chcialo czekac tych 5 czy 20 sekund na zobaczenie rezultatow.

"W praktyce" to robi + robi to szybciej jesli moze.

[hawk]

Ciekawe rozwiązanie. Faktem jest, że mi by się nie chciało . Zwłaszcza że ja raczej na początku mam rozplanowane klasy, a potem głównie edytuję zawartość nie ruszając struktury.

Co do plików ini... rezultat ten sam. Dla ciekawości, sprawdzałeś, czy są szybsze od zwykłego php? Chociaż wada jest taka, że ja bym w pliku php bardzo chętnie umieszczał mapę wykorzystując jakąś zmienną - główny katalog projektu. Po to, aby ktoś, kto tego używa, nie musiał generować dla siebie mapy tylko dlatego, że u niego pliki php są w innym miejscu na dysku.

[dr_bonzo]

W koncu. Kod przepisany od nowa, lecz jeszcze nie dopieszczony:
http://northslope.lap.pl/dev/Autoloader/0.2-dev/

a tu zrodla do pobrania:
http://northslope.lap.pl/dev/Autoloader/au...-0.2-dev.tar.gz

Kod

generowanie mapy (cache puste):
Array
(
   [files_found] => 468
   [files_parsed] => 468
   [classes_found] => 233
)
Map generation time: :: 12.85658 sec

a z cache (nic nie zmienione):
Array
(
   [files_found] => 468
   [files_parsed] => 0
   [classes_found] => 233
)
Map generation time: :: 0.23078 sec

map from php: :: 0.00076 sec
map from ini: :: 0.00412 sec

php szybsze od INI

PSy: statystki (files_parsed)w udostepninych zrodlach sa nieprawidlowe, te wyzej sa ok.
Testowane na P3 @ 560

Ten post edytował dr_bonzo 5.06.2005, 02:21:56

[Imperior]

dr_bonzo: http://wiki.php.pl/index.php/RecursiveDirectoryIterator
Czyż nie łatwiej?

[hawk]

Fajny kod. Nawet zdążyłem już użyć .

Kilka uwag:
- nie można podać extension np. "class.php", co jest bardzo przydatne, bo pozwala pominąć unit testy ("test.php"). Musiałbyś zmienić metodę isFileExtensionOnTheList tak, aby po prostu sprawdzała, czy nazwa pliku kończy się jednym z podanych wzorców
- przy pierwszym wywołaniu dostaję warning, że cache jeszcze nie istnieje. Potem jest OK.
- RecursiveDirectoryIterator uprościłby sprawę
- przydatną funkcją byłoby parsowanie tagów phpdoc (jeżeli są), aby pomijać klasy "wewnętrzne", czyli takie, które wykorzystywane są tylko w obrębie danego pliku i nie ma potrzeby wystawiać ich na zewnątrz. Ja takich mini-klas używam, i brakuje mi możliwości zrobienia prywatnej klasy wewnętrznej jak w Javie...

[dr_bonzo]

- nie można podać extension np. "class.php", co jest bardzo przydatne, bo pozwala pominąć unit testy ("test.php"). Musiałbyś zmienić metodę isFileExtensionOnTheList tak, aby po prostu sprawdzała, czy nazwa pliku kończy się jednym z podanych wzorców

[DONE] Jest w TODO

- przy pierwszym wywołaniu dostaję warning, że cache jeszcze nie istnieje. Potem jest OK.

Kod jeszcze "niedopieszczony" wersja dev -- dziala ok, pokazuje wszystkie bledy na ekranie, potem dodam malpy do funkcji

- RecursiveDirectoryIterator uprościłby sprawę

[DONE] Zgadza sie. Poprawie

- przydatną funkcją byłoby parsowanie tagów phpdoc (jeżeli są), aby pomijać klasy "wewnętrzne", czyli takie, które wykorzystywane są tylko w obrębie danego pliku i nie ma potrzeby wystawiać ich na zewnątrz.

Troche roboty bedzie. A jaki znacznik odpowiada za plikowy zasieg klasy?

PS. [files_found] to liczba znalezionych plikow z wybranymi rozszerzeniami
-------------------------------------------
EDIT

podglad kodu online
http://northslope.lap.pl/dev/Autoloader/au...-0.3.0-sources/

paczka
http://northslope.lap.pl/dev/Autoloader/au...er-0.3.0.tar.gz

Ten post edytował dr_bonzo 7.06.2005, 11:27:39

[hawk]

A co powiecie na taką implementację?

[PHP] pobierz, plaintext 
<?php
$i = new RecursiveIteratorIterator(new RecursiveDirectoryIterator(PHIEND_HANDLE_DIR));
$i = new KeyFilter($i, '.class.php$');
$w = new PhiendClassMapWriter('./handle_map.php');
$w->setTargetDir('e:/Web Files/lib/phiend2/phiend.handle', 'PHIEND_HANDLE_DIR');
 
$generator = new ClassMapGenerator();
$generator->addIterator($i);
$generator->setCreator(new BasicClassMapCreator());
$generator->addFilter(new SortClassMapFilter());
$generator->setWriter($w);
 
$generator->run();
?>
[PHP] pobierz, plaintext 

Wynik:

[PHP] pobierz, plaintext 
<?php
 
/*
 * File generated by phiend.autoload
 */
 
define('PHIEND_HANDLE_DIR', 'e:/Web Files/lib/phiend2/phiend.handle');
 
$aMap = array(
	'IHandle' => PHIEND_HANDLE_DIR . '/IHandle.class.php',
	'BasicHandle' => PHIEND_HANDLE_DIR . '/BasicHandle.class.php',
// (ciach)
	'SerializedHandle' => PHIEND_HANDLE_DIR . '/SerializedHandle.class.php',
);
 
Autoloader::addMap($aMap);
 
?>
[PHP] pobierz, plaintext 

Mogę tu zrobić praktycznie wszystko, dzięki dobremu rozbiciu na klasy i dziedziczeniu. W pełni wykorzystuje SPL - do tego stopnia, że moje php 5.0.3 jest za stare i musiałem wykomentować kilka linii .
Kod: http://cvs.sourceforge.net/viewcvs.py/phie...hiend.autoload/
Zip: http://phiend.sourceforge.net/misc/phiend-autoload.zip

[cagrET]

hawk: widzę, że używasz "notacji węgierskiej", ciekawi mnie czy może czytałeś ten artykuł: http://www.joelonsoftware.com/articles/Wrong.html > I’m Hungary . Joel pisze tam o tym, że w zły sposób używa się notacji węgierskiej, nie w taki jakim intencją autora tej notacji było.

[hawk]

Nie, nie czytałem do tej pory. Przedrostki określające typ stosuję tylko w php, bo tak jest mi wygodniej. W językach ze ścisłą typizacją są mi niepotrzebne.

[matid]

- przydatną funkcją byłoby parsowanie tagów phpdoc (jeżeli są), aby pomijać klasy "wewnętrzne", czyli takie, które wykorzystywane są tylko w obrębie danego pliku i nie ma potrzeby wystawiać ich na zewnątrz. Ja takich mini-klas używam, i brakuje mi możliwości zrobienia prywatnej klasy wewnętrznej jak w Javie...

Który znacznik phpDoc oznacza klasy wewnętrzne?
Edit: Ale ja jestem ślepy, że nie widziałem wcześniej tego pytania . Nie zmienia to faktu, że nie udzielono na nie odpowiedzi, więc dołączam się do grona pytających

BTW. Bardzo mi się twój kod podoba, szczególnie te wszechobecne interface i rozdrabnianie tego na wiele klas. Chyba trzeba przepisać mój Autoloader

Ten post edytował matid 13.06.2005, 21:10:21

[dr_bonzo]

cagrET: link do swietnego serwisu.

Z jakiegoś powodu większość ludzi zdaje się rodzić bez tej części mózgu, która jest potrzebna do rozumienia wskaźników

, LOL

hawk: do takiego OOP troche mi jeszcze brakuje

notacja wegierska: pamietam jak na poczatku '90 czytalem o tym w pc kurierze -- nawet mam wyrwane strony z tym artem

[hawk]

@dr_bonzo i matid:
Niestety, phpDocumentor nie ma, AFAIK, takiego znacznika. Szkoda. Chociaż nic nie stoi na przeszkodzie taki sobie stworzyć. Precedens jest - w Javie wiele narzędzi posiada swoje własne znaczniki javadoc. Może @no-autoload?

[matid]

@dr_bonzo i matid:
Niestety, phpDocumentor nie ma, AFAIK, takiego znacznika. Szkoda. Chociaż nic nie stoi na przeszkodzie taki sobie stworzyć. Precedens jest - w Javie wiele narzędzi posiada swoje własne znaczniki javadoc. Może @no-autoload?

Osobiście byłbym nie za wprowadzeniem znacznika mówiącego o wykluczeniu klasy z autoładowania (choć może to też by był dobry pomysł), lecz znacznika mówiącego o zasięgu klasy.
Czyli prędzej coś jak:
@range: file
@class-range: file
@classrange: file

Oprócz tego można by się pokusić o bardziej rozbudowany sysem autoloadingu, który np. pozwalałby na uwzględnienie takich czynników jak zakres folderowy (@range: directory), no i oczywiście @range: global, czyli to samo co w przypadku gdy pomijamy parametr zasięgu.

To takie moje 3 grosze.

BTW. Jak widzicie wykorzystanie takiego autoloadera w systemie? Czy korzystacie z niego dla całego kodu, czy tylko dla bibliotek? Bo dość niewygodne może być korzystanie z tego w modułach/akcjach. Bo akcja powinna być unikalna w obrębie modułu, ale może się pojawić w innych.

Ten post edytował matid 14.06.2005, 06:28:12

[hawk]

@matid: Pomysł wręcz genialny w swojej prostocie. Popieram. Jeżeli mamy wybrać najlepszą nazwę, to ja proponuję

Kod

@scope file

Scope, bo to bardziej odpowiednie słowo niż range. Bez dwukropka, bo chyba tagi phpdoc nie używają normalnie dwukropka? W każdym razie ja nigdy nie pisałem...

Co do zastosowania: generalnie używałbym na biblioteki. Albo na intensywnie wykorzystywaną część kodu - zawsze mogę wygenerować sobie mapę np. dla części Modelu. Natomiast dla akcji - nie warto, bo i tak w akcjach musi być porządek, wiadomo dobrze gdzie jaka akcja jest, i wykonujemy jedną akcję na raz. Szkoda mapę robić.

[matid]

No rzeczywiście, @scope będzie lepsze.
Spróbuję to jakoś zaimplementować, zobaczymy, co z tego wyjdzie

Bez dwukropka, bo chyba tagi phpdoc nie używają normalnie dwukropka? W każdym razie ja nigdy nie pisałem...

Literówka Miało być bez.

Ludzie rozwodzicie sie nad mapa, ktora mozna napisac nawet recznie. A wy tworzycie jakies skrypty bog wie po co i tworzycie nowa teorie do tego. Bezsensu. Zamiast napisac prosty skrypt do tworzenia mapy wy tworzycie 50 klas do wygenerowania tablicy 100 klas.

Ja osobiście wole mieć tablice generowane automatycznie. Nie dość że mogę sobie dowolnie zmieniać położenie klas to w dodatku skrypt przy następnym uruchomieniu sam uaktualnia mapę.

A co do podzielenia na 50 klas. Pierwsza wersja mojego skryptu do autoloadingu to była jedna klasa. Ale po przejrzeniu kodu @hawk widzę, ile jeszcze mojemu OOP brakuje. Dopiero podział tej klasy na kilka mniejszych pokazał mi, o ile większe możliwości w późniejszym rozszerzaniu daje OOP.

[hawk]

@serafin:

1) Nikt ci nie broni tworzyć takiej mapy ręcznie. Gratuluję odwagi. I nadmiaru wolnego czasu.

2) Bezsensu. Zamiast napisac prosty skrypt do wyświetlania stron wy tworzycie 50 klas, jakieś wzorce, MVC, biblioteki. Przecież każdy wie, że najlepiej jest wrzucić wszystko do jednego pliku, a całe to OOP to jedna wielka ściema.

Pokaż swój autoloader, a wtedy możemy porozmawiać, co jest lepsze, a co jest bez sensu.

[hawk]

Hawk: Wykorzystales 10 plikow na to co mozna zrobic w jednej klasie (i nie powiedziane ze gorzej).

No dobrze, porozmawiajmy wobec tego konkretnie. Na przykładach. Zaraz zobaczymy, czy rzeczywiście nie gorzej.

1. Chcę włączyć do mapy pliki z katalogów foo/first i foo/second, ale absolutnie nie z foo/third, bo tam są przykłady, które niepotrzebnie zaśmiecałyby mapę.

2. Chcę przeparsować tylko pliki o rozszerzeniu .class.php. Wszystko inne jest niepotrzebne.

3. Chcę, żeby w mojej mapie klasy były posortowane alfabetycznie. Dzięki temu mogę łatwo sprawdzić, jakie klasy są w moim pakiecie, a ponieważ jest to bez różnicy dla przetwarzania, nie ma żadnego powodu, żeby tego nie zrobić. No chyba że narzędzie jest gorsze...

4. Chcę wygenerować mapę dla zewnętrznego pakietu, który inaczej nazywa swoje klasy. Jak narzędzie, to uniwersalne, a nie ograniczone do mojego kodu, prawda? Pakiet jest w zupełnie innym miejscu na dysku.

5. Chcę w wersji produkcyjnej wrzucić kilka klas, które zawsze zawsze są używane, do jednego pliku, żeby przyspieszyć działanie i żeby prościej to wyglądało dla użytkownika końcowego. To częsta praktyka - weźmy np. mojavi.

6. Chcę za pomocą mojego autoloadera wygenerować mapę, która może być od razu wykorzystana w twoim systemie. Czyli plik wygląda tak samo. Chcę mieć uniwersalny autoloader, którego mogą używać inni, a nie nieprzenośne narzędzie.

7. Mam 2 mapy, do 2 różnych pakietów, i chcę obydwie naraz wbić do autoloadera.

Wierzę ci, że można to zrobić w jednej klasie. Która do tego będzie mniejsza niż suma tych plików i w ogóle lepsza. Tylko nie wiem, jak...

A co do tych 10 plików... rozmijasz się z prawdą . To są cztery pliki, plus trzy interfejsy (do niczego niepotrzebne, możesz je spokojnie wywalić, ale ja mam do interfejsów sentyment), plus trzy klasy dziedziczące z klas podstawowych (miałem kaprys zrobić coś extra, ale przecież w ogóle nie musisz ich wykorzystywać). Na podobnej zasadzie możemy zacząć liczyć unit testy i pliki readme.

---------------------------------------------

OOP ma sens owszem, ale nie w przypadku gdy tworzy sie klasy na 1-3 metod

Wiesz, co to jest Observer? Abstract Factory? Iterator? Registry? Intercepting Filter? Chain of Responsibility? Command? Strategy? Takie różne wzorce projektowe, ale niestety bez sensu, bo mają za mało metod.

Oczywiscie ze rozbijanie problemu na mniejsze czescie jest bardzo dobra technika ale sa pewne granice. Czasami mozna kilka z tych czesci polaczyc w jedna wieksza calosc bez szkody dla ostatecznego rozwiazania.

To ty zacząłeś tą dyskusję, więc odpowiedz: dlaczego rozbijanie problemu na mniejsze klasy jest szkodliwe? Bo z tego, co piszesz, wynika, że wg ciebie jest.

[matid]

Ma to sens?

A dlaczego nie? Mi osobiście bardzo się podoba sposób potraktowania typów zmiennych jako klas, a samych zmiennych jako obiektów.
Oprócz php programuję też w Javie i uważam to za świetne rozwiązanie, więc dlaczego by nie używać tego w php?

Teraz pewnie padnie pytanie, co mi da takie "utrudnianie" sobie życia. Ale zdecydowaną korzyścią płynącą z takiego rozwiązania jest możliwość programowania własnych typów danych, czego przy standardowym traktowaniu zmiennych przez php nie zrobisz.

Do notki poniżej: Skoro w php są typy podstawowe, to dlaczego miałoby nie być rozszerzonych? Nie będę nikogo nawracał rzucenia podstawowych typów w php na rzecz klas, bo dla niektórych to może być masochizm. Ja osobiście nie mam nic przeciwko
A co do pisania w php systemu operacyjnego. Wiem, że to było pytanie retoryczne, ale odpowiem - bo php się do tego kompletnie nie nadaje

A teraz wracając do autoloadera. Zaimplementowałem parsowanie tagów phpDoc, ale jeszcze borykam się z jedym problemem.
Powiedzmy, że moja mapa wygląda tak:

[PHP] pobierz, plaintext 
<?php
 
/*
* File generated by OpenForce Autoloader
*/
 
	$aMap = array(
		'GLOBAL' => array(
			'IteratorFilter' => '/var/www/matid/Current/Autoloader/lib/IteratorFilter.class.php',
			'iClassMapGenerator' => '/var/www/matid/Current/Autoloader/lib/autoloader/interfaces/ClassMapGenerator.interface.php',
			'iClassMapEntry' => '/var/www/matid/Current/Autoloader/lib/autoloader/interfaces/ClassMapEntry.interface.php',
			'iClassMapWriter' => '/var/www/matid/Current/Autoloader/lib/autoloader/interfaces/ClassMapWriter.interface.php',
			'iClassMapCreator' => '/var/www/matid/Current/Autoloader/lib/autoloader/interfaces/ClassMapCreator.interface.php',
			'ClassMapGenerator' => '/var/www/matid/Current/Autoloader/lib/autoloader/ClassMapGenerator.class.php',
			'OFClassMapWriter' => '/var/www/matid/Current/Autoloader/lib/autoloader/OFClassMapWriter.class.php',
			'ClassMapEntry' => '/var/www/matid/Current/Autoloader/lib/autoloader/ClassMapEntry.class.php',
			'ClassMapCreator' => '/var/www/matid/Current/Autoloader/lib/autoloader/ClassMapCreator.class.php',
			'ClassMapWriter' => '/var/www/matid/Current/Autoloader/lib/autoloader/ClassMapWriter.class.php',
			'ClassNotFoundException' => '/var/www/matid/Current/Autoloader/lib/exceptions/ClassNotFoundException.class.php',
			'Autoloader' => '/var/www/matid/Current/Autoloader/lib/Autoloader.class.php',
			'Test2' => '/var/www/matid/Current/Autoloader/test/1/2.php',
			'Test3' => '/var/www/matid/Current/Autoloader/test/3.php',
		),
		'/var/www/matid/Current/Autoloader/test/1' => array(
			'Test1' => '/var/www/matid/Current/Autoloader/test/1/1.php',
		),
	);
 
Autoloader::addMap( $aMap );
 
?>
[PHP] pobierz, plaintext 

Klasa Test1 ma ustawiony zasięg directory, reszta jest global. I teraz mam taką sytuację:

1. Jakaś klasa próbuje się odwołać do klasy Test1, ale nie znajduje się w tym samym folderze, autoloader rzuca ClassNotFoundException (a w zasadzie tworzy klasę Test1, której kostruktor rzuca wyjątek).
2. Jakaś inna klasa próbuje się odwołać do klasy Test1 i znajduje się w tym samym folderze, niestety, wcześniej została już utworzona klasa Test1, która domyślnie rzuca ClassNotFoundException.

Ma ktoś jakiś inny pomysł na implementację tego? Chyba jedynym wyjściem jest pozbycie się wyjątków z autoloadera, ale może ktoś wymyśli coś innego.

Ten post edytował matid 16.06.2005, 22:23:05

[bela]

Ma to sens?

A dlaczego nie? Mi osobiście bardzo się podoba sposób potraktowania typów zmiennych jako klas, a samych zmiennych jako obiektów.

To było takie pytanie retoryczne :] Nie ma to sensu. Po co opakowywać typy wbudowane jak i tak nic nie zyskasz, no może poza estetyką kodu

Oprócz php programuję też w Javie i uważam to za świetne rozwiązanie, więc dlaczego by nie używać tego w php?

W Javie też są typy podstawowe i jakoś nikt z tego powodu nie umarł :]
Btw. czemu nie piszesz w php systemu operacyjnego

Teraz pewnie padnie pytanie, co mi da takie "utrudnianie" sobie życia. Ale zdecydowaną korzyścią płynącą z takiego rozwiązania jest możliwość programowania własnych typów danych, czego przy standardowym traktowaniu zmiennych przez php nie zrobisz.

Emm a czy klasy nie są typami danych?

[Imperior]

Ma ktoś jakiś inny pomysł na implementację tego? Chyba jedynym wyjściem jest pozbycie się wyjątków z autoloadera, ale może ktoś wymyśli coś innego.

1. Olać tworzenie pustej klasy
2. Olać zasięgi
3. Każdą klasę tworzyć pośrednio

ad 3. Proxy... zamiast $obj = new Test1(); robić $obj = Class::New('Test1');
A metoda New z Class wszystkim sie zajmie (zwróci instancję, rzuci wyjątek itp)

[matid]

Chyba jednak skorzystam z opcji 1, ponieważ zasięgów nie chcę olewać, a tworzenie instancji przez proxy wyklucza takie przypadki jak dziedziczenie, implementowanie interfaców, itp.