Envoie des images dans la base de données

**MN_jacques** · 20/03/2019, 09h36

Bonjour à tous, j'ai un petit probleme voici mon code php:

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
 
/****ENVOI IMAGES DANS LE DOSSIER******************/
$workid = $_GET['id'];
//$work_id = "3602";
//var_dump($workid);
//var_dump($work_id);
 if (isset($_FILES['images'])) {
    $manager = new ImageManager();
	$images = $_FILES['images']['tmp_name'];
	 	 foreach($images as $image){
	 	 	$db ->query("INSERT INTO images SET work_id=$workid"); 
			$image_id = $db ->lastInsertId();
			$image_name =  $image_id . '.jpg';
			$image_name = $db->quote($image_name);
			$db ->query("UPDATE images SET name=$image_name WHERE id=$image_id");
   	        $manager->make($image)->orientate()->fit(150,100)
   	        ->save('../img/tmp_img/' .$image_id . '.jpg');
			}         
          }

Quand j’exécute ce code avec l'initialisation directe de ma variable workid voir le code en commentaire: $workid = "3602"; l'insertion dans la BDD $db ->query("INSERT INTO images SET work_id=$workid"); passe bien mais si j’essaie de l’exciter avec le workid récupéré dans ma variable GET, $workid = $_GET['id']; ce qui est mon but Rien ne marche et pourtant le $_GET['id'] contient belle et bien l'id de la réalisation, quand j’essaie de vérifier le deux variables celle initialisée et celle récupérée var_dump($workid); et var_dump($work_id);
ça donne ça:

Code :

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

1
2
3
C:\wamp\www\site\users\work_edit.php:60:string '3602' (length=4)
 
C:\wamp\www\site\users\work_edit.php:61:string '3602' (length=4)

donc ça donne exactement la même chose, je ne comprend vraiment pourquoi avec le $workid = $_GET['id']; indique 0, votre aide svp

Invité · 20/03/2019, 11h42

Bonjour,

1- il faudrait déjà utiliser la bonne syntaxe pour l'INSERT...

2-

Il faut utiliser des requêtes préparées !

**MN_jacques** · 20/03/2019, 12h09

Merci pour votre réaction voici je viens de changer mon code avec des requettes préparées:

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
 
$workid = $_GET['id'];
 if (isset($_FILES['images'])) {
    $manager = new ImageManager();
    $images = $_FILES['images']['tmp_name'];
         foreach($images as $image){
            $req = $db->prepare("INSERT INTO images SET work_id= ?");
            $req->execute([$workid]);
            $image_id = $db ->lastInsertId();
            $image_name =  $image_id . '.jpg';
            $image_name = $db->quote($image_name);
            $req = $db->prepare("UPDATE images SET name=$image_name WHERE id= ?");
            $req->execute([$image_id]);
            $manager->make($image)->orientate()->fit(150,100)
            ->save('../img/tmp_img/' .$image_id . '.jpg');
            }         
          }

Mais l'insertion ne se fais toujours pas il y'a t-il moyen de peut être convertir le *$_GET['id'] * du string en interger? psk je crois que le problème vient de là, quand j'essai de faire

Code :

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

1
2
$work_id = $db-> $_GET['id'];
$workid = (int) $work_id;

donc essayé de convertir la valeur que je recois de *$_GET['id'] * en un entier
là les images sont bien enregistrées dans la BDD mais avec un faut workid qui est de 0; votre aide svp

**andry.aime** · 20/03/2019, 12h19

Bonjour,

La syntaxe de la requête insert n'est pas bonne.
Utilise une seule requête pour les deux colonnes au lieu de faire un update pour la seconde.

Code :

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

insert into images(work_id, name) values (? , ?)

A+.

**MN_jacques** · 20/03/2019, 12h48

Merci bien pour votre réaction je crois que le problème n'est pas au niveau de la requette sql, cette dernière marche parfaitement le problème réside à la valeur de ma variable $workid = $_GET['id']. Quand j’essaie d'initialiser cette variable avec l'id de l'un de work par exemple: $work_id = "3602"; tout marche parfaitement mais quand je l'initialise avec l'id récupéré de l'url $workid = $_GET['id'] l'insertion ne marche pas.
et pourtant l'id est bien récupéré de l'url et après un var_dump il affiche bien l'id du work, alors je ne comprend pas le pourquoi.
Merci bien encore

**andry.aime** · 20/03/2019, 13h04

insert ... set est seulement valable que pour mysql mais c'est pas standard. Essaie d'afficher l'erreur.

A+.

Invité · 20/03/2019, 16h03

Code sql :

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

            $req = $db->prepare("INSERT INTO images SET work_id= ?");

Cette syntaxe est FAUSSE !

La BONNE syntaxe :

Code sql :

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

            $req = $db->prepare("INSERT INTO images (work_id) VALUES (?)");

2- De même :

Code :

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

1
2
            $req = $db->prepare("UPDATE images SET name=$image_name WHERE id= ?");
            $req->execute([$image_id]);

Tant qu'à faire une requête préparée, fais-le correctement :

Code :

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

1
2
            $req = $db->prepare("UPDATE images SET name=? WHERE id=?");
            $req->execute([$image_name, $image_id]);

3- Enfin (?), la préparation des requête peut/doit se faire UNE SEULE FOIS, en mettant les 2 $db->prepare(...) AVANT la boucle foreach().

En prenant soin de distinguer les 2 :

$req_insert = $db->prepare(...);,
$req_update = $db->prepare(...);,
et en modifiant les exécutions (qui restent DANS la boucle) en conséquence.

**andry.aime** · 21/03/2019, 07h17

Bonjour,

Envoyé par jreaux62

Code sql :

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

            $req = $db->prepare("INSERT INTO images SET work_id= ?");

Cette syntaxe est FAUSSE !

Cette syntaxe est valable pour mysql (seulement), mais c'est pas un SQL standard. Après, je ne sais pas pourquoi il insiste à utiliser une telle syntaxe.

A+.

**MN_jacques** · 21/03/2019, 10h32

Merci beaucoup pour toutes ces corrections j'ai vraiment beaucoup appris durant ces échanges, voici j'ai tout modifié selon vos remarquent et voici mon nouveau code et ça passe bien, mais mon problème là persiste toujours:

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
 
//$work_id = $db->quote($_GET['id']);
//$workid = (int) $work_id;
//$workid = intval($_GET['id']);
//$workid = '3602';
$workid = $_GET['id'];
//var_dump($_GET['id']);
//var_dump($workid);exit;
 if (isset($_FILES['images'])) {
    $manager = new ImageManager();
	$images = $_FILES['images']['tmp_name'];
	$req_insert = $db->prepare("INSERT INTO images (work_id) VALUES (?)");
	$req_update = $db->prepare("UPDATE images SET name=? WHERE id=?");
	 	 foreach($images as $image){
		    $req_insert->execute([$workid]);
		    $image_id = $db ->lastInsertId();
		    $image_name =  $image_id . '.jpg';
			$image_name = $db->quote($image_name);
		    $req_update->execute([$image_name, $image_id]);
			$manager->make($image)->orientate()->fit(150,100)
   	        ->save('../img/tmp_img/' .$image_id . '.jpg');
			}         
          }

Invité · 21/03/2019, 10h56

Bonjour,

Dans la table de la base de données, la colonne "work_id" est bien en INT ?

Car si elle est en VARCHAR, ça peut expliquer le problème. (quoi que... )

Teste avec :

Code :

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