Problème avec une boucle

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:

---

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.

Salut

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

Code:

---

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.

Code:

---

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é ^^

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.

@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:

---

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 }

---

Citation:

---

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 )

Citation:

---

Envoyé par anfoni

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 )

---

Arf, désolé :/

Ah je vois, je vais tenter de mettre ca en place.

Un grand merci a tous pour votre aide !!! :P