Condition qui ne s'exécute pas

**monlou** · 18/06/2010, 19h37

Bonjour a vous tous

Pourriez-vous m'aider S.V.P, je travaille depuis un bout a solutionner une condition qui vérifie l'extension dans un formulaire et qui ne se remplie pas si j'utilise if et un parse error, si j'utilise elseif

celle-ci

Code :

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

if(!in_array($extension, $extensions))

alors voila le bout de code et merci de votre aide

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
 $erreur = false;
		$erreur_img = false;
 
        if (isset($_FILES['photo']) && !empty($_FILES['photo'])) {
 
            if ($_FILES['photo']['error'] > 0) {
			switch ($_FILES['photo']['error']) {
                    case 1 : $msgErreur[] = 'Le fichier téléchargé excède la taille de 2 mo.';
                    break;
                    case 2 : $msgErreur[] = 'Le fichier téléchargé excède la taille MAXFILE SIZE de 200 ko';
                    break;
                    case 3 : $msgErreur[] = 'Le fichier a été que partiellement téléchargé.';
                    break;
                    case 4 : $msgErreur[] = 'Aucun fichier a été téléchargé';
                    break;
                    case 5 : $msgErreur[] = 'erreur';
					break;
					case 6 : $msgErreur[] = 'Un dossier temporaire est manquant';
					break;
					case 7 : $msgErreur[] = 'Échec de l\'ecriture du fichier sur le disque';
					break;
					case 8 : $msgErreur[] = 'Une extension PHP a arrété l\'envoi de fichier';
					break;
						$erreur_img = true;
				}
				if(!in_array($extension, $extensions)) {  //Si l'extension n'est pas dans le tableau
 
 
					$msgErreur[] = 'Vous devez uploader un fichier de type png, gif, jpg, jpeg, ou bmp...';
					$erreur_img = true;
						//echo 'Vous devez uploader un fichier de type png, gif, jpg, jpeg, ou bmp...';
 
	}
					//$erreur_img = true;
 
		}
			else {
 
            $msgErreur[] = 'Veuillez rechercher une photo';
        }
 
		if ($erreur_img == false) {
 
			$fichier = basename($_FILES['photo']['name']);
			$taille_maxi = 200000;
			$taille = filesize($_FILES['photo']['tmp_name']);
			$extensions = array('image/png', 'image/gif', 'image/jpg', 'image/jpeg', 'image/bmp');
			//$extension = strrchr($_FILES['photo']['name'], '.');
			$extension = $_FILES['photo']['type'];
	        $fichier = strtr($fichier, '
                    ÀÁÂÃÄÅÇÈÉÊËÌÍÎÏÒÓÔÕÖÙÚÛÜÝàáâãäåçèéêëìíîïðòóôõöùúûüýÿ',
                    'AAAAAACEEEEIIIIOOOOOUUUUYaaaaaaceeeeiiiioooooouuuuyy');
	        $fichier = preg_replace('/([^.a-z0-9]+)/i', '-', $fichier);
	        $dossier = '../images/';
	        if (move_uploaded_file($_FILES['photo']['tmp_name'], $dossier . $fichier) == false) {
                    $erreur_img = true;
	        }
 
            }
 
        else {
            // Aucune image de recherchée
            $erreur_img = true;
		}
	}

**sabotage** · 19/06/2010, 00h20

As tu fais le minimum de debugage consistant a controler la valeur des deux variables ?

**monlou** · 19/06/2010, 01h26

j'ai fait un var_dump
et ya rien

Code :

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

1
2
3
if(!in_array($extension, $extensions)) {  //Si l'extension n'est pas dans le tableau
 
				var_dump($extensions);

En lisant dans la doc, je crois comprendre que vu que on est dans une intruction switch, le if provoque une perte de comparaison mais je sais pas comment remplacer le= if ici, peut -être le = for mais ma syntaxe n'est pas bonne

**sabotage** · 19/06/2010, 13h20

ton indentation est nulle mais pour moi ton if{} n'est pas dans le switch.

le var_dump() doit par contre être fait avant ton if() puisque ton problème est justement au niveau du if()

**Eric2a** · 19/06/2010, 13h52

Salut,

Je plussoie pour l'indentation...

Bon, j'ai essayé d'améliorer la lisibilité...

Sous quelle condition voulais-tu qu'apparaisse la ligne ?

Code :

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

$msgErreur[] = 'Veuillez rechercher une photo';

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
$erreur = false;
$erreur_img = false;
 
if (isset($_FILES['photo']) && !empty($_FILES['photo'])) {
	if ($_FILES['photo']['error'] > 0) {
		switch ($_FILES['photo']['error']) {
			case 1 : $msgErreur[] = 'Le fichier téléchargé excède la taille de 2 mo.';
			break;
			case 2 : $msgErreur[] = 'Le fichier téléchargé excède la taille MAXFILE SIZE de 200 ko';
			break;
			case 3 : $msgErreur[] = 'Le fichier a été que partiellement téléchargé.';
			break;
			case 4 : $msgErreur[] = 'Aucun fichier a été téléchargé';
			break;
			case 5 : $msgErreur[] = 'erreur';
			break;
			case 6 : $msgErreur[] = 'Un dossier temporaire est manquant';
			break;
			case 7 : $msgErreur[] = 'Échec de l\'ecriture du fichier sur le disque';
			break;
			case 8 : $msgErreur[] = 'Une extension PHP a arrété l\'envoi de fichier';
			break;
		}
		$erreur_img = true;
	} else {
		if(!in_array($extension, $extensions)) {  //Si l'extension n'est pas dans le tableau
			$msgErreur[] = 'Vous devez uploader un fichier de type png, gif, jpg, jpeg, ou bmp...';
			$erreur_img = true;
		}
	}
 
	if ($erreur_img == false) {
		$fichier = basename($_FILES['photo']['name']);
		$taille_maxi = 200000;
		$taille = filesize($_FILES['photo']['tmp_name']);
		$extensions = array('image/png', 'image/gif', 'image/jpg', 'image/jpeg', 'image/bmp');
		//$extension = strrchr($_FILES['photo']['name'], '.');
		$extension = $_FILES['photo']['type'];
		$fichier = strtr($fichier, '
				ÀÁÂÃÄÅÇÈÉÊËÌÍÎÏÒÓÔÕÖÙÚÛÜÝàáâãäåçèéêëìíîïðòóôõöùúûüýÿ',
				'AAAAAACEEEEIIIIOOOOOUUUUYaaaaaaceeeeiiiioooooouuuuyy');
		$fichier = preg_replace('/([^.a-z0-9]+)/i', '-', $fichier);
		$dossier = '../images/';
		if (move_uploaded_file($_FILES['photo']['tmp_name'], $dossier . $fichier) == false) {
			$erreur_img = true;
		}
	}else {
		// Aucune image de recherchée
		$erreur_img = true;
	}
}

**monlou** · 19/06/2010, 14h02

ok, var_dump ($extensions); donne ceci

Notice: Undefined variable: extensions in C:\wamp\www\weziwezo\voiturier\update_voiturier2.php on line 17
NULL

echo $erreur_img; ne renvoie rien

et echo $action; renvoie: UPDATE_Voiturier

**sabotage** · 19/06/2010, 14h09

ton $extensions est défini après dans le code donc forcemment ca ne colle pas.

**monlou** · 19/06/2010, 14h30

Excusez pour l'indentation... dans mon code il est correct, la prochaine fois, je le reviserai après l'avoir collé ici

Sous quelle condition voulais-tu qu'apparaisse la ligne ?

Eric2a

Code :

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

$msgErreur[] = 'Veuillez rechercher une photo';

en effet, tu as raison, il est de trop je pense, je vais l'enlever

Sabotage

ton $extensions est défini après dans le code donc forcemment ca ne colle pas.

ok, maintenant var_dump($extensions); ca donne

array(5) { [0]=> string(9) "image/png" [1]=> string(9) "image/gif" [2]=> string(9) "image/jpg" [3]=> string(10) "image/jpeg" [4]=> string(9)
"image/bmp" }
ca va mieux maintenant mais il reste encore

si je fais echo $erreur_img ya rien et l,erreur n'est pas traité sauf les case error

**monlou** · 19/06/2010, 14h57

je ne sais pas si quelqu'un peut expliquer ceci

apres avoir essayer cette fois
var_dump ($extension)
ca donne ceci apres avoir inserer une image qui n'est pas déclarée dans la variable $extensions
string(24) "application/octet-stream"

**monlou** · 19/06/2010, 15h32

alors voici ce que j'ai trouvé et cerné ou var_dump ne donne plus sa valeur

c'est entre cette partie du code ci-bas et c'est justement la aussi ou est demandé la comparaison des extensions

partout ailleurs dans le script var_dump donne sa valeur

bizarre!!
alors je me demande comment réecrire cette aprtie du 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
38
if ($_FILES['photo']['error'] > 0) {   // de ici
 
			switch ($_FILES['photo']['error']) {
                    case 1 : $msgErreur[] = 'Le fichier téléchargé excède la taille de 2 mo.';
                    break;
                    case 2 : $msgErreur[] = 'Le fichier téléchargé excède la taille MAXFILE SIZE de 200 ko';
                    break;
                    case 3 : $msgErreur[] = 'Le fichier a été que partiellement téléchargé.';
                    break;
                    case 4 : $msgErreur[] = 'Aucun fichier a été téléchargé';
                    break;
                 // case 5 : $msgErreur[] = 'erreur';
			  //    break;
			  	    case 6 : $msgErreur[] = 'Un dossier temporaire est manquant';
			   	    break;
			  	    case 7 : $msgErreur[] = 'Échec de l\'ecriture du fichier sur le disque';
			  	    break;
			  	    case 8 : $msgErreur[] = 'Une extension PHP a arrété l\'envoi de fichier';
 
						$erreur_img = true;
					break;	
				}
				//var_dump($extension);
				//var_dump($extensions);
				if(!in_array($extension, $extensions)) {  //Si l'extension n'est pas dans le tableau
 
 
					$msgErreur[] = 'Vous devez uploader un fichier de type png, gif, jpg, jpeg, ou bmp...';
					$erreur_img = true;
 
				}
 
					//echo 'Vous devez uploader un fichier de type png, gif, jpg, jpeg, ou bmp...';
 
 
				//	$erreur_img = true;
 
		}  // a ici

**Eric2a** · 19/06/2010, 15h41

Peux-tu remplacer les lignes

Code :

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

1
2
3
4
if(!in_array($extension, $extensions)) {  //Si l'extension n'est pas dans le tableau
	$msgErreur[] = 'Vous devez uploader un fichier de type png, gif, jpg, jpeg, ou bmp...';
	$erreur_img = true;
}

par ces lignes

Code :

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

1
2
3
4
5
6
7
8
9
$erreur_img=(!in_array($extension, $extensions));
if(!$erreur_img) { // Si l'extension est admise
	if(($nfo=@getimagesize($_FILES['photo']['tmp_name']))!==false){ // On teste le type de l'image
		$erreur_img=(!in_array((int)$nfo[2],array(IMAGETYPE_GIF,IMAGETYPE_JPEG,IMAGETYPE_PNG,IMAGETYPE_BMP),true));
	}
}
if(!$erreur_img){
	$msgErreur[] = 'Vous devez uploader un fichier de type png, gif, jpg, jpeg, ou bmp...';
}

Ceci permet au script de rejeter les fichiers image non reconnus.

**monlou** · 19/06/2010, 17h04

merci Eric2a
je suis content de ta suggestion

j'ai essayé ce que tu m'as dit mais ca donne le même résultat étant donné que comme j'ai dit dans mon post précédent, la valeur de la variable $extension n'est pas reconnu a l'intérieur de la portion du script que j'ai inclu plus haut

partout en dehors de ce script la valeur est reconnue mais pas a l'intérieur de celle-ci

**Eric2a** · 19/06/2010, 17h31

Place les lignes ci-dessous

Code :

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

1
2
3
4
5
6
 
$fichier = basename($_FILES['photo']['name']);
$taille_maxi = 200000;
$taille = filesize($_FILES['photo']['tmp_name']);
$extensions = array('.gif', '.jpeg', '.jpg', '.png', '.bmp');
$extension = strrchr($_FILES['photo']['name'], '.');

juste avant celle ci

Code :

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

$erreur_img=(!in_array($extension, $extensions));

qui correspond au dernier code donné.

En prenant soin de supprimer celles-ci

Code :

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

1
2
3
4
5
6
7
 
$fichier = basename($_FILES['photo']['name']);
		$taille_maxi = 200000;
		$taille = filesize($_FILES['photo']['tmp_name']);
		$extensions = array('image/png', 'image/gif', 'image/jpg', 'image/jpeg', 'image/bmp');
		//$extension = strrchr($_FILES['photo']['name'], '.');
		$extension = $_FILES['photo']['type'];

**monlou** · 19/06/2010, 18h02

ca plante ici, ca ne reconnait plus le $_POST je crois

Code :

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

if (isset($_FILES['photo']) && !empty($_FILES['photo']))

message d'erreur

Parse error: parse error, expecting `T_VARIABLE' or `'$'' in C:\wamp\www\weziwezo\voiturier\update_voiturier2.php on line 22

**Eric2a** · 19/06/2010, 18h58

Essayons avec un nouveau 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
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
 
<?php
$extensions=array('.gif','.jpg','.jpeg','.png','.bmp');
 
$message='';
if(isset($_FILES['photo']) && !empty($_FILES['photo'])){
	if($_FILES['photo']['error']==UPLOAD_ERR_OK){
		$file=basename($_FILES['photo']['name']);
		$ext=strtolower(substr($file,strrpos($file,'.')));
		$f=(in_array($ext,$extensions));
		if(($f)and(($nfo=@getimagesize($_FILES['photo']['tmp_name']))!==false))
			$f=(in_array((int)$nfo[2],array(IMAGETYPE_GIF,IMAGETYPE_JPEG,IMAGETYPE_PNG,IMAGETYPE_BMP),true));
		if($f){
			$path=dirname(__FILE__).DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'images'.DIRECTORY_SEPARATOR;
			$file=strtr($file,
				  'ÀÁÂÃÄÅÇÈÉÊËÌÍÎÏÒÓÔÕÖÙÚÛÜÝàáâãäåçèéêëìíîïðòóôõöùúûüýÿ',
				  'AAAAAACEEEEIIIIOOOOOUUUUYaaaaaaceeeeiiiioooooouuuuyy');
			$file=preg_replace('/([^.a-z0-9-]+)/i','-',$file);
			if(!@move_uploaded_file($_FILES['photo']['tmp_name'],$path.$file))
				$message='Erreur lors du transfert du fichier';
		}else
			$message='Vous devez uploader un fichier de type png, gif, jpg, jpeg, ou bmp...';
	}else{
		switch($_FILES['photo']['error']){
			case 1:$message='Le fichier téléchargé excède la taille de 2 mo.';
				break;
			case 2:$message='Le fichier téléchargé excède la taille MAXFILE SIZE de 200 ko';
				break;
			case 3:$message='Le fichier a été que partiellement téléchargé.';
				break;
			case 4:$message='Aucun fichier a été téléchargé';
				break;
			case 6:$message='Un dossier temporaire est manquant';
				break;
			case 7:$message='Échec de l\'ecriture du fichier sur le disque';
				break;
			case 8:$message='Une extension PHP a arrété l\'envoi de fichier';
				break;
			default:
				$message='erreur';
		}
	}	
}else
	$message='Veuillez uploader une photo';
 
if($message!=='')echo $message; // Erreur
?>

**monlou** · 20/06/2010, 02h09

merci Eric 2a
mon sujet est enfin résolu

merci de ton support et de ton temps que j'apprécie beaucoup
j'ai appris beaucoup de choses dans cette affaire

y avait 2 erreurs finalement,
déclarer les variables $ extensions et $extension au tout début avec les autres

et

mettre la condition extension au bon endroit soit apres l'accolade fermante de l'instruction

Code :

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

if ($_FILES['photo']['error'] > 0) {

Condition qui ne s'exécute pas

Langage PHP

Discussions similaires

Partager

Partager