Problème avec une boucle

**yayacameleon** · 09/03/2006, 09h22

Bonjour,

Voila mon souci :

J'ai fait une page d'identification (user,password) gerer par un base de donnée.

Jusque la tout va bien, le problème que j'ai c'est que si l'utilisateur se trompe dans le user ou le password le message d' erreur s'affiche autant de fois qu'il y a d'enregistrements dans ma base. (pb dans ma boucle).

Mais je n'arrive pas a résoudre le problème.

Voici le code de ma boucle:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
while ($donnees = mysql_fetch_array($reponse))
           {
              if (($donnees['nom'] == $_POST['user']) && ($donnees['password'] == $_POST['pass']))
              {
              renvoi("validation.php3");
              }
              else
              {
              echo "<table  align=\"center\"><tr><td>";
              echo "<br><br><br><br><br><br><br><br><br>";
              affichage_alerte ("5", "red", "Nom d&rsquo;utilisateur ou mot de passe incorrect");
              echo "</td></tr></table>";
              }
           }

Si quelqu'un peut m'aider un peu.

D'avance merci.

**Yogui** · 09/03/2006, 09h27

Salut

Je pense que le problème se situe en amont : il faut mettre une condition WHERE dans ta requête, par exemple :

Code :

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

1
2
3
4
SELECT *
FROM user
WHERE username = "$username"
   AND password = MD5("$password");

Ainsi, il te suffit d'exécuter la requête et de voir si mysql_fetch_assoc() te retourne un tuple : si oui, c'est ok ; sinon, il y a eu une erreur.

Le problème est que tu fais une boucle, ironiquement.

Invité · 09/03/2006, 09h35

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
 
$user = strip_tags(substr($_POST['user'],0,MAX_USER_TAILLE)); 
$pass = strip_tags(substr($_POST['pass'],0,MAX_PASS_TAILLE)); 
while ( $donnees = mysql_fetch_array($reponse) )
{
	if ($donnees['nom'] == $user) 
    { 
		if($donnees['password'] == $pass)
		{
			renvoi("validation.php3"); 
		}
		else 
		{ 
			echo "<table  align=\"center\"><tr><td>"; 
			echo "<br><br><br><br><br><br><br><br><br>"; 
			affichage_alerte ("5", "red", "Nom d&rsquo;utilisateur ou mot de passe incorrect"); 
			echo "</td></tr></table>"; 
			break; //Arretez la boucle while
		} 
	}    
}

/me se remet au PHP ^^

[EDIT après le message de Kirkis ]

Ta méthode est meilleure, et je me suis cassé la tête pour rien en plus moi, mais je laisse quand même mon code modifié ^^

**yayacameleon** · 09/03/2006, 09h38

Merci pour ta reponse, mais je suis "newbie" en php donc effectivement c'est peut être un peu maladroit.

Je ne comprend pas trop ta dernière phrase:

Ainsi, il te suffit d'exécuter la requête et de voir si mysql_fetch_assoc() te retourne un tuple : si oui, c'est ok ; sinon, il y a eu une erreur.

merci.

**Yogui** · 09/03/2006, 09h38

@anfoni : Non ! Il n'y a pas besoin de faire une boucle ! MySQL peut ne retourner qu'une ligne correspondant aux infos, il faut l'utiliser sous peine de faire des tonnes de calculs inutiles !

@yayacameleon : Voici un exemple :

Code :

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

1
2
3
4
5
6
7
8
$sql = '...'; // cf. ci-dessus
$result = mysql_unbuffered_query($sql) or die(mysql_error());
if(mysql_fetch_assoc($result)){
   // user logged in
}
else{
   // erreur de login ou mot de passe
}

Invité · 09/03/2006, 09h40

Envoyé par Kirkis

@anfoni : Non ! Il n'y a pas besoin de faire une boucle ! MySQL peut ne retourner qu'une ligne correspondant aux infos, il faut l'utiliser sous peine de faire des tonnes de calculs inutiles !

Maisheu, j'ai dit que tu avais raison dans mon édition de message ! (ou alors tu as posté ce message alors que j'éditais le mien )