Vous n'êtes pas identifié(e).
- Contributions : Récentes | Sans réponse
Pages :: 1
#1 04-02-2010 10:09:02
- david7
- Membre
- Inscription : 05-06-2009
- Messages : 87
des suggestions ?
salut a tous 
j'aimerais savoir si vous avez des suggestions pour mon code...
// on teste si le visiteur a soumis le formulaire
if (isset($_POST['inscription']) && $_POST['inscription'] == 'Inscription') {
// on teste l'existence de nos variables. On teste également si elles ne sont pas vides
if ((isset($_POST['login']) && !empty($_POST['login'])) && (isset($_POST['pass']) && !empty($_POST['pass'])) && (isset($_POST['pass_confirm']) && !empty($_POST['pass_confirm']))) {
// on teste les deux mots de passe
if ($_POST['pass'] != $_POST['pass_confirm']) {
$erreur = 'Les 2 mots de passe sont différents.';
}
else {
$base = mysql_connect ('serveur', 'login', 'password');
mysql_select_db ('nom_base', $base);
// on recherche si ce login est déjà utilisé par un autre membre
$sql = 'SELECT count(*) FROM membre WHERE login="'.mysql_escape_string($_POST['login']).'"';
$req = mysql_query($sql) or die('Erreur SQL !<br />'.$sql.'<br />'.mysql_error());
$data = mysql_fetch_array($req);
if ($data[0] == 0) {
$sql = 'INSERT INTO membre VALUES("", "'.mysql_escape_string($_POST['login']).'", "'.mysql_escape_string(md5($_POST['pass'])).'")';
mysql_query($sql) or die('Erreur SQL !'.$sql.'<br />'.mysql_error());
session_start();
$_SESSION['login'] = $_POST['login'];
header('Location: membre.php');
exit();
}
else {
$erreur = 'Un membre possède déjà ce login.';
}
}
}
else {
$erreur = 'Au moins un des champs est vide.';
}
}
?>
<html>
<head>
<title>Inscription</title>
</head>
<body>
Inscription à l'espace membre :<br />
<form action="inscription.php" method="post">
Login : <input type="text" name="login" value="<?php if (isset($_POST['login'])) echo htmlentities(trim($_POST['login'])); ?>"><br />
Mot de passe : <input type="password" name="pass" value="<?php if (isset($_POST['pass'])) echo htmlentities(trim($_POST['pass'])); ?>"><br />
Confirmation du mot de passe : <input type="password" name="pass_confirm" value="<?php if (isset($_POST['pass_confirm'])) echo htmlentities(trim($_POST['pass_confirm'])); ?>"><br />
<input type="submit" name="inscription" value="Inscription">
</form>
<?php
if (isset($erreur)) echo '<br />',$erreur;
?>
</body>
</html>
Hors ligne
#2 04-02-2010 10:55:45
Re : des suggestions ?
Tu peux rejoindre isSet et Empty dans un seul Empty (qui appelle isSet dans sa définition).
Quand au mysql_fetch_array que tu fais je le remplacerai par un mysql_fetch_rows (compter le nombre d'enregistrement retourné).
Hors ligne
#3 04-02-2010 12:39:33
- david7
- Membre
- Inscription : 05-06-2009
- Messages : 87
Re : des suggestions ?
tu peux me dire pourquoi stp ? j'apprend en meme temps^^ enfin pour la premiere suggestion tu pourrais me faire un model car comment "rejoindre" les 2 en 1 ? 
pour ta 2eme remarque, ok mais pourquoi ?
Hors ligne
#4 04-02-2010 12:47:46
- Maljuna Kris
- Infantimigulo
- Lieu : Douarnenez 29100 Breizh Izel
- Inscription : 08-05-2009
- Messages : 2 453
- Site Web
Re : des suggestions ?
Tu peux rejoindre isSet et Empty dans un seul Empty (qui appelle isSet dans sa définition).
Quand au mysql_fetch_array que tu fais je le remplacerai par un mysql_fetch_rows (compter le nombre d'enregistrement retourné).
Tout cela m'a l'air 'cor'assez approximatif, SCROGNEUGNEU !
Gloire à qui n'ayant pas d'idéal sacro-saint,
Se borne à ne pas trop emmerder ses voisins. G. Brassens Don Juan 1976.
Avĉjo MoKo kantas
La chaîne YouTube MoKo Papy
Hors ligne
#5 04-02-2010 14:36:16
- Maljuna Kris
- Infantimigulo
- Lieu : Douarnenez 29100 Breizh Izel
- Inscription : 08-05-2009
- Messages : 2 453
- Site Web
Re : des suggestions ?
S'agissant de IsSet et Empty, rappel d'un billet de ce bon vieux Fredoche : isset_empty
Quant au reste, attention à ne pas confondre mysql_fetch_row() et mysql_num_rows()
Gloire à qui n'ayant pas d'idéal sacro-saint,
Se borne à ne pas trop emmerder ses voisins. G. Brassens Don Juan 1976.
Avĉjo MoKo kantas
La chaîne YouTube MoKo Papy
Hors ligne
#7 04-02-2010 18:23:22
- david7
- Membre
- Inscription : 05-06-2009
- Messages : 87
Re : des suggestions ?
xtg : j'ai regardé mais.. enfin je comprend pas trop! ni ta premiere remarque ni ta 2eme j'ai pourtant lu la petite doc dessus dans ton lien mais je comprend pas le rapport avec ta phrase "Tu peux rejoindre isSet et Empty dans un seul Empty"
j'ai demandé a un ami il comprend pas non plus... c'est pas trés claire pour un debutant. réexplique moi STP si sa te dérange pas biensur xtg ! (ou Maljuna Kris si ta conpris se qu'il voulais dire..)
Hors ligne
#8 04-02-2010 19:24:08
Re : des suggestions ?
Lis le lien qu'a posté Kris, tu devrais comprendre tout est expliqué dedans. ^^
Pour la seconde (malgré ma faute de frappe) je disais :
inutile d'utiliser une fonction qui récupère des éléments alors que tu ne souhaites que vérifier un nombre d'enregistrement. Si ta requête renvoie un enregistrement c'est qu'il y a déjà un membre portant ce pseudo, on a rien besoin de savoir d'autre. Donc au lieu d'utiliser mysql_fetch_array et un count dans ta requête je te conseille plutôt un mysql_num_rows qui compte le nombre d'enregistrement de la requête :
$sql = 'SELECT id FROM membre WHERE login="'.mysql_escape_string($_POST['login']).'"';
$req = mysql_query($sql) or die('Erreur SQL !<br />'.$sql.'<br />'.mysql_error());
if( mysql_num_rows($req) == 1 )
echo 'Compte existant.';
else
echo 'inscription';
Hors ligne
#9 04-02-2010 22:12:21
- david7
- Membre
- Inscription : 05-06-2009
- Messages : 87
Re : des suggestions ?
"Lis le lien qu'a posté Kris, tu devrais comprendre tout est expliqué dedans. ^^"
"-> j'ai pourtant lu la petite doc dessus dans ton lien" A premiere vu tu as pas lu entierement mon post oh pas biennnn^^
sinon pour le mysql_num_rows daccord mais c'est pas trés grave.. si ? enfin je veux dire c'est du propre aussi quoi non ?
Dernière modification par david7 (04-02-2010 22:13:09)
Hors ligne
Pages :: 1