Amélioration script Upload image

**micus** · 07/07/2016, 14h52

Bonjour,

J'ai un script qui upload tous type de fichier,
Je souhaiterai le border et autoriser que l'upload de jpg ou png

je penser réutiliser cette fonction :

Code :

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

1
2
$extension_upload = strtolower( substr(strrchr($_FILES['image']['name'], '.') ,1) );
if ( in_array($extension_upload,$extensions_valides) ) echo "Extension correcte";

mais je ne vois pas comment la mettre en évidence dans mon script.
je déclare un array avec les extensions autorisé.
Puis je suppose que je doit mettre une condition qui contrôle l'extension du fichier.

Voici mon script :

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
<html>
<head>ll</head>
<?php
if(isset($_POST['upload_img'])){
	$file_name = $_FILES['image']['name'];
	$file_type = $_FILES['image']['type'];
	$file_size = $_FILES['image']['size'];
	$file_error = $_FILES['image']['error'];
	$file_tmp_name = $_FILES['image']['tmp_name'];
 
	if($file_name){
	 move_uploaded_file($file_tmp_name,"img/$file_name");
	}
}
?>
<body>
<form method="post" action="" enctype="multipart/form-data">
<label>upload</label><br />
<input type="file" name="image"><br /><br />
<input type="hidden" name="MAX_FILE_SIZE" value="1048576" />
<input type="submit" value="Upload Image" name="upload_img">
</form>
 
<?php
$folder = "img";
if(is_dir($folder)){	
		if($handle = opendir($folder)){	
 
			while(($file=readdir($handle)) != false){			
				if($file==='.' || $file==='..'  )continue;
 
				echo '<img src="img/'.$file.'" width="150" height="150" alt="">';
			}
			closedir($handle);			
		}			
}
?>

merci de votre aide

**sabotage** · 07/07/2016, 15h32

Il existe des fonctions pour avoir l'extension :

http://php.net/manual/fr/splfileinfo.getextension.php

**Celira** · 07/07/2016, 15h35

Ou plus basiquement pathinfo avec le flag PATHINFO_EXTENSION

A part ça, tu as tout à fait compris le principe : contrôler et si le contrôle échoue, rejeter l'upload. Quel est ton problème exactement ?

Invité · 07/07/2016, 15h39

Bonjour,

voir cet exemple :

Traitement Photo (configuration)

**micus** · 07/07/2016, 15h41

Actuellement avec le script je peux uploader tous type de fichiers.
Je souhaiterai que pour l'upload il n'y a que les extension jpg et png.

Donc faudrait que j'utilise PATHINFO_EXTENSION

mais je dois la placer dans une condition avant d’exécuter l'upload

**Celira** · 07/07/2016, 16h09

Oui c'est bien ça. Tu as l'air d'avoir bien compris ce qu'il faut faire. Quel est le problème exactement ?

**micus** · 07/07/2016, 16h26

Déja avant tous merci pour votre aide.

j'ai donc regardé comment utiliser pathinfo_extension

j'ai essayé cela
crée une variable extension_valides compenant :

Code :

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

$extensions_valides = array( 'jpg' , 'jpeg' ,'png' );

Puis j'ai crée une variable file_extension

Code :

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

$file_Extension = strtolower(pathinfo($_FILES['image']['name'],PATHINFO_EXTENSION));

J'ai crée un if($a === $b) puis englobé l'a suite de la requete mais cela ne passe pas.

voici le code

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
<?php
$extensions_valides = array( 'jpg' , 'jpeg' , 'gif' , 'png' );
$file_Extension = strtolower(pathinfo($_FILES['image']['name'],PATHINFO_EXTENSION));
 
$folder = "img";
 
if ($file_Extension === $file_Extension){
	if(is_dir($folder)){	
			if($handle = opendir($folder)){	
 
				while(($file=readdir($handle)) != false){			
					if($file==='.' || $file==='..'  )continue;
 
					echo '<img src="img/'.$file.'" width="150" height="150" alt="">';
				}
				closedir($handle);			
			}			
	}
}
?>

j'obtiens l'erreur :Notice: Undefined index: image in C:\wamp64\www\uploadtest\formulaire.php on line 26

**sabotage** · 07/07/2016, 16h37

Code :

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

if ($file_Extension === $file_Extension){

**micus** · 07/07/2016, 17h25

Envoyé par sabotage

Code :

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

if ($file_Extension === $file_Extension){

j'avoue

crackage lol

sinon je voulais mettre

Code :

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

if ($file_Extension === $extensions_valides){

mais toujours une erreur a la ligne 26
soit :

Code :

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

$file_Extension = strtolower(pathinfo($_FILES['image']['name'],PATHINFO_EXTENSION));

**Celira** · 07/07/2016, 17h51

Code :

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

if ($file_Extension === $extensions_valides){

Bof.

Vu que $extensions_valides est un tableau, j'utiliserai plutôt in_array :

Code :

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

if (in_array($file_Extension, $extensions_valides)){

Et sinon, pour ton erreur, ce serait sans doute une bonne idée de ne faire le test que si un fichier a été envoyé

**micus** · 08/07/2016, 08h12

Quand tu veut dire faire le test que si un fichier est envoyé, donc faut que je test la différence soit $a != $b ?
voici le code testé
tant que $extensions_valides est different de $file_extension afficher erreur ....
est ce le bon procedé ?

car il me renvoie toujours une erreur Undefined index sur la ligne

Code :

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

$file_Extension = strtolower(pathinfo($_FILES['image']['name'],PATHINFO_EXTENSION));

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
 
<?php
$extensions_valides = array( 'jpg' , 'jpeg' , 'gif' , 'png' );
$file_Extension = strtolower(pathinfo($_FILES['image']['name'],PATHINFO_EXTENSION));
 
$folder = "img";
 
 
	if(is_dir($folder)){	
			if($handle = opendir($folder)){	
 
				while ($extensions_valides != $file_Extension){
					echo 'le fichier uploader n est pas une image';
 
				while(($file=readdir($handle)) != false){			
					if($file==='.' || $file==='..'  )continue;
 
					echo '<img src="img/'.$file.'" width="150" height="150" alt="">';
				}
				closedir($handle);			
			}			
	}
	}
?>

J'ai également essayé ta suggestion.
Comme tu me la évoquer les valeurs appartient a un table dons j'ai utilisé le in_array
Quand il verifie les erreur si le fichier porte la bonne extension alors on ouvre le dossier et on upload
sinon on affiche une erreur.

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
 
<html>
<head>ll</head>
<?php
if(isset($_POST['upload_img'])){
	$file_name = $_FILES['image']['name'];
	$file_type = $_FILES['image']['type'];
	$file_size = $_FILES['image']['size'];
	$file_error = $_FILES['image']['error'];
	$file_tmp_name = $_FILES['image']['tmp_name'];
 
	if($file_name){
	 move_uploaded_file($file_tmp_name,"img/$file_name");
	}
}
?>
<body>
<form method="post" action="" enctype="multipart/form-data">
<label>upload</label><br />
<input type="file" name="image"><br /><br />
<input type="hidden" name="MAX_FILE_SIZE" value="1048576" />
<input type="submit" value="Upload Image" name="upload_img">
</form>
 
<?php
$extensions_valides = array( 'jpg' , 'jpeg' , 'gif' , 'png' );
$file_Extension = strtolower(pathinfo($_FILES['image']['name'],PATHINFO_EXTENSION));
 
 
 
$folder = "img";
 
 if (in_array($file_Extension, $extensions_valides)){
	if(is_dir($folder)){	
			if($handle = opendir($folder)){	
 
 
				while(($file=readdir($handle)) != false){			
					if($file==='.' || $file==='..'  )continue;
 
					echo '<img src="img/'.$file.'" width="150" height="150" alt="">';
				}
				closedir($handle);			
			}			
	}
 }
 else
	 echo "le fichier n'est pas une photo"; 
?>

Mais toujours l'erreur sur index image ( ma variable image n'est pas définie )

Invité · 08/07/2016, 10h00

bonjour,

1/ c'est bien de

les réponses....
Encore faut-il comprendre les corrections données :

Code php :

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

if (in_array($file_Extension, $extensions_valides)){

Quand on ne comprend pas, on fait une recherche : in_array()

2/ Quant au message d'erreur...
Merci de nous le MONTRER (complet), car il fourni des infos utlises à sa résolution

3/ Et explique-nous POURQUOI le test d'extension ne se fait pas à la récupération des données (dans ton 1er script PHP), AVANT d'uploader le fichier...

Ton code a clairement un problème de LOGIQUE, et d'enchainement des évènements.
(qui explique aussi l'erreur "image" non définie...)

Toi, tu vérifies l'extension APRES avoir uploadé le fichier (et ce, uniquement au moment d'AFFICHER le fichier).
Non : il faut le faire AVANT, lors du traitement (inutile de télécharger des fichiers non-valides !)

N.B. Je t'avais mis un lien, avec exemple de traitement et gestion d'erreur.
As-tu au moins pris le temps de le LIRE ??

Ca peut te permettre de comprendre l'enchainement des actions.

**ABCIWEB** · 08/07/2016, 19h20

Envoyé par jreaux62

bonjour,
1/ c'est bien de

les réponses....
Encore faut-il comprendre les corrections données

C'est moi qui a plussoyé le dernier message de Celira afin qu'il soit bien pris en compte par micus...

En fait cela reprend le premier bout de code qu'il avait mis dans son premier message, mais qu'il n'a pas su exploiter.

Pour chipoter à mon avis tu ne devrais pas parler de contrôler l'extension en php avant d'avoir uploadé le fichier, car il faut bien que le fichier soit uploadé (certes dans le répertoire temporaire) pour pouvoir y accéder en php donc cela peut prêter à confusion.

Je précise donc pour micus :
On fait le contrôle en php après que le fichier soit uploadé dans le répertoire temporaire mais avant qu'il soit transféré dans son emplacement définitif avec move_uploaded_file. Etant donné que les fichiers temporaires sont effacés automatiquement en fin de script, il n'en restera aucune trace si on ne les exploite pas avec move_uploaded_file ou une autre fonction php de transfère ou de copie de fichier.

Si tu veux un script qui puisse afficher la vignette de l'image dès la sélection du fichier et refuser le téléchargement avant l'upload du fichier en cas d'extension non conforme, il faut utiliser javascript couplé à php. Cela permet aussi d'afficher la progression du téléchargement et de multiples possibilités comme le recadrage/crop avant upload, etc. Tu peux regarder ici pour un module complet avec des exemples prêts à l'emploi. L'avantage pour l'utilisateur est qu'il n'aura pas à attendre le téléchargement complet du fichier avant d'avoir les informations.

Invité · 08/07/2016, 19h45

Envoyé par ABCIWEB

...tu ne devrais pas parler de contrôler l'extension en php avant d'avoir uploadé le fichier, car il faut bien que le fichier soit uploadé (certes dans le répertoire temporaire) pour pouvoir y accéder en php donc cela peut prêter à confusion...

C'est très juste.

Il fallait donc comprendre : "AVANT le move_uploaded_file()"

@ABCIWEB : Et je plusse ou plussois (

) ton "module complet"
(que je n'ai toujours pas eu pris le temps de tester... mais que je reprendrais* certainement pour mes prochains développements de sites)
* un bon exemple de collaboration "croisée" entre membres DVP !

**ABCIWEB** · 21/07/2016, 04h27

Envoyé par jreaux62

C'est très juste.

@ABCIWEB : Et je plusse ou plussois (

) ton "module complet"
(que je n'ai toujours pas eu pris le temps de tester... mais que je reprendrais* certainement pour mes prochains développements de sites)
* un bon exemple de collaboration "croisée" entre membres DVP !

Oui et d'ailleurs tu retrouveras certaines de tes fonctions de redimensionnement et crop dans ce module. Je les ai modifiées pour les besoins internes mais j'ai laissé les liens vers ton article dans le code source de ces fichiers

C'est le moment idéal pour s'y intéresser d'autant plus que je viens de poster à l'instant la version 6.0 avec des exemples compatibles pour navigateurs obsolètes >= IE7. Il n'y a donc plus d'inconvénient à l'utiliser en "tout terrain" et les visiteurs ayant des navigateurs modernes profiteront de toutes les fonctionnalités avancées

(j'ai testé le mode compatibilité IE7 depuis IE10. Il est très probable que les exemples soient compatibles également IE6 mais j'ai pas de moyens pour tester)

Amélioration script Upload image

Langage PHP

Discussions similaires

Partager

Partager