amélioration du code / suggestion

Répondre
dumont7
le 20/02/2010 à 15:13
dumont7
salut a tous :)

mon code fonctionne parfaitement mais je souhaite savoir si il y a des améliorations a faire.. ou bien des suggestions de votre part :
si oui : donnez moi un maximum d'explications SVP car je ne suis pas un expert du tout :)

inscription.php
<?php

include("../inc/conf.php");
include("../inc/fonctions.php");

if (isset($_POST["action"]) && $_POST["action"] == "send")
{
$nom = postInput($_POST["nom"]);
$prenom = postInput($_POST["prenom"]);
$email = postInput($_POST["email"]);
$login = postInput($_POST["login"]);
$password = postInput($_POST["password"]);
$confirmation = postInput($_POST["confirmation"]);

if ($nom == "")
{
$erreur = "Veuillez introduire votre nom";
}
else
{
if ($prenom == "")
{
$erreur = "Veuillez introduire votre prénom";
}
else
{
if (!EmailOK($email))
{
$erreur = "Votre email est invalide";
}
else
{
if (!okpseudo($login))
{
$erreur = "Votre login est invalide";
}
else
{
if (!okpseudo($password))
{
$erreur = "Votre mot de passe est invalide";
}
else
{
if ($password != $confirmation)
{
$erreur = "Veuillez vérifer la confirmatino de votre mot de passe";
}
else
{
$r = mysql_query("select email from membres where email='$email';");
if (mysql_num_rows($r)!=0)
{
$erreur = "Cet email existe déjà dans notre base de données";
}
else
{
$r = mysql_query("select login from membres where login='$login';");
if (mysql_num_rows($r)!=0)
{
$erreur = "Ce login existe déjà dans notre base de données";
}
else
{
$nom = addslashes($nom);
$prenom = addslashes($prenom);

$password = md5($password);
$codeact = GenererCode();

$r = mysql_query("insert into membres values ('','$login','$password','','$nom','$prenom','','$email','','$dateheure','$codeact','0');");

@mail($email,"[NomSite] Inscription","Bonjour,

Suite à votre inscription sur NomSite,
nous vous demandons de bien vouloir valider votre compte en allant à la page suivante :

http://www.nomsite.fr/membres/activation.php

et en tappant le code d'activation suivant :

$codeact

Cordialement, NomSite
-----
Ceci est un message automatique
Merci de ne pas répondre","From: ".$config["noreply"]);

header("location: activation.php?email=$email&msg=Un+email+de+confirmation+vient+de+vous+être+envoyé");
exit();
}
}
}
}
}
}
}
}
}
else
{
$login = "";
$password = "";
}

include("../inc/header.php");

?>

<h1>Inscription</h1>

<?php if (isset($erreur)) echo '<br /><br />',$erreur; ?>

<form action="inscription.php" method="post">
<input type="hidden" name="action" value="send" />

<table border="0" cellspacing="1" cellpadding="3">
<tr>
<td>Nom * :</td>
<td><input type="text" name="nom" size="30" value="<?php echo($nom); ?>" maxlength="55" /></td>
</tr>
<tr>
<td>Prénom * :</td>
<td><input type="text" name="prenom" size="30" value="<?php echo($prenom); ?>" maxlength="55" /></td>
</tr>
<tr>
<td>Email * :</td>
<td><input type="text" name="email" size="30" value="<?php echo($email); ?>" maxlength="155" /></td>
</tr>
<tr>
<td>Login * :</td>
<td><input type="text" name="login" size="30" value="<?php echo($login); ?>" maxlength="20" /></td>
</tr>
<tr>
<td>Mot de passe * :</td>
<td><input type="password" name="password" size="30" value="" maxlength="75" /></td>
</tr>
<tr>
<td>Confirmation * :</td>
<td><input type="password" name="confirmation" size="30" value="" maxlength="75" /></td>
</tr>
</table>

<br />

<input type="submit" value="Valider" />

</form>

<?php

include("../inc/footer.php");

?>
Répondre

Ecrire un message

Votre message vient d'être créé avec succès.
LoadingChargement en cours