[aide]espace membre

[TRasHED]

Salut, je voudrais vous poser une question sur le code ci joint:

Code PHP :

<?php

session_start();[/color]

if ((empty($_POST['login'])) AND (empty($_POST['passe']))) /* on controle que les champs ne sont pas vide ou nul et si c'est le cas on affiche un message d'erreur */

{ echo 'L'un des champ de connexion n'est pas remplie, merci de refaire un essai: <a href="vers_formulaire">ICI</a>'; }

else { /* les champs du formulaire de connexion ne sont pas vide on peut affecter les valeurs du post au

variable pour les comparer à la table des utilisateur */

$login=mysql_real_escape_string($_POST['login']); /* ici on fait la protection des variable pour éviter les injection mysql on peut éventuellement rajouter un stripslashes() aux Post avant */

$passe=mysql_real_escape_string($_POST['passe']);

$reponse = mysql_query("SELECT login, passe FROM membre WHERE login=$login AND passe=$passe") or die(mysql_error()); /* on selectionne le champ login et passe du membre pour controle */

$donnees = mysql_fetch_array($reponse);

if ($donnees['login']!="" AND $donnees['passe']!="") /* on vérifie le retour mysql sur login et passe */

{

$_SESSION['login_session']=$donnees['login']; /* l'utilisateur est reconnu on peut mettre son login en variable de session mais pas le mot de passe */

}

else

{ echo 'erreur de connexion'; } } /* l'utilisateur n'est pas reconnu on lui affiche un message d'erreur ou plus */

?>

[Imhotep2]

Bonjour,

mais pourquoi parcourir toute la table , et ne pas faire simplement :

Code :

$reponse = mysql_query("SELECT  passe FROM membre WHERE login=$login ) or die(mysql_error()); /
$passe2=mysql_result($reponse,"passe")

if ($passe2==$passe)
{
$_SESSION['login_session']='$login'; /* l'utilisateur est reconnu on peut mettre son login en variable de session mais pas le mot de passe */
}
else
{ echo 'erreur de connexion'; }  /* l'utilisateur n'est pas reconnu on lui affiche un message d'erreur ou plus */
?>

Ca n'est pas mieux ?

[TRasHED]

sans couleur, c'est pa stres lisible, mais tu me conseillerais desimplement aller chercher le passe qui correspond au pseudo et le comparer au passe entré?

[Imhotep2]

Oui, c'est vrai sans couleur c'est pas génial.
Sinon, pour ce qui est de l'opération, oui c'est ce que je fais. Je ne sais pas si c'est le mieux et le plus rapide (un programmeur plus expérimenté te donnera sans doute autre chose) mais en tout cas ça marche. Ca me semble être le plus intuitif en tout cas.

[Amrac]

Sa solution me parait correct, sinon tu peut simplement ton:

Code PHP :

<?php 

if ($donnees['login']!="" AND $donnees['passe']!="") 

[TRasHED]

ok bon je vais retenir l'idée de imhotep, merci a tous