[POO] Première classe [Résolu]

[SirDarken]

Voila bonjour à tous j'ai décider de me mettre à la réalisation de classe (aprés tout on doit crée le besoin de l'utiliser:p)
Donc voila j'ai la un projet ou je doit souvent faire de l'upload, donc je me suis dit qu'une classe faisant cela serai le parfait exemple pour débuter.

Donc voila j'aimerai quelque conseil sur ce que j'ai fait, ce que je devrai éviter ou pas.
/!\ : j'ai pas encore tester la classe mais ca devrait marcher.

La voici:

Code :

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
 
<?php
/*
Utilisation.
monupload = new Upload();
monupload->set_fichier($_FILES['x']['tmp_name'],$_FILES['x']['name']
monupload->set_repertoire("../res/upload");
monupload->set_extensions(".jpg,.jpeg,.gif,.png");
if(monupload->verif_extension()){}
if(monupload->move_fichier()){}
$name = monupload->get_fichier();
*/
class upload{
	private $fichier;//vaut $_FILES['']['tmp_name']
	private $nomfichier//vaut $_FILES['']['name']
	private $repertoire;
	private $extensions;
	private $monextension;
	private $monnom;
	private $monfichier;
	public __construct(){
		$this->extensions=array();
		$this->repertoire="";
		$this->monnom="";
		$this->fichier="";
		$this->monnom="";
		$this->monfichier="";
	}
	//Défini le fichier.
	public function set_fichier($fichier,$nom){
		if($fichier != "" && !empty($fichier)){
			$this->fichier = $fichier;
		}
		if($nom != "" && !empty($nom)){
			$this->nomfichier = $nom;
		}
	}
	//Défini le repertoire de déplacement.
	public function set_repertoire($repertoire){
		if($repertoire != "" && !empty($repertoire)){
			$this->repertoire = $repertoire;
		}
	}
	//Défini les extensions authorisé.
	public function set_extensions($extension){
		if($extension != ""){
			$this->extensions = explode(",",$extension);
		}
	}
	//Vérifie l'extension.
	public function verif_extension(){
		$extension = strrchr($this->nomfichier, '.');
		$this->monextension = $extension;
		if(in_array($extension, $extensions)){
			return true;
		}
		else{
			return false;
		}
	}
	//Déplace le fichier avec nom prédéfini sur la date du jour.
	public function move_fichier(){
		$jour = getdate();
		$this->monnom = $jour['year'].$jour['mday'].$jour['hours'].$jour['minutes'].$jour['seconds'];
		if(move_uploaded_file($fichier,$repertoire.'/'.$this->monnom.$this->monextension)){
			$this->monfichier = $repertoire.'/'.$this->monnom.$this->monextension;
			return true;
		}
		else{
			return false;
		}
		return false;
	}
	//Déplace le fichier avec un nom donné.
	public function move_fichier($setnom){
		if(move_uploaded_file($fichier,$repertoire.'/'.$setnom.$this->monextension)){
			$this->monfichier = $repertoire.'/'.$setnom.$this->monextension;
			return true;
		}
		else{
			return false;
		}
		return false;
	}
	//Retourne le chemin du fichier.
	public function get_fichier(){
		return $this->monfichier;
	}
	public __destruct(){
	}
}
?>

Vos critiques ?

[kaymak]

Salut,

si je n'en avais pas déjà une, je m'en servirait volontier, c'est suffisament simple et compréhensible pour l'utiliser.

Cependant, j'aurai encapsuler l'usage de $_FILES, tous les uploads passe par là, cela ne sert à rien de laisser trainer ces appels un peu partout dans le code.

Après je suis simplement sceptique sur la manière doit tu renvois le résultat, c'est très lié à ta vision de la chose.
C a d que tu déplaces le fichier dans un sous répertoire nommés avec la date ect.

Sinon j'aime beaucoup la vérificaiton sur l'extension, minimaliste, mais utile. Et aussi les commentaires.

A titre de comparaison, voici celle que j'utilise

Code :

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
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
 
<?
 
$toto = MyUploadedFile::construireIndex( "monFichier", NULL );
if( $toto != NULL ) // Au cas ou l'index appelé n'existe pas.
{
    if( $toto->estEnErreur() )
        echo $toto->message_Erreur();
    else
        file_put_contents("somewhere.txt",$toto->lireContenu());
}
 
class MyUploadedFile
{
	public $name;
	public $type;
	public $size;
	public $tmp_name;
	public $error;
 
	protected function __construct()
	{
		$this->name 		= "";
		$this->type 		= "";
		$this->size 		= "";
		$this->tmp_name 	= "";
		$this->error 		= "";
	}
 
	public function estEnErreur()
	{
		$retour = false;
		switch ($this->error)
		{
			case UPLOAD_ERR_OK:
				$retour = false;
			break;
			case UPLOAD_ERR_INI_SIZE:
			case UPLOAD_ERR_CANT_WRITE:
			case UPLOAD_ERR_FORM_SIZE:
			case UPLOAD_ERR_PARTIAL:
			case UPLOAD_ERR_NO_FILE:
			case UPLOAD_ERR_NO_TMP_DIR:
				$retour = true;
			break;
			default:
				$retour = true;
			break;
		}
	}
 
	public function message_Erreur()
	{
		$retour = "";
		switch ($this->error)
		{
			case UPLOAD_ERR_OK:
				$retour = "";
			break;
			case UPLOAD_ERR_INI_SIZE:
				$retour = "La taille du fichier ".($this->name)." excede la taille autorisé de ".ini_get("upload_max_filesize")."-." ;
			break;
			case UPLOAD_ERR_FORM_SIZE:
				$retour = "La taille du fichier ".($this->name)." excede la taille autorisé de ".ini_get("upload_max_filesize")."." ;
			break;
			case UPLOAD_ERR_PARTIAL:
				$retour = "Le fichier ".($this->name)." n'a pas était écrit avec succes." ;
			break;
			case UPLOAD_ERR_NO_FILE:
				$retour = "Le fichier ".($this->name)." n'a pas était téléchargé sur le serveur pour une raison inconnue." ;
			break;
			case UPLOAD_ERR_NO_TMP_DIR:
				$retour = "Le dossier temporaire n'existe pas." ;
			break;
			case UPLOAD_ERR_CANT_WRITE:
				$retour = "Le fichier ".($this->name)." n'a pu etre écrit sur le disque." ;
			break;
			default:
				$retour = "Une erreur inconnue a empécher le téléchargement du fichier ".($this->name)." sur le serveur." ;
			break;
		}
 
		return $retour;
	}
 
	public function lireContenu()
	{
		return file_get_contents($this->tmp_name);
	}
 
	public function ouvrirUnPointeurSurLeContenu()
	{
		return fopen($this->tmp_name,"r");
	}
 
	public static function construireIndex( $nom, $index = NULL )
	{
		$retour = NULL;
 
		if( $index === NULL )
		{
			if( isset($_FILES[$nom]["name"]) )
			{
				if( is_uploaded_file($_FILES[$nom]['tmp_name']) && $_FILES[$nom]["name"] != '' )
				{
					$retour 			= new MyUploadedFile();
					$retour->name 		= $_FILES[$nom]["name"];
					$retour->type 		= $_FILES[$nom]["type"];
					$retour->size 		= $_FILES[$nom]["size"];
					$retour->tmp_name 	= $_FILES[$nom]["tmp_name"];
					$retour->error 		= $_FILES[$nom]["error"];
				}
			}
		}
		else
		{
			if( isset($_FILES[$nom]["name"][$index]) )
			{
				if( is_uploaded_file($_FILES[$nom]['tmp_name'][$index]) && $_FILES[$nom]["name"][$index] != '' )
				{
					$retour 			= new MyUploadedFile();
					$retour->name 		= $_FILES[$nom]["name"][$index];
					$retour->type 		= $_FILES[$nom]["type"][$index];
					$retour->size 		= $_FILES[$nom]["size"][$index];
					$retour->tmp_name 	= $_FILES[$nom]["tmp_name"][$index];
					$retour->error 		= $_FILES[$nom]["error"][$index];
				}
			}
		}
 
		return $retour;
	}
}
?>

[Wharenn]

Salut !

J'ai récemment fait une classe dans le genre pour simplifier l'upload d'images sur mon site et ca c'est vraiment révélé être un gain de temps et de clarté énorme !

Ca ressemblait pas mal à la tienne, cependant le rôle de ta méthode verif_extensions pourrait être plus optimisée si par exemple tu l'appelais obligatoirement dans ta méthode move_fichier. Car imaginons que tu oublies de l'appeler dans un bout de code, tu vérifiera pas les extensions et tout pourra passer.

[SirDarken]

Merci de vos remarques, ce qui me rassure c'est surtout que je me suis pas complétement ramasser dans sa création
Faut je prenne un peu de bouteille à présent, notament sur le fait d'encapusler $_Files, et de d'appeler moins de méthodes (dans le sens le faire dans la classe elle même).