Ma Programmation !

Répondre
bdavid
le 26/06/2006 à 19:16
bdavid
bonjour, alors voila j'ai programmé une page php d'inscription pour un espace membre, je souhaite savoir si c une bonne programmation ou pas, vos avis ...

merci

<?php

if (isset($_POST['Submit']) && $_POST['Submit'] == 'Adherer')
{

if ($_POST["nom"] == "")
{
$msg = "Veuillez introduire votre nom !";
}
else
{

if ($_POST["prenom"] == "")
{
$msg = "Veuillez introduire votre prénom !";
}
else
{

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

if (!verif_email($_POST["email"]))
{
$msg = "Veuillez introduire une adresse e-mail valide !";
}
else
{

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

$req = mysql_query("select id from membres where email="".$_POST["email"]."" and (etat="0" or etat="1")") or die(mysql_error());

if (mysql_num_rows($req)!=0)
{
$msg = "Cet e-mail existe déjà dans la base de données !";
}
else
{

if (!verif_password($_POST["password"]))
{
$msg = "Veuillez introduire un mot de passe valide !";
}
else
{

if ($_POST["password"] != $_POST["confirmation"])
{
$msg = "Veuillez introduire un mot de passe (confirmation) valide !";
}
else
{

if (isset($_POST["reglement"]) != "1")
{
$msg = "Pour pouvoir vous inscrire, vous devez accepter les conditions d'utilisation !";
}
else
{

$code = gener_password();

$req = mysql_query("insert into membres values ("","".addslashes($_POST["nom"])."","".addslashes($_POST["prenom"])."","".$_POST["email"]."","".md5($_POST["password"])."","".date("Y-m-d H:i:s")."","".$_POST["newsletter"]."","$code","0")") or die(mysql_error());

mysql_close($connexion);

header("location: inscription.php?msg_ok=Un+email+vient+de+vous+être+envoyé+!");

exit();

}
}
}
}
}
}
}
}

?>


vos avis, svp
fabrice11901
le 26/06/2006 à 19:34
fabrice11901
Salut
Moi pour la vérification des post bien saisis dans le formulaire j'aurais pas fais comme ça mais j'aurai fais un message global "vous devez remplir tous les champs" mais une autre façon d'utiliser les if (isset...
Mais bon si ça marche...
Par contre le if .. submit ça te sersà quoi ça?
moi, j'adore le php !
moogli
le 26/06/2006 à 21:09
moogli
pourquoi demander la même chose sur plusieurs forums ?

http://phpdebutant.org/forum_msg246699.php
Il en faut peu pour être heureux !!!!!
bdavid
le 26/06/2006 à 21:41
bdavid
moogli : pour avoir plusieurs avis :) je pense pas que c mal vu si c pas de la pub biensur

merci pour vos réponses a tous
bdavid
le 26/06/2006 à 21:46
bdavid
moogli : si j'affiche le $msg mais j'ai simplement mi le code des verifs, req

sinon explique toi stp sur :

"Perso je fait comme toi, mais en utilisant le if plutot que la else ( je vérifie que les infos soit conforme a ce que je souhaite, avec isset et / ou empty !"

pourrai tu me faire un exemple ?

merci d'avance pour vos réponses
Bzh
le 26/06/2006 à 23:49
Bzh
Commence par remplacer:

$_POST["nom"] == ""

par

empty($_POST["nom"]) !!!

Ce sera déja beaucoup plus propre...
bdavid
le 27/06/2006 à 00:41
bdavid
ton code n'est pas bon !!

tu ma dit de faire ca :

if (empty($_POST["nom"]) == "")
{
$msg = "Veuillez introduire votre nom !";
}
else
{


quand je clic sur le bouton du formulaire et que je rempli pas le champ "nom" il m'affiche aucun msg d'erreur alors que normalement il doi m'affiché un message !! tinquiete j'affiche bien les messages dans la vrai page ;) donc marche pas ta solution!!dsl
Bzh
le 28/06/2006 à 18:35
Bzh
Pffff !!!

J'ai pas dis ça smiley !!!!

J'ai dis:
<?php
if ( empty( $_POST["nom"] ) )
{
$msg = "Veuillez introduire votre nom !";
}
else
{

}


Ce sont des abitudes apprendre car, si $_POST["nom"] n'est pas déclaré, cela ne retourne aucun message d'erreur !!!!

Le moindre message d'erreur est une faille !!! Car un message renvoit beaucoup, beaucoup d'infos sur l'arboressence de ton site...

Ne pas l'oublier...
Répondre

Ecrire un message

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