Ma Programmation !

Répondre
Citer ce message
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

  1. <?php

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

  6. if ($_POST["nom"] == "")
  7. {
  8. $msg = "Veuillez introduire votre nom !";
  9. }
  10. else
  11. {

  13. if ($_POST["prenom"] == "")
  14. {
  15. $msg = "Veuillez introduire votre prénom !";
  16. }
  17. else
  18. {

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

  22. if (!verif_email($_POST["email"]))
  23. {
  24. $msg = "Veuillez introduire une adresse e-mail valide !";
  25. }
  26. else
  27. {

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

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

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

  40. if (!verif_password($_POST["password"]))
  41. {
  42. $msg = "Veuillez introduire un mot de passe valide !";
  43. }
  44. else
  45. {

  47. if ($_POST["password"] != $_POST["confirmation"])
  48. {
  49. $msg = "Veuillez introduire un mot de passe (confirmation) valide !";
  50. }
  51. else
  52. {

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

  61. $code = gener_password();

  63. $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());

  65. mysql_close($connexion);

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

  69. exit();

  71. }
  72. }
  73. }
  74. }
  75. }
  76. }
  77. }
  78. }

  80. ?>


vos avis, svp
Citer ce message
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 !
Citer ce message
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 !!!!!
Citer ce message
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
Citer ce message
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
Citer ce message
Bzh
le 26/06/2006 à 23:49
Bzh
Commence par remplacer:

$_POST["nom"] == ""

par

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

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

tu ma dit de faire ca :

  1. if (empty($_POST["nom"]) == "")
  2. {
  3. $msg = "Veuillez introduire votre nom !";
  4. }
  5. else
  6. {


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
Citer ce message
Bzh
le 28/06/2006 à 18:35
Bzh
Pffff !!!

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

J'ai dis:
  1. <?php
  2. if ( empty( $_POST["nom"] ) )
  3. {
  4. $msg = "Veuillez introduire votre nom !";
  5. }
  6. else
  7. {

  9. }


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