Discussion :

[Tehko]

Bonjour,

Je souhaite créer deux objets de même type, initialisés tous les deux par un troisième objet de même type en paramètre. Voici le code de mon constructeur:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	private SupplyChain sc;
	private SupplyChain sc2;
	private int demand;
	private int fluct_demand;	
 
	public SimulatedScenario(SupplyChain sc)
	{
		this.sc = new SupplyChain();
		this.sc2 = new SupplyChain();
		this.sc = sc;
		this.sc2 = sc;		
		demand = (Integer) sc.getSegmentoMercato().getDomanda();
	}

Ensuite, je souhaite modifier un attribut par l'appel d'un "setter" sur l'un des deux objets.

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
public void computeVarOptimumMarket(int market_state, int[] coalition){
                ...
                //compute current optimum point
		current_optimum = new OttimoCosti(sc, coalition);
		current_optimum.calcolaOttimo();
		current_optimum.calcolaOttimiTotali();
		
		//sc2 = sc with new demand
		sc2 = sc;
		sc2.getSegmentoMercato().setDomanda(fluct_demand);
		//compute new_optimum point for new demand
		new_optimum = new OttimoCosti(sc2, coalition);
		new_optimum.calcolaOttimo();
		new_optimum.calcolaOttimiTotali();
                ...
}

Or, je me rends compte que la valeur en question est aussi modifiée sur l'autre objet de même type, chose que je ne veux pas.
J'ai remarqué que les ID de mes deux objets changeaient lorsque que j'écris

Code :

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

1
2
		this.sc = sc;
		this.sc2 = sc;

et prennent tous deux le même ID que l'objet en paramètre.

Comment puis-je contourner ce problème ? L'objectif étant s'initialiser les deux objets sc et sc2 avec les mêmes valeurs que l'objet en paramètre pour ensuite changer certaines valeurs de l'objet sc2 sans modifier celles de sc.

Cordialement,
Tehko

[wax78]

Il est logique que cela se comporte ainsi puisque sc et sc2 sont tout les 2 le meme objet lorsque tu fais ceci :

Code :

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

1
2
this.sc = sc;
this.sc2 = sc;

Pourquoi fait tu cela en fait, c'est cela qui pose problème.

[Hizin]

Code java :

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

1
2
3
4
this.sc = new SupplyChain(); // création d'un objet SupplyChain et mise dans la variable sc
this.sc2 = new SupplyChain(); // création d'un objet SupplyChain et mise dans la variable sc2
this.sc = sc; // attribution de sc à sc ... inutile (et sans doute enlevé à la compilation). sc vaut sc, il est inutile de tenter de l'attribuer à lui-même
this.sc2 = sc; // attribution de sc à sc2, donc sc et sc2 sont maintenant 2 variables pointant vers le même objet et l'objet anciennement référencé par sc2 n'existe plus

Ce bout de code est équivalent à :

Code java :

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

sc = sc2 = new SupplyChain();

[thelvin]

Tu as besoin de créer ton propre "constructeur de copie," cette notion du C++ qu'on peut imiter en Java ainsi :

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
public class SupplyChain {
 
  int champ1;
  int champ2;
  // ...
 
 
  /** Construit un SupplyChain initialisé au même contenu qu'un autre. */
  public SupplyChain(SupplyChain original) {
    champ1 = original.champ1;
    champ2 = original.champ2;
    // ...
  }
 
}

[Tehko]

Wax78, je pensais qu'en faisant cela je pourrais récupérer les valeurs de l'objet en paramètre dans les deux autres objets.

Hizin, c'est bien ce que j'ai remarqué. Les deux objets pointent vers l'objet en paramètre alors que je veux qu'ils copient seulement les variables contenues dans l'objet en paramètre.

Tu as besoin de créer ton propre "constructeur de copie," cette notion du C++ qu'on peut imiter en Java ainsi :

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
public class SupplyChain {
 
  int champ1;
  int champ2;
  // ...
 
 
  /** Construit un SupplyChain initialisé au même contenu qu'un autre. */
  public SupplyChain(SupplyChain original) {
    champ1 = original.champ1;
    champ2 = original.champ2;
    // ...
  }
 
}

Thelvin, j'ai donc suivi ton conseil et écris le constructeur de copie:

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
public class SupplyChain {
	private SegmentoMercato segmentoMercatoSC;
	private ArrayList<Vector> AND_OR = new ArrayList<Vector>();
 
	/**
         * Constructor allowing copy of an original SupplyChain
         * @param original
         */
	public SupplyChain(SupplyChain original){
		this.AND_OR = original.AND_OR;
		.  .  .
                .  .  .
                .  .  .
		this.segmentoMercatoSC = original.segmentoMercatoSC;
	}

Les objets n'ont donc plus le même ID mais en ce qui concerne les variables (ex: segmentoMercatoSC), chaque variable de sc pointe vers les mêmes ID que celle de sc2.
Y a-t-il quelque chose que j'ai mal fait ? Ou mal compris ?

Voici l'implémentation au cas où...:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public class SimulatedScenario {
	private SupplyChain sc;
	private SupplyChain sc2;
        public SimulatedScenario(SupplyChain sc)
	{
		this.sc = new SupplyChain(sc);
		this.sc2 = new SupplyChain(sc);
                ...
         }
...
}

[wax78]

Utlise des clones a la place des = dans ton constructeur de copie en l'implementant convenable dans les classes necessaire.

Pour l'arraylist aussi,il faut en recréer une et soit remettre les elements soit les cloner aussi dans la nouvelle arraylist.