Performances de requêtes

Version imprimable

slt
j'ai une petite interrogation;

je fais la chose suivante pour charger ma base de donnée, je reçois un tableau multidimentionnel qui contient des invités liés a un organisateur, je lis donc ce tableau avec for each et a chaque boucle j'envois une requete avec le nouveaux contact:
Code:

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 public function setInvites($listeInvites) { foreach ($listeInvites as $objet => $element) { foreach ($element as $key => $value) { ($key == "_idFete")?$idFete=$value:$idFete = null ; ($key == "_nom")?$nom=$value: $nom = null ; ($key == "_prenom")?$prenom=$value:$prenom = null; ($key == "_email")?$email=$value:$email = null; ($key == "_idOR")?$idOR=$value:$idOR = null; $requete = "INSERT INTO `invites` (`idIN`, `nomIN`, `prenomIN`,`emailIN`,`idOR`,`idFete`) VALUES (NULL, '$nom', '$prenom','$email','$idOR','$idFete')"; $resultat = $this->db->query($requete); } } }
la question que je me pose, c'est de savoir si c'est correct au niveau performance pour ma base et si c'est très propre de codé comme ça.

je ne veut pas serialisé mon tableau pour pouvoir effectué des opération sur mes invités.

j'espere avoir était clair.

merci A+

29/07/2009, 13h16
Z3c33

Salut,
Jusqu'à preuve du contraire, je pense pas qu'il y ait de gros soucis dans ton code. De toute façon, quelles solutions tu voudrais adopter à la place ? Ca aurait été, je pense, plus problématique si tu avais une requête qui liait plusieurs tables. Dans ce cas là, il vaut mieux faire deux requêtes sur une table et de lier les deux grâce à PHP.
++
29/07/2009, 14h19
Petibidon

A première vue, ça doit pas vraiment fonctionner ce code, ou alors ça risque de te mettre un joli bordel dans ta DB.

à chaque $key tu éxécutes une requête d'insertion avec des éléments qui n'ont soit pas encore été récupérés, soit des éléments obsolètes.

pourquoi tiens tu à extraire le contenu du tableau avant de le passer à la requête ?
29/07/2009, 17h16
kaymak

Citation:

Envoyé par Petibidon

A première vue, ça doit pas vraiment fonctionner ce code, ou alors ça risque de te mettre un joli bordel dans ta DB.

à chaque $key tu éxécutes une requête d'insertion avec des éléments qui n'ont soit pas encore été récupérés, soit des éléments obsolètes.

pourquoi tiens tu à extraire le contenu du tableau avant de le passer à la requête ?

+1 ya une probleme, mais pour essayer de le reformuler c'est du au fait que lorsque tu itéres sur chaque propriété de $element, tu executes aussi une requete d'insertion.
Hors, ce n'est qu'après avoir itéré ton objet et attribué les variables que tu dois insérer.

M'enfin la quesiton se pose, pourquoi tu ne fais pas simplement $element["truc bidule"] ou $element->truc_bidule.... ? Tu ne dois pas connaitre la concaténation j'imagines ? http://www.php.net/

bye

performance de requete

slt
et désolé pour le code j'ai retapé vite fait , j'aurais du le recopiez(je vous le joint) , ce code fonctionne parfaitement , ma question portée sur la methode , la facon de faire , je me suis dit qu'il y avait peut être une autre solution moins gouteuse en performances que d'effectuer ces requetes a la queue leuleu.

merci pour vos conseils et encore 1000 excuses pour le code que j'ai posté ds mon 1er post.

Code:

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
 
public function setInvites($listeInvites)
	{
 
 
		   $idFete = 0;
		   $nom = "";
		   $prenom = "";
		   $email = "";
		   $idOR = 0;
 
 
		foreach ($listeInvites as $objet => $element)
		{
 
			foreach ($element as $key => $value)
			{
				if($key == "_idFete"){
 
					$idFete=$value;
 
				}elseif ($key == "_nom"){
 
					$nom=$value;
 
				}elseif ($key == "_prenom"){
 
					$prenom = $value;
 
				}elseif ($key == "_email"){
 
						$email = $value;	
 
				}elseif ($key == "_idOR"){
 
						$idOR = $value;
					}
 
			}
 
 
 
			$requete = "INSERT INTO `invites` (`idIN`, `nomIN`, `prenomIN`,`emailIN`,`idOR`,`idFete`) VALUES (NULL, '$nom', '$prenom','$email','$idOR','$idFete')";
 
			$resultat = $this->db->query($requete);
 
			if($resultat)
			{
				$resultat->closeCursor();
 
			}
 
		}
 
 
		return  $resultat;
 
 
		//$resultat = $this->db->query($requete);
 
		//return $resultat;
 
 
	}

merci A+

30/07/2009, 09h54
kaymak

slt

Pas de souci.

A la rigueur tu peux faire un insert en une fois, mais cela pose de nouveaux problèmes, notamment pour récupérer les nouvelles clés auto incrémenter.

Exemple :

Code:

INSERT INTO ma_table_b (colonne_a, colonne_c) VALUES ('valeur_a_1','valeur_b_1'), ('valeur_a_2','valeur_b_2'), ('valeur_a_3','valeur_b_3');

http://dev.mysql.com/doc/refman/5.0/fr/insert.html

a plus
30/07/2009, 10h37
andry.aime

Salut,
Il y a un benchmarck sur les if, switch et l'opérateur ternaire sur le site, je te conseille de le voir :
http://m-fernandez.developpez.com/ar...hp/bench/#LV-A
30/07/2009, 11h30
Petibidon
On peut imaginer que le tableau que tu passes en argument contient toujours les mêmes clés ?

Dans ce cas le bloc foreach( $element as $key => $value ) est absolument inutile.

tu affecte tes variables à ta requete tout simplement par des concaténations :
Code:

1 2 $req = 'INSERT INTO table (champ1, champ2 ) VALUES ( "' . $element['champ1'] . '" , "' . $element['champ2'] . '" )';
Plus poussé, si tu es un gros fainéant comme moi :

Code:

$req = vsprintf( 'INSERT INTO table (champ1, champ2 ) VALUES ( "%s", "%s" )' , $element );

Attention cependant à être sûr que les valeurs dans l'array $element sont dans le même ordre que les champs de la table sinon faut corriger l'ordre d'affectation du vsprintf (voir notice).
30/07/2009, 13h17
bruno.rotrou

concatenation et performance

slt
merci a tous pour vos lumières tjrs aussi précieuses.

a bientot