Class Loader => simple register directory

**ascito** · 27/09/2012, 02h45

je voudrais avoir votre avis, sur le code qui va suivre,
vous pouvez tester PHP 5.2, ca fonctionne, mais j'ai besoin d'avis sur la mise en forme du code. Il y a peu être des choses plus simple, et je ne suis pas au courant.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
 
<?php
class Loader{ 
 
	protected static $_directory;
	private static $_erreurs;
 
	public function register($name,$dir,$ext='php'){
 
		if(isset($this->directory->$name)){self::messageErreur("Name $name already exist in Loader::directory"); return;}
 
 		$this->directory->$name->root      = $dir;
		$this->directory->$name->extention = $ext;
		self::$_directory = $this ;
 		spl_autoload_register(array($this,'__autoload'),true);
 
		return $this->directory->$name;
 	}
 
	public function unregister($name){
 		unset($this->directory->$name); 
		return $this;
 	}
 
	private function __call($class,$null=null)
	{
		try{
 			$erreur    =  true;
 			foreach ($this->directory  as $property ) { 
				$ext       = $property->extention;
				$path      = self::getPath($class);
				$file      = $property->root.$path.'/'.$class.'.'.$ext;
				if(file_exists($file)){
					require_once $file;
					$erreur = false;
				} 
			}
			if($erreur){throw new Exception("File $class not found , in Loader::directory ");}
		}
		catch (Exception $e) {
			self::messageErreur($e); 
		}
 	}
	public static function directory(){
		return self::$_directory;
	}
	public function autoload($path,$options=null){ 
 
		$pathTab=explode('::',$path);
		$class  = $pathTab[0] ;
		$method =isset($pathTab[1]) ? $pathTab[1] : null; 
		$this->__call($class); 
 
		try{
			if(!null == $class && !null == $method && class_exists($class) && method_exists($class,$method)){
				$class = new $class;
				$class->$method($options);
			}else{
				throw new Exception("Class $class or method $method no found , call file path : $path ");
			}		
		}
		catch (Exception $e) {
			self::messageErreur($e); 
		}
 	}
 
	private static function getPath($path){ 
		return str_replace('_','/',$path);
	}
 
	public static function messageErreur($msg = ''){
		self::$_erreurs[] = $msg ;
		if (!defined('SHOW_MESSAGE_ERREUR') || SHOW_MESSAGE_ERREUR == 'SHOW_MESSAGE_ERREUR' || SHOW_MESSAGE_ERREUR === true ){
			try { 
			throw new Exception($msg);
			}
			catch (Exception $e) {
			echo "Message : ",  $e->getMessage(), "\n\r<br/><br/>";
			}
		}
	}
 
}
 
 
/*
  par exemple créer un repertoire dans /library/class/View/View.php


Class View{
	
	public function accueil($string){
		echo $string;
	}
}
*/
 
define('__DIR__' , dirname(__FILE__));
 
$loader = new Loader; 
 
$loader->register('library',__DIR__.'/library/class/'     ,'php');  
$loader->register('public' ,__DIR__.'/public/'      ,'php'); 
$loader->register('route' ,__DIR__.'/'      ,'php');
 
print_r(Loader:: directory());
echo "<br/><br/>";
$loader->register('public',__DIR__.'/library/'     ,'php');
 
$loader->autoload('View::accueil',"c'est un test");
$loader->autoload('View::test',"c'est un test");
 
 
$view = new View; 	
 
 
?>

**rawsrc** · 27/09/2012, 09h43

Salut,

ta fonction autoload() est très étrange !
L'autoloader doit juste gérer le mécanisme te permettant de trouver parmi tes répertoires le fichier content le code de la classe appelée et rien de plus.
En gros, il ne doit se limiter qu'à ne faire au final qu'un simple include ''; et renvoyer true ou false. Chez toi, il instancie la classe et exécute une méthode en lui passant des options !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$class = new $class;
$class->$method($options);

T'es sûr de ton approche parce que là, il me semble que ton Loader fait bien trop de choses.

**ascito** · 28/09/2012, 03h55

Alors, pour le coup je peux te répondre ;

j'ai prévu la function register pour alimenter la liste des chemins, et la fonction autoload pour carément taper dans le pratique, pour gagner du temps, accessible , mais je ne suis pas si sur de mes appellations, ni du découpage...

a relire, il faut que je renomme la fonction autoload;

merci a toi, et au personnes qui me permettrons d'affiner mon script.