Discussion :

[sam01]

Bonjour, je fait un test sur mon url : http://www.test.com/liste_new.php?ca..._cat=batteries

avec le code suivat :

Code :

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

1
2
3
 
 
if (((!empty($_GET['cat'])) and ($_GET['cat'] == 'camescopes')) and ((!empty($_GET['sous_cat'])) and ($_GET['sous_cat'] == 'batteries')))

d'après mon url, le code ne doit s'exécuter puisque je demande cat=camescopes
et sous_cat=batteries

Ce que je ne comprends c'est que le code s'exécute?

Pouvez-vous m'explqiuer pourquoi?

Merci d'avance pour votre aide.

[s.n.a.f.u]

Bonjour,

Dans ton url , j'ai cat = accessoires photo...

[Amara]

Bonjour,

Dans ton url , j'ai cat = accessoires photo...

Justement il compare ça avec camescopes et ça passe quand même

[haltabush]

Tes parenthèses ne sont pas bonnes.
Essaye avec ça :

Code :

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

if (!empty($_GET['cat']) and $_GET['cat'] == 'camescopes' and !empty($_GET['sous_cat']) and $_GET['sous_cat'] == 'batteries')

[mathieugamin]

Ou avec ça :

Code :

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

if ((!empty($_GET['cat'])) and ($_GET['cat'] == 'camescopes') and (!empty($_GET['sous_cat'])) and ($_GET['sous_cat'] == 'batteries'))

En fait tu n'as pas besoin de faire autant de sous-groupe si tu utilises AND.

[Amara]

Il y avait des parenthèses inutiles mais je pense pas que ça influe sur le résultat final donc le problème reste entier

[More]

oui,

je crois que le probleme est des variables recuperes, peut etre il faut afficher les variables pour savoir son contenu pour detecte d'ou viens l'erreur.


________________________

Agir, rectifier, remettre tout a Zero mais ne jamais reste inactif a la recherche du parfait.

[mathieugamin]

Peux-tu leur affecter une valeur directement pour voir si le problème vient du GET?

Code :

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

1
2
3
4
5
6
7
8
<?php	$_GET['cat'] = 'camescopes';
	$_GET['sous_cat'] = 'batteries';
	if (((!empty($_GET['cat'])) and ($_GET['cat'] == 'camescopes')) and ((!empty($_GET['sous_cat'])) and ($_GET['sous_cat'] == 'batteries'))){
		echo 'camescopes';
	}
	else {echo'Accessoires photos';}
	echo'<hr>';
?>

Ce code fonctionne chez moi...

[haltabush]

Il y avait des parenthèses inutiles mais je pense pas que ça influe sur le résultat final donc le problème reste entier

Je maintiens : il y a une parenthèse de trop après la 1ère condition

Code :

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

if (((!empty($_GET['cat'])) and ($_GET['cat'] == 'camescopes')) and ((!empty($_GET['sous_cat'])) and ($_GET['sous_cat'] == 'batteries')))

Edit: tiens, non c'est bon en fait... désolé

[Amara]

Il avait juste regroupé deux par deux les conditions, je compte neuf parenthèses ouvrantes et autant fermantes donc pas de souci syntaxique...


Edit : rôôôôô tu m'as obligé à vérifier pour rien alors

[s.n.a.f.u]

Bon déjà, j'aime bien mettre des varibales intermédiaires qui permettent de debugger plus facilement :

Code :

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

1
2
3
 
$truc = $_GET['machin']
if (empty($truc)) {...}

Et du coup, c'est facile à savoir si tes variables sont alimentées en mettant des echos avant le test...