par
macgawel » 31 mars 2010, 15:09
Bonjour.
1. Penses à gérer le cas où on arrive directement sur la page - $_POST vide.
2. Penses à sécuriser les données reçues.
3. Il vaut mieux stocker les mots de passes chiffrés - c'est plus sûr.
Au moment de l'INSERT/UPDATE, tu encodes le mot de passe.
Et ici, tu compares le encode(mot de passe envoyé) au mot de passe encodé que tu as stocké.
Pour ça, tu peux utiliser md5() ou (mieux) sha1().
4. Je ne sais pas combien tu as d'enregistrements dans ta table, mais c'est inutile de tout sélectionner, vu que tu ne veux qu'un enregistrement précis.
Je te conseillerais d'ajouter à la clause WHERE de ta requête le login et le mot de passe (chiffré). ensuite, il te suffit de regarder si la requête renvoie bien un enregistrement...
5. Le
mysql_close() est inutile (voir
le manuel de PHP).
6. Dans ton WHILE, tu boucles sur
$data et tu testes la valeur de
$dataset...
Bonjour.
1. Penses à gérer le cas où on arrive directement sur la page - $_POST vide.
2. Penses à sécuriser les données reçues.
3. Il vaut mieux stocker les mots de passes chiffrés - c'est plus sûr.
Au moment de l'INSERT/UPDATE, tu encodes le mot de passe.
Et ici, tu compares le encode(mot de passe envoyé) au mot de passe encodé que tu as stocké.
Pour ça, tu peux utiliser md5() ou (mieux) sha1().
4. Je ne sais pas combien tu as d'enregistrements dans ta table, mais c'est inutile de tout sélectionner, vu que tu ne veux qu'un enregistrement précis.
Je te conseillerais d'ajouter à la clause WHERE de ta requête le login et le mot de passe (chiffré). ensuite, il te suffit de regarder si la requête renvoie bien un enregistrement...
5. Le [b]mysql_close()[/b] est inutile (voir [url=http://fr.php.net/manual/fr/function.mysql-close.php]le manuel de PHP[/url]).
6. Dans ton WHILE, tu boucles sur [b]$data[/b] et tu testes la valeur de [b]$dataset[/b]...