Discussion :

[bruno.rotrou]

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 :

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
 
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+

[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.
++

[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 ?

[kaymak]

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

[bruno.rotrou]

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 :

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
 
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+

[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 :

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

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