requêtes sql multiples

**allhambra** · 29/04/2014, 18h25

Bonjour les codeurs

.
j'ai un formulaire basique dont je souhaite ventiler les résultats dans deux tables.
J'ai donc fait deux requêtes dans ma page de traitement, la première fonctionne sans soucis mais la seconde me mets une erreur sql et l'insert ne se fait pas.
voici ma page :

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
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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
<?php require_once('../Connections/config.php'); ?>
<?php
 
// on crée nos variables, et au passage on retire les éventuels espaces	
if (isset($_POST['slide1_titre'])){
		$slide1_titre = ($_POST['slide1_titre']);
	}
if (isset($_POST['slide1_text'])){
		$slide1_text = ($_POST['slide1_text']);
	}
if (isset($_POST['slide2_titre'])){
		$slide2_titre = ($_POST['slide2_titre']);
	}
if (isset($_POST['slide2_text'])){
		$slide2_text = ($_POST['slide2_text']);
	}
if (isset($_POST['slide3_titre'])){
		$slide3_titre = ($_POST['slide3_titre']);
	}
if (isset($_POST['slide3_text'])){
		$slide3_text = ($_POST['slide3_text']);
	}
 
 
	$chemin = "../uploads/" ;
 
		if(!empty($_FILES["logo"]["name"]))
{
	//nom du fichier choisi:
	$nomFichier    = $_FILES["logo"]["name"] ;
	//nom temporaire sur le serveur:
	$nomTemporaire = $_FILES["logo"]["tmp_name"] ;
	//type du fichier choisi:
	$typeFichier   = $_FILES["logo"]["type"] ;
	//poids en octets du fichier choisit:
	$poidsFichier  = $_FILES["logo"]["size"] ;
	//code de l'erreur si jamais il y en a une:
	$codeErreur    = $_FILES["logo"]["error"] ;
 
 
 
	if(copy($nomTemporaire, $chemin.$nomFichier))
		//echo("<p>l'upload du logo a réussi</p>") ;
		echo("<p>Upload du logo : ". OK."</p>");
	else
		echo("<p>l'upload du logo1 a échoué</p>") ;
}//fin if
else
{
	echo("Vous n'avez pas choisit de fichier!!". RETOUR);
	//echo("<a href=\"./biens.php\">Retour</a><br>") ;
}//fin else
 
	if(!empty($_FILES["image"]["name"]))
{
	//nom du fichier choisi:
	$nomFichier2    = $_FILES["image"]["name"] ;
	//nom temporaire sur le serveur:
	$nomTemporaire = $_FILES["image"]["tmp_name"] ;
	//type du fichier choisi:
	$typeFichier   = $_FILES["image"]["type"] ;
	//poids en octets du fichier choisit:
	$poidsFichier  = $_FILES["image"]["size"] ;
	//code de l'erreur si jamais il y en a une:
	$codeErreur    = $_FILES["image"]["error"] ;
 
 
 
	if(copy($nomTemporaire, $chemin.$nomFichier2))
		//echo("<p>l'upload de l'image principale a réussi</p>") ;
		echo '<p>upload de l\'image principale : '. OK.'</p>';
	else
		echo("<p>l'upload de l'image principale a échoué</p>") ;
}//fin if
else
{
	echo("Vous n'avez pas choisit de fichier!!". RETOUR);
	//echo("<a href=\"./biens.php\">Retour</a><br>") ;
}//fin else
 
	if(!empty($_FILES["favicon"]["name"]))
{
	//nom du fichier choisi:
	$favicon    = $_FILES["favicon"]["name"] ;
	//nom temporaire sur le serveur:
	$nomTemporaire = $_FILES["favicon"]["tmp_name"] ;
	//type du fichier choisi:
	$typeFichier   = $_FILES["favicon"]["type"] ;
	//poids en octets du fichier choisit:
	$poidsFichier  = $_FILES["favicon"]["size"] ;
	//code de l'erreur si jamais il y en a une:
	$codeErreur    = $_FILES["favicon"]["error"] ;
 
 
 
	if(copy($nomTemporaire, $chemin.$favicon))
		//echo("<p>l'upload de l'image principale a réussi</p>") ;
		echo '<p>upload du favicon : '. OK.'</p>';
	else
		echo("<p>l'upload de l'image principale a échoué</p>") ;
}//fin if
else
{
	echo("Vous n'avez pas choisit de fichier!!". RETOUR);
	//echo("<a href=\"./biens.php\">Retour</a><br>") ;
}//fin else
	if(!empty($_FILES["slide1"]["name"]))
{
	//nom du fichier choisi:
	$slide1    = $_FILES["slide1"]["name"] ;
	//nom temporaire sur le serveur:
	$nomTemporaire = $_FILES["slide1"]["tmp_name"] ;
	//type du fichier choisi:
	$typeFichier   = $_FILES["slide1"]["type"] ;
	//poids en octets du fichier choisit:
	$poidsFichier  = $_FILES["slide1"]["size"] ;
	//code de l'erreur si jamais il y en a une:
	$codeErreur    = $_FILES["slide1"]["error"] ;
 
 
 
	if(copy($nomTemporaire, $chemin.$slide1))
		//echo("<p>l'upload de l'image principale a réussi</p>") ;
		echo '<p>upload du slide 1 : '. OK.'</p>';
	else
		echo("<p>l'upload du slide 1 a échoué</p>") ;
}//fin if
else
{
	echo("Vous n'avez pas choisit de fichier!!". RETOUR);
	//echo("<a href=\"./biens.php\">Retour</a><br>") ;
}//fin else
 
if(!empty($_FILES["slide2"]["name"]))
{
	//nom du fichier choisi:
	$slide2    = $_FILES["slide2"]["name"] ;
	//nom temporaire sur le serveur:
	$nomTemporaire = $_FILES["slide2"]["tmp_name"] ;
	//type du fichier choisi:
	$typeFichier   = $_FILES["slide2"]["type"] ;
	//poids en octets du fichier choisit:
	$poidsFichier  = $_FILES["slide2"]["size"] ;
	//code de l'erreur si jamais il y en a une:
	$codeErreur    = $_FILES["slide2"]["error"] ;
 
 
 
	if(copy($nomTemporaire, $chemin.$slide2))
		//echo("<p>l'upload de l'image principale a réussi</p>") ;
		echo '<p>upload du slide 2 : '. OK.'</p>';
	else
		echo("<p>l'upload du slide 2 a échoué</p>") ;
}//fin if
else
{
	echo("Vous n'avez pas choisit de fichier!!". RETOUR);
	//echo("<a href=\"./biens.php\">Retour</a><br>") ;
}//fin else
 
if(!empty($_FILES["slide3"]["name"]))
{
	//nom du fichier choisi:
	$slide3    = $_FILES["slide3"]["name"] ;
	//nom temporaire sur le serveur:
	$nomTemporaire = $_FILES["slide3"]["tmp_name"] ;
	//type du fichier choisi:
	$typeFichier   = $_FILES["slide3"]["type"] ;
	//poids en octets du fichier choisit:
	$poidsFichier  = $_FILES["slide3"]["size"] ;
	//code de l'erreur si jamais il y en a une:
	$codeErreur    = $_FILES["slide3"]["error"] ;
 
 
 
	if(copy($nomTemporaire, $chemin.$slide3))
		//echo("<p>l'upload de l'image principale a réussi</p>") ;
		echo '<p>upload du slide 3 : '. OK.'</p>';
	else
		echo("<p>l'upload du slide 3 a échoué</p>") ;
}//fin if
else
{
	echo("Vous n'avez pas choisit de fichier!!". RETOUR);
	//echo("<a href=\"./biens.php\">Retour</a><br>") ;
}//fin else
 
 
mysql_select_db($database_config, $config);
$req1 = "UPDATE site SET logo='$nomFichier', image='$nomFichier2', favicon='$favicon' WHERE id=1";
$req2 = "INSERT INTO slides VALUES
('', '".$slide1_titre."', '".$slide1_text."', '".$slide1."'),
('', '".$slide2_titre."', '".$slide2_text."', '".$slide2."'),
('', '".$slide3_titre."', '".$slide3_text."', '".$slide3."')
";
mysql_query($req1) or die('Erreur SQL requete update !'.$sql.'<br>'.mysql_error());
mysql_query($req2) or die('Erreur SQL requete insert !'.$sql.'<br>'.mysql_error());
 
echo 'Installation : '. OK;	
 
print("<script type=\"text/javascript\">setTimeout('location=(\"etape5.php\")' ,1000);</script>");
 
?>

l'erreur générée :

Upload du logo : OK

upload de l'image principale : OK

upload du favicon : OK

upload du slide 1 : OK

upload du slide 2 : OK

upload du slide 3 : OK

Erreur SQL requête insert !
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'entreprise', 'slide_1.jpg'), ('', 'Conception', 'sites
et applications we' at line 2

toute aide est bienvenue, a force de bloquer la dessus, je n'ai plus les yeux en face des trous

**sabotage** · 29/04/2014, 19h05

Il faut echapper tes valeurs :

Code :

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

$slide1_titre = mysql_real_escape_string($_POST['slide1_titre']);

**ABCIWEB** · 29/04/2014, 19h49

Peut-être une rupture de chaine. Fais echo $req2 pour voir si ta chaine de requête est bien formée avec les bonnes variables attendues.

Ensuite la logique de ton code laisse à désirer. Si aucun fichier n'est téléchargé par exemple pour $_FILES["slide1"] alors la variable $slide1 n'est pas définie. Et ce n'est pas parce que tu fais un echo que le script s'arrête, donc la requête est construite avec des variables indéfinies (ce que php n'aime pas beaucoup).

Ensuite les scripts d'upload sont très incomplets.
- Ils ne gèrent pas le dépassement "post_max_size" du serveur et dans ce cas tu auras une page blanche car $_POST et $_FILES seront vides.
- C'est bien de faire afficher un message d'erreur mais c'est mieux si cela peut aider. Genre "l'upload de l'image principale a échoué" oui mais encore, le fichier est-il trop lourd (dépassement max_file_size du serveur), mauvaise extension, problème d'upload ? Si tu ne précise rien il est difficile de savoir quoi faire en retour.
- Je ne vois pas que tu contrôle l'extension des fichiers, c'est très imprudent.
- Je ne vois pas que tu corrige le nom des fichiers s'il est incompatible avec une adresse serveur. Tu pourras te retrouver avec des images inexploitables simplement parce que leur nom initial est incorrect.

Si tu veux tu peux essayer cette classe d'upload Elle est fournie avec un dossier de tests "Tests-Scripts-Upload" prêt à l'emploi. Et dans le dossier "Exemples" tu pourras tester le fichier nommé "Formulaire_inscription_complet_affichage_vignette.php" qui ressemble beaucoup à tes besoins, avec possibilité de rendre indispensable l'upload d'une image pour enregistrer le formulaire. L'avantage c'est que toutes les erreurs sont gérées et qu'il y a de nombreuses options. Eventuellement si tu veux l'utiliser je pourrais t'aider un peu (après que tu aies chercher à comprendre de ton côté et défini clairement les différents cas de figure de ton script).

requêtes sql multiples

PHP & Base de données

Discussions similaires

Partager

Partager