Condition : le else ne s'execute pas

Version imprimable

Bonjour,
j'ai un petit script pour lancer une session j'ai donc fait un if else et le else ne s'exécute pas, bien que les infos soient érronées.

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
 
<?php
include('connect.php');
if(isset($_POST['login'])AND isset($_POST['pass']))
	{
	$login = $_POST['login'];
	$pass = $_POST['pass'];
	if($login != NULL AND $pass != NULL)
		{
		$rep = mysql_query("SELECT * FROM acces_boutique WHERE login_acces_boutique = '$login' AND pass_acces_boutique = '$pass'");
		if (!empty($rep))
			{
			while($tab = mysql_fetch_array($rep))
				{
				$id_acces_boutique = $tab['id_acces_boutique'];
				$gerant_acces_boutique = $tab['gerant_acces_boutique'];
				$login_acces_boutique = $tab['login_acces_boutique'];
				$pass_acces_boutique = $tab['pass_acces_boutique'];
				session_start();
				$_SESSION['id'] = $id_acces_boutique;
				$_SESSION['gerant'] = $gerant_acces_boutique;
				$_SESSION['login'] = $login_acces_boutique;
				$_SESSION['pass'] = $pass_acces_boutique;
				header("location:admin-boutique.php");
				}
			}
		else
			{
			header("location:acces_boutique.html");
			}
		}
	}
mysql_close();
?>

Merci de vos aides!

26/04/2009, 17h36
Séb.
Je n'ai pas tout lu, mais pour commencer :
Code:

1 2 3 $login = $_POST['login']; $pass = $_POST['pass']; if($login != NULL AND $pass != NULL)
$login et $pass ne seront jamais NULL car $_POST['login'] et $_POST['pass'] ne contiendront jamais NULL mais au pire, s'ils sont définis, une chaîne vide.

oui ok séb.,

je ne veux pas parraître agressif mais essaies de lire tout avant de poster sinon il y a aucun interet sauf faire monter tes stats sur le site et essaies de ne poster pas des choses sans rapport avec le sujet car mon else n'intervient pas pour faire une alternative à mes vérifications de contenu de variable mais pour faire une alternative à la vérification du contenu de ma requête.

En deuxième lieu je peux que te conseiller d'essayer les deux scripts suivants et de voir le résultat et la tu verra que m'a manière de vérification par != NULL est belle et bien efficace:

formulaire.html:
Code:

1 2 3 4 5 6 <form method="post" action="testvariable.php"> login : <input type="text" name="login" /><br /> pass : <input type="password" name="pass" /><br /> <input type="submit" /> </form>
testvariable.php
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 <?php if(isset($_POST['login'])AND isset($_POST['pass'])) { $log = $_POST['login']; $pass = $_POST['pass']; if($log != NULL AND $pass !=NULL) { echo 'j\'avais raison'; } else { echo 'j\'avais aussi raison'; } } ?>
test1 : remplis le formulaire
test2 : remplis pas le formulaire

Merci aux membres du forum qui souhaitent vraiment faire avancer certain sujet de bien lire le post et si toute fois des propositions sans certitudes sont faites dans un esprit de reflexion mutuel n'hésitez pas on est la pour ça.
:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D:D

26/04/2009, 21h30
Séb.

Citation:

Envoyé par xess91

En deuxième lieu je peux que te conseiller d'essayer les deux scripts suivants

Au temps pour moi. L'habitude d'utiliser !== et === :\

Citation:

et de voir le résultat et la tu verra que m'a manière de vérification par != NULL est belle est efficace:

Ce n'est ni beau et ni efficace, mais ce n'est que mon avis hein, car : tu testes si une variable vaut NULL alors qu'elle ne le vaudra jamais. En fait ça passe uniquement parce que NULL == '' donne TRUE, merci PHP.

Citation:

Merci aux membres du forum qui souhaitent vraiment faire avancer certain sujet de bien lire le post et si toute fois des propositions sans certitudes sont faites dans un esprit de reflexion mutuel n'hésitez pas on est la pour ça

Je vais m'occuper de mes stats. EOT.
26/04/2009, 21h45
xess91

C'est bien ce que je disais ça n'avance pas le sujet et le problème de ce post.

Ceci étant c'est comme je disais, je ne voulais pas être agressif et c'est sans rancunes car ça a juste pourrit le post !

Si toutes fois des personnes veulent encore poster c'est sans problème, sinon demain le sujet partira à la poubelle.

Aller bonne soirée à séb. et aux autres :D

Bonjour,

je ne suis pas sûr que ceci fonctionne comme tu l'attends :
Code:

1 2 3 4 5 6 7 $rep = mysql_query("SELECT * FROM acces_boutique WHERE login_acces_boutique = '$login' AND pass_acces_boutique = '$pass'"); if (!empty($rep)) { . . }
Dans tous les cas, $rep ne sera pas vide puisqu'il contiendra au moins le résultat de la requete.

j'aurais plutôt fait ceci :
Code:

1 2 3 4 5 6 7 8 $rep = mysql_query("SELECT * FROM acces_boutique WHERE login_acces_boutique = '$login' AND pass_acces_boutique = '$pass'"); $nb = mysql_num_rows($rep); if ($nb != 0) { . . }

27/04/2009, 09h35
xess91

Salut N1bus,

éffectivement ça peut-être ça, car le problème que tu soulèves m'est déjà arrivé en faisant juste if($rep){}forcement je tombais toujours dans le if car de toute façon la requête s'executait quelque soit le résultat retourné.

Je vais tester mais à première vue il semblerai que ça régle mon PB.

Merci N1bus, je reviens tenir au courrant.:D

Citation:
Envoyé par N1bus
Bonjour,

je ne suis pas sûr que ceci fonctionne comme tu l'attends :
Code:

1 2 3 4 5 6 7 $rep = mysql_query("SELECT * FROM acces_boutique WHERE login_acces_boutique = '$login' AND pass_acces_boutique = '$pass'"); if (!empty($rep)) { . . }
Dans tous les cas, $rep ne sera pas vide puisqu'il contiendra au moins le résultat de la requete.

j'aurais plutôt fait ceci :
Code:

1 2 3 4 5 6 7 8 $rep = mysql_query("SELECT * FROM acces_boutique WHERE login_acces_boutique = '$login' AND pass_acces_boutique = '$pass'"); $nb = mysql_num_rows($rep); if ($nb != 0) { . . }
Humm nonn normalement ça donne le même résultat
on pourrais même faire
Code:

1 2 3 4 5 6 7 8 9 10 Code: 1234567 $rep = mysql_query("SELECT * FROM acces_boutique WHERE login_acces_boutique = '$login' AND pass_acces_boutique = '$pass'"); if (!$rep) { . . }
J'avais lu que cette solution était plus performante en terme de vitesse d'exécution mais je n'ai pas vérifier personnellement
Pour revenir au problème il faudrait tout d'abord s'assurer qu'on entre dans les 2 premiers if ...
essaie donc de mettre des echo pour en être sur car je ne vois pas trop le problème

Bonjour,

Cette ligne ne sert à rien. Si tu arrives d'un formulaire, la condition sera toujours vraie, car vide ou rempli, le champs existe.

Code:

if(isset($_POST['login'])AND isset($_POST['pass']))

Celle-ci sera plus efficace :

Code:

if(!empty($_POST['login']) && !empty($_POST['pass']))

Outre la remarque de Seb. plus haut, par la même occasion, cela rend celle-ci inutile :

Code:

if($log != NULL AND $pass !=NULL)

Comme déjà dit plus haut, cette ligne :

Code:

if (!empty($rep))

sera toujours vrai, sauf erreur dans l'exécution de la requête. Si tu fait :

Code:

echo $rep;

tu devrais trouver quelque chose comme "ressource id #xx", même si la ligne recherchée n'existe pas.

Cette boucle est totalement inutile, car compte tenu de ce que tu cherches, la requête ne devrait retourner qu'un résultat :

Code:

while($tab = mysql_fetch_array($rep))

Cette ligne est inutile, car le "close" est automatiquement effectué en fin de script :

Code:

mysql_close();

Quelle est l'utilité de ces variables intermédiaire ?
Code:

1 2 3 4 $id_acces_boutique = $tab['id_acces_boutique']; $gerant_acces_boutique = $tab['gerant_acces_boutique']; $login_acces_boutique = $tab['login_acces_boutique']; $pass_acces_boutique = $tab['pass_acces_boutique'];
Ceci n'est pas conseillé dans une requête :

Code:

SELECT *

Il vaut mieux nommer les colonnes.

Enfin bref, dans un cas comme celui-ci, j'efface tout et je recommence.

Toutes ces remarques n'ont qu'un but purement humanitaire : abrèger tes souffrances.:D

code corrigé et fonctonnel (j'ai pas dis optimal):

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
 
<?php
session_start();
include('connect.php');
if(isset($_POST['login'])AND isset($_POST['pass']))
	{
	$login = $_POST['login'];
	$pass = $_POST['pass'];
	if($login != NULL AND $pass != NULL)
		{
		$rep = mysql_query("SELECT * FROM acces_boutique WHERE login_acces_boutique = '$login' AND pass_acces_boutique = '$pass'");
                          $tab = mysql_fetch_array($rep);
		if ($tab[0] != NULL)
			{
				$_SESSION['id'] = $tab['id_acces_boutique'];
				$_SESSION['gerant'] = $tab['gerant_acces_boutique'];
				$_SESSION['login'] = $tab['login_acces_boutique'];
				$_SESSION['pass'] = $tab['pass_acces_boutique'];
				header("location:admin-boutique.php");
			}
		else
			{
			header("location:acces_boutique.html");
			}
		}
                    else
                         {
                          header("location:acces.html");
                         }
	}
mysql_close();
?>