insertion non voulue, lors de verifications

Version imprimable

Bonjour,

J'ai éssayé de mettre un "anti doublon" lors d'insertion de données via mon formulaire

J'introduis donc des urls dans une base de donnée, & je veux m'assurer que cette url n'éxiste pas déja avant de l'insérer.

Seulement systématiquement quand je charge la page du formulaire pour la premiere fois (base vide) le script envoie directement une valeur vide dans la base. ( pas la deuxieme, vu que la il dit que l'adresse éxiste déja :roll: )

mon code ressemble à :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 <?php mysql_connect(*********) or die(mysql_error()); mysql_select_db("******") or die(mysql_error()); // Ajout du lien dans la base de donnée $result = mysql_query("SELECT url FROM liens WHERE url = '" . $url . "'") or die(mysql_error()); if(mysql_num_rows($result) == 0) { //ajouter l'url mysql_query("INSERT INTO liens VALUES('','" . $url . "','" . $nom . "', '" . $type . "', '" . $urlsubmit . "')"); echo 'l\'annuaire '.$nom.' à été ajouté dans la liste'; } else { echo 'L\'annuaire '.$nom.' est déja présent dans la liste'; } ?> <form action="add-liens.php" method="post" enctype="multipart/form-data"> . . .
Comment puis-je éviter cet ajout automatique ?

Cordialement

Salut,

du peu que je comprenne, tu as mutualisé ton script de sorte qu'il serve a la fois pour afficher le formulaire et a la fois pour l'insert.

Du coup qd tu requete ce script, il est normal que le serveur qu'il fasse ce qu'on lui dit ( ce qu'il y a sur ce script) peut être.

que la partie insertion devrait être conditionnée par la détection du $_POST.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 mysql_connect() or die(mysql_error()); mysql_select_db("") or die(mysql_error()); // Ajout du lien dans la base de donnée if(isset($POST["url"])) { $result = mysql_query("SELECT url FROM liens WHERE url = '" . $url . "'") or die(mysql_error()); if(mysql_num_rows($result) == 0) { //ajouter l'url mysql_query("INSERT INTO liens VALUES('','" . $url . "','" . $nom . "', '" . $type . "', '" . $urlsubmit . "')"); echo 'l\'annuaire '.$nom.' à été ajouté dans la liste'; } else echo 'L\'annuaire '.$nom.' est déja présent dans la liste'; } ?> <form action="add-liens.php" method="post" enctype="multipart/form-data"> . .

04/08/2006, 20h34
Him

Salut

je n'ai pas trop compris ce que tu voulais dire avec le POST (noobs powaa)

mais je crois avoir compris que tu me conseilles de splitter les pages

avec une page form.php qui réenvoie sur une page formok.php c'est bien cela ?
04/08/2006, 21h07
gisele

non du tout,

garde comme çà ne split pas, fait juste comme j'ai fait.

A ce propros je te conseille de prendre l'habitude d'utiliser le super global $_POST.

C'est à dire $_POST["nom_du_champ"] plutot que directement $nom_du_champ.

donc pour le cas ou ton champ texte pur l'url s'appelle url, utiliser $_POST["url'] plutot que $url.

-Cela rend ton code plus portable, notamment pour le cas ou tu devrais t'accomoder d'un serveur avec register globals a off.

-cela permet du premier coup d'oeil, pour quiconque ne connait pas ton script de determiner la provenance de ta variable (parametre passé en methode post).d'ailleurs celame fait pense que j'ai supposé que tu est en post mais si çà se trouve tu utilse la méthode get.
A cet instant précis je ne sais pas pour toi mais l'inconvénient de ne pas utiliser les superglobaux est palpables :-)
=> donc si t'es en get remplace $_POST["url"] par $_GET["url"]
non je déconne, y a un bout de ton formulaire qui dépasse en bas de ton script :-)

-il est accessible de partout, de tous les contextes que ce soit de l'interieur d'une fonction utilisateur, ou d'une classe etc.
04/08/2006, 22h25
Him

ok, je test ca :)

merci à toi pour toutes ces explications !
04/08/2006, 22h29
Him

ah au fet,
il faut que j'applique cela sur chacunes des autres variables du formulaire alors ?

Cordialement
04/08/2006, 22h35
gisele

Bah si elles marchaient comme çà non, t'enquiquine pas pour çà mais prend cette habitude parcontre a l'avenir, c'est beaucoup plus parlant (surtout pour les autres) dans un script de voir un $_POST["ma_var"] que de voir débarquer $ma_var de nulle part.

Bref c'est un détail, l'essence de ma solution c'est surtout de tester justement $_POST["url"] avant d'appliquer ou pas la portion de script qui fait l'insertion en base, l'idée étant de s'assurer avant ce travail que le formulaire avait bien été soumis, çà évite l'insertion des le premier acces au script ;-)