Discussion :

[Invité]

Hem... Je sais que j'abuse un peu...Mais c'est mon dernier post.

Voilà, j'insère mes données qui proviennent d'Ajax, de cette manière avec Php, dans la base de données mysql :

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
/*Function qui insère les notes : */
 function insert_evaluation() {  
 
/* Récupération des données POST provenant du Front End */
 $data = json_decode(file_get_contents("php://input"));    
 
 
/* SQL. */
$q = "INSERT INTO evaluations VALUES ('','".$data->idintervenant."','".$data->idresto."','".$data->noteglobale."','".$data->service."','".$data->ambiance."','".$data->attente."','".$data->caisses."','".$data->cuisines."','".$data->toilettes."')"; 
 
$qry = mysql_query($q);
 
 if ($qry) {
        $arr = array('msg' => "Impression enregistree avec succès!!!", 'error' => '');
        $jsn = json_encode($arr);
        // print_r($jsn);
    } else {
        $arr = array('msg' => "", 'error' => 'Erreur dans la mise à jour de l enregistrement');
        $jsn = json_encode($arr);
        // print_r($jsn);
    }
    exit();  
 
}

Comme vous pouvez le constater, c'est hyper lourd de réecrire le nom de chaque variable dans la requête SQL ...


Je me dit qu'il doit exister peut être une méthode pour écrire toutes les variables automatiquement dans l'insert, je ne sais pas.
Peut être connaissez vous un truc ?
Sinon je pense à une boucle for peut être ...

Le truc c'est qu'il ne doit pas y avoir d'erreur dans l'ordre ou l'on place les variables dans la requête INSERT.

Ce qui serait génial, si cela marchait, c'est que l'insert serait auto-évolutif en fonction des variables reçues par le front end... Je veux dire, au lieu qu'il y ait 10 variables reçues, il pourrait il y en avoir 12 et l'insert cela fonctionnerait encore...(Bien sur, il faudra modifier la table SQL avant, mais au moins, il ne faudra plus retoucher le code PHP)

je pense vraiment à une boucle FOR, mais bonjour, c'est dur !

[sabotage]

La bonne solution est d'utiliser une requête préparée PDO.
Déjà ça corrige ta faille de sécurité puisque ton script actuel ne contrôle pas les données reçus et ensuite ça te permet de construire dynamiquement ta requête avec autant de paramètre que tu veux.

[Invité]

Merci beaucoup, oui il n'y a pas de sécurité pour l'instant.
PDO ? Glup c'est hard à apprendre! Merci.

Dans mon script, apparemment la piste serait de faire une requête de ce genre avec un implode (Mais celle ci ne marche pas):

Code :

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

$q = "INSERT INTO evaluations VALUES ('','"'.implode('","', (array)$data) . '")";

ça mettrait toutes les variables $data directement dans la requête INSERT.

le premier champs est un id incrémental il doit être vide.
Bon ben j'essayerais avec PDO plus tard.

A mon sens, dans mon script, je dois faire une boucle for qui transfère toutes les variables dans un ARRAY, puis l'imploder dans la requête INSERT, mais je sais pas trop comment faire.
Et puis, c'est chaud si il y a des valeurs de type INT et des valeurs de type STRING, pas sur que cela fonctionne.

Ah j'ai une piste ici :http://stackoverflow.com/questions/1...mysql-database

et ici :

http://stackoverflow.com/questions/1...abase-with-php

ils ont des supers exemples par contre mon problème c'est que j'ai une première colonne vide vu que c'est incrémental

[sabotage]

PDO s'utilise quasiment pareil que l'extension mysql donc non ce n'est pas dur du tout.

Dans ton essai tu t'es emmelé dans les guillemets, ça se voit à la coloration :

Code :

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

$q = "INSERT INTO evaluations VALUES ('','" . implode("','", (array)$data) . "')";

[Invité]

Ca y est, ça marche presque j'ai écris ça :

Code :

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

1
2
3
4
5
6
 $data = json_decode(file_get_contents("php://input"), true);    
$columns = implode(", ",array_keys($data));
$escaped_values = array_map('mysql_real_escape_string', array_values($data));
$values  = implode(", ", $escaped_values);
var_dump($escaped_values);
$sql = "INSERT INTO evaluations VALUES ($values)";

Résultat, mon var_dump de $escaped_values m'affiche bien les bonnes valeurs à insérer dans ma requête SQL.

Mais mon dernier problème à rêgler est que j'ai une colonne vide à insérer dans $value en premier, ou dans la requête SQL en premiere valeur à insérer.

Il fallait écrire true à la première ligne pour spécifier que le json_decode doit retourner un array.

Pour que ça marche, il faudrait que je puisse faire un "PUSH()" dans $values au premier rang, mais un PUSH d'une valeur NULL ou VIDE !! C'est hard !

[sabotage]

En enlevant ce qui ne sert pas :

Code :

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

1
2
3
4
$data = json_decode(file_get_contents("php://input"), true);    
$escaped_values = array_map('mysql_real_escape_string', $data);
$values  = implode(", ", $escaped_values);
$sql = "INSERT INTO evaluations VALUES ('', $values)";