Nettoyage de code ?

Salutations,

Nous avions besoin d'un masque de saisie PHP / MySQL.
Nous butons sur l'insertion dans la base de données SQL. En effet, cela renvoie à chaque fois une erreur SQL. Toutes les variables sont pourtant bien déclarées.
Nous sommes en PHP Version 5.4.19.

Voici la partie du code en question :

Code:

---

1 2 3 4 5 6

(...) // Insertion SQL $requete = "INSERT INTO " . $table . " (Id,Dispo,Ecart,Donateur,Date,Ref,IP,Edition,Type,Exemplaires,Etat,Commentaire,Acqui,Collection,Valorisation,Condition,Localisation) VALUES ('','$dispo','$ecart','$donateur','$date','$ref','$ip','$edition','$type','$exemplaires','$etat','$commentaire','$acqui','$collection','$valorisation','$condition','$localisation')";   mysql_query($requete) or die('<strong>Erreur SQL !</strong> :<br /><em>'.$requete.'</em><br />'.mysql_error().'<br /><br /><a href="[http://wda-fr.org/test/SQL/Destock/saisie.php]"><strong>-=> Nouvelle insertion <=-</strong></a>'); (...)

---

La base SQL ne reçoit pas les datas, alors que le listing des précédentes s'effectue sans pb.

Auriez-vous une idée ?
Cela serait-il du à notre récente migration de PHP 5.2.17 en PHP 5.4.19 ?

A vous lire,

Bonjour,
un message d'erreur ? (lequel ?)

N.B. L'extension mysql_ est obsolète.
Utiliser mysqli_ ou PDO.

Citation:

---

Envoyé par jreaux62

(...)
N.B. L'extension mysql_ est obsolète.
Utiliser mysqli_ ou PDO.

---

Tout le monde me dit cela, mais j'ai déjà du mal à comprendre la syntaxe actuelle... :)
Comment devrais-je faire pour du mysqli_ ou PDO ?

=> Comprendre PDO

L’intérêt des requêtes préparées : sécurisation des données.

Un exemple d'utilisation : Système de Gestion-Affichage de Nouvelles

N.B. Sauf si tu est sur free.fr
Aux dernières nouvelles, PDO n'est pas implémenté.

Citation:

---

Envoyé par jreaux62

=> Comprendre PDO
(...)

---

Merci de ce tuto. Je regarderais cela quand j'aurais un peu de temps à y consacrer.

Pour mon pb, any idea ?

Précision ?
Une question précise amène une réponse précise...

Citation:

---

Envoyé par Casio

Auriez-vous une idée ?

---

... n'est pas une question précise.

Citation:

---

Envoyé par jreaux62

Précision ?
Une question précise amène une réponse précise...
(...)

---

Citation:

---

Envoyé par Casio

(...)
Nous butons sur l'insertion dans la base de données SQL. En effet, cela renvoie à chaque fois une erreur SQL. Toutes les variables sont pourtant bien déclarées.
(...)

---

"Qu'est-ce qui ne fonctionne pas dans ce code ?"

Citation:

---

Envoyé par jreaux62

un message d'erreur ? (lequel ?)

---

J'ai donné une correction.
A-t-elle été utilisée/testée ?

Citation:

---

Envoyé par Casio

Je regarderais cela quand j'aurais un peu de temps à y consacrer.

---

8O

Citation:

---

Envoyé par jreaux62

J'ai donné une correction.
A-t-elle été utilisée/testée ?
(...)

---

Je suis en train de chercher sur différents site ce que je dois mettre dans cette partie ("SET")... Nomenclature, etc...
Ceci dit, l'ensemble des sites consultés ne semblent pas parler de l'obligation de SET.

Citation:

---

Envoyé par jreaux62

(...)C'est un GAG ???

---

Non, je ne suis pas développeur. Le dernier langage que j'ai essayé était du GW-BASIC en 1984.
Je me borne à recopier bêtement ce que l'on me conseille à droite à gauche, car je fais cela pour une structure associative bénévole qui n'a personne d'autre pour le faire.

Nous aussi, nous sommes bénévoles.

Le minimum, quand on donne une réponse/correction, est de dire si oui ou non cela répond à la problématique.

La 2ème étant de montrer le message d'erreur.

J'en conviens, et merci de me répondre.
J'ai donc bien détaillé que j'avais cherché, EN VAIN, comment utiliser cette option SET.
J'ai aussi donné le message d'erreur que je reçois.

Je ne comprends pas cette virulence à mon égard. Si je savais le faire, je ne serais surement pas la.

Si le sujet doit partir en confiture, autant en rester la pour ne pas tomber dans de l'hors-sujet.
Si quelqu'un veut bien m'aider, je suis toutes ouïes. ;-)

Bonjour
Tu peux, mais ce n'est pas indispensable, utiliser SET dans une requête INSERT. Pour plus d'informations, je te conseille la doc officielle ici http://dev.mysql.com/doc/refman/5.0/fr/insert.html

Concernant ta requête, si le champ id est un autoincrément, tu n'as pas à le faire figurer, ni comme champ, ni dans la partie VALUES.Il se remplira automatiquement. Ce qui donnerait

Code:

---

$requete = "INSERT INTO " . $table . " (Dispo,Ecart,Donateur,Date,Ref,IP,Edition,Type,Exemplaires,Etat,Commentaire,Acqui,Collection,Valorisation,Condition,Localisation) VALUES ('$dispo','$ecart','$donateur','$date','$ref','$ip','$edition','$type','$exemplaires','$etat','$commentaire','$acqui','$collection','$valorisation','$condition','$localisation')";

---

Enfin, je n'ai pas trouvé dans tes posts le message d'erreur MySQL ...

Que donne :

Code:

---

echo $requete

---

Quel est le message d'erreur retourné par mysql.

Sans cela difficile de t'aider correctement.

Bien...
En 3 réponses, solution

Citation:

---

CONTITION est un mot réservé pour MySQL. Change le nom du champ ou entoure-le d'accents graves (Alt Gr - 7)

---

.

Je passe le sujet en résolu.

Si dès le départ on avait eu le message d'erreur mysql , on aurait pu répondre aussi :) .

Quand on à un problème la moindre des choses c'est de fournir le plus d'infos possible pour aider à le résoudre.