Discussion :

[StephBretagne]

phpMyAdmin Version: 3.4.10.1
MySQL Version du serveur: 5.1.63-cll

Bonjour,

Je travaille sur un site présentant des films documentaires et des séries de films documentaires.
Je viens de rajouter une 18e table à ma BDD pour que les prix (récompenses) attribués aux documentaires puissent se mettre à jour automatiquement.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
-- --------------------------------------------------------
--
-- Structure de la table documentaire 
--
idDoc int(3) NOT NULL AUTO_INCREMENT
...
...
idPrix int(4) ,
CONSTRAINT pk_documentaire
	PRIMARY KEY (idDoc),
...
CONSTRAINT fk_prixDocSerie_documentaire
       FOREIGN KEY (idPrix) REFERENCES prixDocSerie (idPrix) ON DELETE CASCADE
) ENGINE=InnoDB  DEFAULT CHARSET=utf8 AUTO_INCREMENT=1;
 
-- --------------------------------------------------------
--
-- Structure de la table prixDocSerie 
--
CREATE TABLE prixDocSerie (
idPrix INT(4) NOT NULL AUTO_INCREMENT ,
nomPrix_fr VARCHAR (100),
nomPrix_en VARCHAR(100),
CONSTRAINT pk_prixDocSerie
	PRIMARY KEY (idPrix) 
) ENGINE=InnoDB  DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;

J'aimerai envoyer les données en base en utilisant le formulaire de saisie que j'ai déjà conçu pour les documentaires (et qui, pour le reste, fonctionne parfaitement) mais je n'y arrive pas.

Si je rajoute les infos directement à la main dans MySQL, il me renvoie les codes suivants :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
INSERT INTO `prixDocSerie` (`idPrix`, `nomPrix_fr`, `nomPrix_en`) VALUES (NULL, 'test_fr', 'test_en');
UPDATE `documentaire` SET `idPrix` = 'x1' WHERE `documentaire`.`idDoc` =x2;

Pour le début du formulaire j'ai déjà établi que x2=$idDocumentaire.

Comment écrire cette requête dans mon formulaire, pour que l'idPrix auto-incrémenté de prixDocSerie soit reporté à la place du x1 de la table documentaire ?

Merci d'avance de vos lumières.

[ericd69]

salut,

et ça te dit quoi quand tu le fais à la main?

ton problème c'est quoi coté sql ou le formulaire?

[StephBretagne]

Merci pour ta réponse Eric

Quand je le fais à la main, ça ne me dit rien de spécial, ça me rajoute bien les infos dans la BDD.

Mon problème, c'est pour le formulaire. Je commence comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
// ***** récup nomPrix dans récompense ***** 
if(isset($_POST['nomPrix_fr']) && isset($_POST['nomPrix_en'])) 
$nomPrix_fr=mysql_real_escape_string ($_POST ["nomPrix_fr"]);	 
$nomPrix_en=mysql_real_escape_string ($_POST ["nomPrix_en"]);

mais je n'arrive pas à construire mon $req=mysql_query(...);

[ericd69]

déjà un bon truc abandonne l'extension mysql et passe à pdo ou au moins mysqli vu qu'elle va disparaitre

ensuite à savoir que le nombre entre parenthèse ne correspond pas du tout à la taille de l'entier mais juste au chiffres significatifs, pour l'adapter tu dois utiliser les types mysql correspondant (tinyint (8 bits), smallint (16 bits), mediumint (24 bits), int (32 bits) et bigint (64 bits))

donc actuellement tu te donnes 2 milliards et des brouettes de possibilité pour chaque int que tu as... au passage tu perds la moitié des valeurs (celles qui sont négatives vu qu'un général on le utilise pas) donc mettre du unsigned te permet pour du int d'avoir 4 milliards de possibilités...

déjà 2 réflexions sur tes tables:
et si tu as plusieurs prix pour un doc?
le "on cascade delete" est-il bien raisonnable ?

pour le formulaire
tu peux le rajouter sous forme d'un "<select>" que tu alimentes en lisant ta table prixDocSerie en récupérant la valeur numérique plutôt que le nom... ce qui permet de mieux sécuriser contre l'injection et évite une requête pour récupérer cette valeur...

[StephBretagne]

Pour le pdo, ce sera pour une prochaine fois vu que je suis en train de finir le site, que j'ai des centaines de requêtes comme ça et que j'ai vraiment pas le courage de tout reprendre .

Pour le type de colonne, tu as raison, un smallint (4) unsigned serait suffisant.

Pour le nombre de prix, il n'y en aura qu'un par film puisque c'est juste du texte qui peut en indiquer plusieurs (ex: Primé aux festivals de Leipzig, Popoli, FIPA, Leningrad, Clermont-Ferrand, Montréal).

Pour le formulaire, il ne sera accessible qu'à l'administrateur qui n'injectera rien (c'est à peine s'il sait cliquer sur OK ).

Pour info maintenant, je sais faire la requête s'il s'agit de remplir une seule table. Par exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
// ****** récup du n° de film dans la série / ex : film 5 (sur 10) *******
if(isset($_POST['noSerieDoc'])) { 
$noSerieDoc=$_POST["noSerieDoc"];
$sqlInsertNoSerieDoc=mysql_query ("UPDATE documentaire SET noSerieDoc = '".$noSerieDoc."' WHERE idDoc = '".$idDocumentaire."'"); 
}

pour le formulaire
tu peux le rajouter sous forme d'un "<select>" que tu alimentes en lisant ta table prixDocSerie en récupérant la valeur numérique plutôt que le nom... ce qui permet de mieux sécuriser contre l'injection et évite une requête pour récupérer cette valeur...

Là, je ne comprends pas...

[ericd69]

moi j'étais parti sur un truc plus ouvert... qu'un texte dans une table déportée...

une table
primé(id,iddoc,idfestival,annee)
festival(id,nom,....)
là tu peux donc avoir une liste déroulante pour les festivals, un champ de saisie pour l'année

quand tu as des trucs comme ça il peut être bon de faire un petit module ajax qui te permet de dynamiquement mettre à jour ta liste

[StephBretagne]

Je te remercie pour ta réponse Eric mais justement, j'y ai bien pensé et non, ce n'est pas comme ça que je voulais faire mon truc.

Des listes déroulantes et des checkboxes, j'en ai pour le genre, la version, la nature, le format et la diffusion, mais là ce n'est pas possible car chaque prix est unique dans la mesure où l'année ne m'est pas toujours fournie et où le même prix n'a pas été décerné deux fois.

De plus, je suis déjà nulle en sql/php, alors l'ajax qui suppose de dominer javascript....

Du coup, serait-il possible qu'il y ait une solution au problème : comment faire le $req=mysql_query(...); pour les tables que j'ai ?
Ou est-ce que ce n'est tout simplement pas faisable ?

[Fench]

Bonjour,

Dans ton cas, comme tu ne peux pas passer par l'ajax, il faut qd même que tu retournes sur le serveur web pour mettre à jour ta requête avec le serveur de ta base de données.

Donc pour y remedier, il faut que qd l'utilisateur choisi le document et le prix, réafficher cette page.
Question: comment marche ton système pour associer le document et le prix?
(j'ai pas vu ou pas compris)

Ensuite nous écrirons une petite fonction javaScript pour accualiser le formulaire et en dernier la mise à jour de la base.

[StephBretagne]

Bonjour French et merci pour ton aide

L'utilisateur va remplir un très long formulaire et entrer dedans, à la fin, le nom du prix en français et en anglais.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<fieldset>
<legend>Ne remplir que si le film a reçu une récompense</legend> 
 
<p class="form">	
	<label for="nomPrix_fr">Nom de la récompense en français<br>
	<span class="info">Ex : Prix du public au Festival de Moscou 1992 <br/>ou<br/>Primé aux festivals de Leipzig, Popoli, FIPA, Leningrad </span></label>
	<input type="text" id="nomPrix_fr" name="nomPrix_fr" value="<?php recupVar("name"); ?>"  maxlength="100" size="50" />
</p>	
 
<p class="form">	
	<label for="nomPrix_en">Nom de la récompense en anglais<br>
	<span class="info">Ex : Audience Prize at Moscow Festival 1992 <br/>ou<br/>Awards from festivals of Leipzig, Popoli, FIPA, Leningrad </span></label>
	<input type="text" id="nomPrix_en" name="nomPrix_en" value="<?php recupVar("name"); ?>"  maxlength="100" size="50" />
</p>
</fieldset>

Ce formulaire fonctionne parfaitement pour tout le reste. Ex pour une liste déroulante partie php :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
// ****récup liste déroulante NaturDoc **** 
if(isset($_POST['naturDoc'])) { 
$idNaturDoc=$_POST["naturDoc"];
$sqlUpdateNaturDoc=mysql_query ("UPDATE documentaire SET idNaturDoc = '".$idNaturDoc."' WHERE idDoc = '".$idDocumentaire."'")  ; 
}

Pour le réaffichage de la page, après le submit final qui met fin à la saisi des infos, l'utilisateur arrive sur une autre page via un window.location lui disant que les infos ont bien été entrées en base et lui donnant les liens pour constater le résultat en fçais et anglais sur les pages publiques (tout ça est déjà fait).

Pour le lien entre les deux tables, il s'effectue via l'idPrix qui est sur la table documentaire (foreign key) et la table prixDocSerie (primary).

[Fench]

Bonjour French

C Fench je sais que la Bretagne est proche des Anglais mais qd même

Sinon, comme je vois la redirection est faite, le prix est sur un champ texte dont l'id est nomPrix_fr pour le prix en Français

Lors du submit, tu récupéres donc ce texte avec le $_POST correspondant et tu peux faire la requête de l'INSERT mais ton pb se pose sur le UPDATE, c'est ça ?

[StephBretagne]

C Fench je sais que la Bretagne est proche des Anglais mais qd même

Oooops ! Ttes mes excuses, j'ai lu trop vite en effet, désolée !

Je viens de refaire des tests et même l'insert n'a pas l'air de fonctionner en fait.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$sqlInsertPrix=mysql_query("INSERT INTO prixDocSerie (idPrix,nomPrixDoc_fr,nomDoc_en,datCopyright,nomRea,nomAut,nomChore) VALUES (''.'".$nomPrix_fr."', '".$nomPrix_en."')");

Mais, sauf si je me trompe grossièrement, puisque j'ai déjà partiellement rempli la table prixDocSerie à la main, c'est déjà un update qu'il faut commencer par faire, non ? puis un second update pour insérer l'idPrix dans la table documentaire.

[Fench]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$sqlInsertPrix=mysql_query("INSERT INTO prixDocSerie (idPrix,nomPrixDoc_fr,nomDoc_en,datCopyright,nomRea,nomAut,nomChore) VALUES (''.'".$nomPrix_fr."', '".$nomPrix_en."')");

Dans cette requête d'insert, l'id est en auto donc pas besoin, ET tous les champs que tu écrits après le nom de table doivent être renseigné, ET tous les champs définis dans la table de ta base comme not null doivent donc être insérés

Ce qui pourrait donner (je traite pas le cas des champs not null)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$sqlInsertPrix=mysql_query("INSERT INTO prixDocSerie (nomPrixDoc_fr,nomDoc_en) VALUES (''.'".$nomPrix_fr."', '".$nomPrix_en."')");

[Fench]

Ensuite une fois l'insert résolu, tu récupére l'id généré par la fonction
mysql_insert_id puisque tu utilises mysql_xx

Et l'update

[StephBretagne]

oh la la, je dois être fatiguée pour avoir laissé des bouts d'une autre requête dans celle-ci, la honte !!!

Mais le code ci-dessous ne marche pas non plus : ma table prixDocSerie n'est pas mise à jour

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
if(isset($_POST['nomPrix_fr']) && isset($_POST['nomPrix_en'])) {
	$nomPrix_fr=mysql_real_escape_string ($_POST ["nomPrix_fr"]);	 
	$nomPrix_en=mysql_real_escape_string ($_POST ["nomPrix_en"]);	   
 
	$sqlInsertPrix=mysql_query("INSERT INTO prixDocSerie (nomPrix_fr,nomPrix_en) VALUES (''.'".$nomPrix_fr."', '".$nomPrix_en."')");
}

[Fench]

C'est quoi la structure de ta table prixDocSerie ?

[StephBretagne]

comme indiqué dans mon premier message j'ai :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
-- --------------------------------------------------------
--
-- Structure de la table prixDocSerie 
--
CREATE TABLE prixDocSerie (
idPrix INT(4) NOT NULL AUTO_INCREMENT ,
nomPrix_fr VARCHAR (100),
nomPrix_en VARCHAR(100),
CONSTRAINT pk_prixDocSerie
	PRIMARY KEY (idPrix) 
) ENGINE=InnoDB  DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;

Edit : comme dit avant, à la place du INT, un smallint (4) unsigned serait suffisant.

[FirePrawn]

Pourquoi tu as un '.' avant la première variable ?

Cette écriture est correcte :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$sqlInsertPrix=mysql_query("INSERT INTO prixDocSerie (nomPrix_fr,nomPrix_en) VALUES ('".$nomPrix_fr."', '".$nomPrix_en."')");

[Fench]

comme indiqué dans mon premier message j'ai :

Oui j'avais pas scrollé sur la deuxième table c pas la fatigue, c la faim

Oui2 donc voir la correction de FirePrawn (en même temps: question: les balises CODEINLINE faut se les taper à chaque fois, ya pas de boutons comme CODE)

[StephBretagne]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$sqlInsertPrix=mysql_query("INSERT INTO prixDocSerie (nomPrix_fr,nomPrix_en) VALUES ('".$nomPrix_fr."', '".$nomPrix_en."')");

Bingo FirePrawn !!! Ce coup-ci, ma table prixDocSerie est bien complétée

Et maintenant pour l'update de documentaire ?

[Fench]

Suite:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
$lastIdPrixDS= mysql_insert_id($sqlInsertPrix);
$sqlInsertNoSerieDoc=mysql_query ("UPDATE documentaire SET noSerieDoc = '".$lastIdPrixDS."' WHERE idDoc = '".$idDocumentaire."'");