Discussion :

[Fluckysan]

Bonjour !
J'ai un chti souci avec un clone placé dans une boucle
J'aimerais dans chaque tour de boucle, cloner un objet, initialiser ses attributs avec des valeurs et le pusher dans tableau
Or, à la sortie de la fonction mon tableau est initialiser avec la même instance dans toute ses cases ...

Voici mon bout de code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
			if($stmt->execute()){
				foreach($stmt->fetchall(PDO::FETCH_ASSOC) as $rowRes){
					$oTemp = clone $this;
					$oTemp->raz();
 
					foreach($oTemp->column as $key => $column)
						$column->setNull(true);
 
					foreach($rowRes as $key => $rowValue)
						$oTemp->column[$key]->setValue($rowValue);
 
					$aReturn[] = $oTemp;
				}
			}

J'ai ensuite pensé que ça venait du fait que j'utilisais une même var ($oTemp)
J'ai changé mon code pour tester autre chose :

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
			if($stmt->execute()){
				$nI = 0;
 
				foreach($stmt->fetchall(PDO::FETCH_ASSOC) as $rowRes){
					$aReturn[$nI] = clone $this;
					$aReturn[$nI]->raz();
 
					foreach($aReturn[$nI]->column as $key => $column)
						$column->setNull(true);
 
					foreach($rowRes as $key => $rowValue)
						$aReturn[$nI]->column[$key]->setValue($rowValue);
 
					$nI++;
				}
			}

Mais le résultat est le même ...
Je viens à vous pour trouver une solution !
Des clones vi mais pô si ... cloné XD

Merchi d'avance !

[kaymak]

salut,

peux tu nous indiquer comment tu fais pour déterminer si les variables du tableau sont clonées ou non.

bye

[Fluckysan]

Bonjour Kaymak et merchi de porter un intérêt pour mon souci
Pour vérifier ce qui cloche dans le clonage je fais un var_dump() de mon tableau $aReturn à la sortie du if et les instances sont toutes les mêmes dedans ...

Si ça peut aider, si je fais un var_dump($aReturn[$nI]); pendant la boucle, le clone est bon (attribut de l'objet bien initialisé)

J'ai réécris mon code pour plus de clarté :

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
if($stmt->execute()){
	$nI = 0;
 
	foreach($stmt->fetchall(PDO::FETCH_ASSOC) as $rowRes){
		$aReturn[$nI] = clone $this;
		$aReturn[$nI]->raz();
 
		foreach($aReturn[$nI]->column as $key => $column){
			$column->setNull(true);
			$column->setValue($rowRes[$key]);
		}
 
		$nI++;
	}
}

[kaymak]

Salut,

pas de souci. Pour l'exemple, le plus top, aurait été d'avoir un exemple fonctionnel que je puisse tester.
Mais bon ce n'est pas grave, je pense avoir saisit le souci ! Du moins j'espère.

En faits, si j'ai bien compris ton code, $this, est une instance d'un objet utilisé pour faire ton mapping bd->objet.
Pour plus de simplicité j'appellerais ces objets des DBMap.

Bref, la classe DBMap expose des colonnes , comme dans ta base, pour attribuer, récupérer les valeurs par colonne.
Et c'est là que le problème apparaît je pense.
En effet lorsque tu clones ton objet, ce n'est pas un attribut de type primitif (int, float, string ect) qui est dupliqué, ce sont des objets colonnes.
Hors, les objets colonnes par défaut, ne sont pas clonés lorsque tu clones, ils sont référencés !
Et comme ils sont référencés, modifiés la valeur de la première colonne de l'instance initiale, ou de la dernière, modifie tous les autres clones !!

Donc, si j'ai visé juste. Il te suffit de ré implémenter __clone dans DPMap pour cloner les colonnes !! Et non laisser faire PHP qui réalisera un référencement par défaut.

Qu'en penses tu ?

En tout cas voici un exemple, fonctionnel, et démonstratif du problème que je pressens :

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
65
66
67
68
69
70
<?
/*
** Exemple 1, sans clone, utilisation des références par défaut
*/
class A{
	public $c;
	public $b_obj;
}
class B{
	public $f;
}
 
$B = new B();
$B->f = "pwet pwet";
 
 
$t = new A();
$t->c = "une valeur initiale";
$t->b_obj = $B;
 
$t_ref = clone($t);
 
 
 
$t->b_obj->f = "pwet pwet je suis différent partout car je suis référencé !!";
 
echo "<pre>";
var_dump($t);
var_dump($t_ref);
echo "</pre>";
 
 
 
/*
** Exemple 2, avec clone, ipmplémentation individuel de __clone pour A qui permet de bien déréférencer les objets internes.
*/
class A_clone{
	public $c;
	public $b_obj;
 
	public function __clone(){
		$this->b_obj = clone($this->b_obj );
	}
}
 
$Bb = new B();
$Bb->f = "pwet pwet";
 
 
$Tt = new A_clone();
$Tt->c = "une valeur initiale";
$Tt->b_obj = $Bb;
 
$Tt_ref = clone($Tt);
 
 
 
$Tt->b_obj->f = "pwet pwet je NE suis PAS différent partout car je NE suis PAS référencé !!";
 
echo "<pre>";
var_dump($Tt);
var_dump($Tt_ref);
echo "</pre>";
 
 
 
 
 
 
?>

a plus

[Fluckysan]

En effet Kaymak c'est tout à fait ce que je fais
Bravo pour avoir tout compris avec ce chti bout de code et merchi beaucoup pour l'explication, ça me tracasse depuis 2 jours lol
Bref si je clone plutôt que de faire un new c'est que j'ai un appel à la BDD dans mon constructeur donc imagine si j'ai 3K enregistrement ... De plus je cherche à avoir le comportement du clonage donc bon XD

Ton exemple sort bien le même comportement que j'ai
Good ça explique le pourquoi

Je vais surchager __clone et je te tiens au courant !

[kaymak]

..Bref si je clone plutôt que de faire un new c'est que j'ai un appel à la BDD dans mon constructeur donc imagine si j'ai 3K enregistrement ..

Ah oui ! ca c'est bad :/ Très très mauvais en fait, selon moi.

Mais bon ce n'est que mon avis, et il est peu suivit, alors je me le garde

J'espère que tu résoudra ton pb

a plus

[Fluckysan]

Oki ça résout mon prob
Par contre je suis intéressé par ton avis si tu veux on peut continuer cette discution par MP