Insertion données avec une requête préparée

Version imprimable

Bonjour,

Je vous expose mon problème simplifié.
J'ai une table produit :
produit (ref_produit,nom_produit,num_sav)
La clé étrangère num_sav est clé primaire de la table SAV
sav-constructeur (num_sav,nom_sav,adresse_sav)

Le base est crée sur phpmyadmin.
Voici ma requête implémentée dans mon code :
Code:

1 2 3 4 5 6 $req = $bdd->prepare('INSERT INTO `produits`(`REF_PRODUIT`, `NOM_PRODUIT`, `NUM_SAV`) VALUES (:ref-produit,:nom-produit,:num-sav)'); // On prépare la requète $req->execute(array( // et on insère 'ref-produit' => $_POST['ref-produit'], 'nom-produit' => $_POST['nom-produit'], 'num-sav' => $donnees[0] ));
$donnees[0] est la clé du sav auquel le produit est associé.

Evidemment cette requête ne marche pas et je ne comprend absolument pas pourquoi ...

Des idées ? Suggestions ? Remarques ?

Cordialement.

C'est mieux de mettre les : dans le nom des paramètres :
Code:

1 2 3 4 5 $req->execute(array( // et on insère ':ref-produit' => $_POST['ref-produit'], ':nom-produit' => $_POST['nom-produit'], ':num-sav' => $donnees[0] ));
Sinon quelle erreur as-tu ?

22/05/2013, 20h47
Mikma

Merci

Tout d'abord merci pour la réponse rapide.

J'ai exactement cette erreur :

Citation:

Warning: PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: parameter was not defined
23/05/2013, 12h17
Celira

A priori, une des données que tu utilises comme paramètre est vide.
Est-ce que tu contrôles ces données avant ?
23/05/2013, 13h56
Meyfarth

Peut-être un problème de quote autour du nom du produit ?

Code:

$req = $bdd->prepare("INSERT INTO `produits`(`REF_PRODUIT`, `NOM_PRODUIT`, `NUM_SAV`) VALUES (':ref-produit',':nom-produit',':num-sav')");

Je n'utilise pas PDO (je suis passé directement du mysql_... à Doctrine) donc à prendre avec des pincettes, mais c'est une des erreurs que je faisais régulièrement

Apparemment c'était un problème de quotes, je n'ai plus l'erreur.
Le problème maintenant est que lors de l'ajout, ça ajoute :
0 pour ref-produit
:nom-produit pour nom-produit
0 pour num-sav
....

Je vous re-montre mon code que j'ai désormais :

Code:

1
2
3
4
5
6
7
 
$req = $bdd->prepare("INSERT INTO `produits`(`REF_PRODUIT`, `NOM_PRODUIT`, `NUM_SAV`) VALUES (':ref-produit',':nom-produit',':num-sav')");
$req->execute(array(
 ':ref-produit' => $_POST['ref-produit'],
 ':nom-produit' => $_POST['nom-produit'],
 ':num-sav' => $donnees[0]
));

Edit : en fait j'ai enlevé les quotes autour de :ref-produit etc ... l'erreur me revient encore, je trouve ça assez énorme ...
Du coup je vous met le code entier de la page :

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
<!DOCTYPE html>
<html>
<head>
<title>Ajout d'un produit</title>
 
<meta charset="utf-8" /></head>
 
<body>
	<?php 
 
 
 
try
{
    // On se connecte en MySQL
    $bdd = new PDO('mysql:host=localhost;dbname=toutebo', 'root', '');
 
}
catch(Exception $e)
{
    // En cas d'erreur, on affiche un message et on arrète tout
        die('Erreur : '.$e->getMessage());
}
 
$reponse = $bdd->query("SELECT NUM_SAV from `toutebo`.`sav-constructeurs` WHERE NOM_SAV='".$_POST['choix']."'");
$donnees = $reponse->fetch();
echo $donnees[0];
 
$req = $bdd->prepare("INSERT INTO `produits`(`REF_PRODUIT`, `NOM_PRODUIT`, `NUM_SAV`) VALUES (:ref-produit,:nom-produit,:num-sav)");
 // On prépare la requète
$req->execute(array( // et on insère
 ':ref-produit' => $_POST['ref-produit'],
 ':nom-produit' => $_POST['nom-produit'],
 ':num-sav' => $donnees[0]
));
 
echo "Le produit ";
echo $_POST['nom-produit'] ;
echo " a été ajouté <br/>";
echo "Il a pour référence : ";
echo $_POST['ref-produit'];
$reponse->closeCursor();
 
?>
 
 
 
</br>
<a href="index.php">Revenir au menu</a>
 
</body>
</html>

Si je fais echo de tous 3 paramètres ça m'affiche bien ce que j'ai entré dans le formulaire

23/05/2013, 16h03
Meyfarth

Un exemple de ce que tu as rentré dans le formulaire ? (Et, pourquoi pas, le code HTML de ce fameux formulaire)
23/05/2013, 16h22
sabotage

Le nom des paramètres ne peut pas contenir un tiret.
Ce n'est malheureusement pas documenté.
23/05/2013, 17h25
Celira

Effectivement, après recherche, sabotage a raison (je me coucherai moins bête ce soir ;)) et effectivement c'est très mal, voire pas du tout documenté.
:arrow: https://bugs.php.net/bug.php?id=43130

En bref, mieux vaut se limiter aux alphanum + underscore (comme pour les variables PHP en fait)
23/05/2013, 21h59
Mikma

Effectivement j'ai changé tous les tirets par des underscores et plus aucune erreur, l'ajout se fait également dans la base de données.

Merci beaucoup pour vos réponses, c'est vraiment gentil.
@+