foreach et requête SQL

Version imprimable

Bonjour,

J'ai un tableau jquery avec la possibilité de rajouter X lignes au click d'un bouton.
Une fois ces X lignes renseignées, je voudrai les enregistrer en BDD
J'ai réussi à faire quelque chose mais qui ne me convient pas vraiment car ça nécessite autant de requettes qu'il y a de lignes.

Voici mes codes.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
<input type="button" value="Ajouter une ligne" onclick="NouvelleLigne('table_fourn')">
<input type="button" value="Supprimer la ligne" onclick="SupprLigne('table_fourn')">
 
<FORM name="tabl_test" method="post" action="bdd.php">
    <table id="table_fourn" style="border:solid 1px black">
        <tbody>
			<tr>
				<td><input type="checkbox" name="suppr_lign[]" ></td>
				<td><input type="text" name="mode_reglement[]" ></td>
				<td><input type="text" name="ref_reglement[]" ></td>
				<td><input type="text" name="paiement_fournisseur[]"></td>
			</tr>
		</tbody>
	</table>
	<input type="submit" value="Valider" name="valider">
</FORM>

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 	$m_r = $_POST['mode_reglement'];
	$r_r = $_POST['ref_reglement']; 
	$p_f = $_POST['paiement_fournisseur'];
 
foreach($m_r as $a => $b){
	echo $m_r[$a] ." - ". $r_r[$a] ." - ". $p_f[$a] ."<br>";
 
 		$insert = Cnx::connectCnx()->prepare("INSERT INTO fournisseurs_commandes (mode_reglement, ref_reglement, paiement_fournisseur) VALUES (".$m_r[$a]." , ". $r_r[$a] ." , ". $p_f[$a]	.") ");
		$insert -> bindValue(':mode_reglement', $m_r[$a]);
		$insert -> bindValue(':ref_reglement', $r_r[$a]);
		$insert -> bindValue(':paiement_fournisseur', $p_f[$a]);
		$insert -> execute(); 
		echo  "ok"; 
	  }

Si quelqu'un a une idée?

10/01/2015, 17h14
sabotage

C'est normal d'avoir une requête par insertion.
Les requêtes préparées sont d'ailleurs faites pour ça.
Par contre dans ta requête préparées tu as mis les valeurs au lieu des paramètres.
10/01/2015, 17h29
jimmo

Salut Sabotage
J'ai changé bindValue par bindParam.
Mais si mon tableau fait 100 lignes, je vais avoir 100 requêtes....
N'y a t il pas une possibilité de faire autrement???
par exemple de boucler uniquement sur les valeurs et ensuite exécuter le tout?
10/01/2015, 20h04
sabotage

Ce n'est pas un problème de bindValue/bindParam, c'est dans ta requête qu'il manque les paramètres nommés ; d'ailleurs c'est étrange qu'il n'y ait pas d'erreur.
Quel est ton problème avec le fait d'executer 100 requêtes ? Comme je te dis, les requêtes paramétrées sont faites pour ça, seules les valeurs changent à chaque execution.
10/01/2015, 23h01
bhamp0

En PDO : http://php.net/manual/fr/pdo.commit.php

Step by step :
1. On commence une transaction.
2. On prépare une requête.
3. Pour chaque donnée à insérer, on fait une exécution avec les bons paramètres.
4. On committe la transaction globale.

L'exemple 1 sur la page PHP.net indique exactement ce principe-là.
11/01/2015, 13h17
sabotage

Les transactions servent à garantir que toutes les requêtes sont bien exécutées ou à tout annuler.
Je ne pense pas que ce soit ça qui soit cherché ici.

Merci de votre participation.

Citation:

Envoyé par sabotage

Par contre dans ta requête préparées tu as mis les valeurs au lieu des paramètres.

Je ne comprend ça que tu veux dire. Ou me suis je planté?

Et concernant le nombre important de requêtes, il me semble que ça va consommer plus de bande passante et ralentir l’insertion en BDD.
Mais peut être que je me plante complètement. Juste des question que je me pose et vous pose :)
Code:

1 2 3 4 5 6 7 8 $insert = Cnx::connectCnx()->prepare("INSERT INTO fournisseurs_commandes (mode_reglement, ref_reglement, paiement_fournisseur) VALUES (:mode_reglement, :ref_reglement, :paiement_fournisseur) "); foreach($m_r as $a => $b){ $insert -> bindValue(':mode_reglement', $m_r[$a]); $insert -> bindValue(':ref_reglement', $r_r[$a]); $insert -> bindValue(':paiement_fournisseur', $p_f[$a]); $insert -> execute(); echo "ok"; }
Est ce mieux?