Condition qui ne s'exécute pas

Version imprimable

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:

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

alors voila le bout de code et merci de votre aide

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
 $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;
		}
	}

19/06/2010, 01h20
sabotage

As tu fais le minimum de debugage consistant a controler la valeur des deux variables ?
19/06/2010, 02h26
monlou
j'ai fait un var_dump
et ya rien
Code:

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
19/06/2010, 14h20
sabotage

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()

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:

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

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
$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;
	}
}

19/06/2010, 15h02
monlou

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
19/06/2010, 15h09
sabotage

ton $extensions est défini après dans le code donc forcemment ca ne colle pas.
19/06/2010, 15h30
monlou

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

Citation:

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

Eric2a

Code:

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

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

Sabotage

Citation:

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
19/06/2010, 15h57
monlou

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"

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:

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

Peux-tu remplacer les lignes

Code:

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:

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.

19/06/2010, 18h04
monlou

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

Place les lignes ci-dessous

Code:

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:

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

qui correspond au dernier code donné.

En prenant soin de supprimer celles-ci

Code:

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'];

19/06/2010, 19h02
monlou

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

Code:

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

message d'erreur

Citation:

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

Essayons avec un nouveau code...

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
 
<?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
?>

20/06/2010, 03h09
monlou

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:

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