Discussion :

[didio]

Bonjour,

Je rencontre un soucis que je ne comprend pas : J'utilise deux objets de même type à un moment donné.
Lorsque j'utilise une méthode d'un objet afin de modifier certains attributs il se trouve que l'autre objet est aussi "impacté" :

Code :

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

1
2
3
4
 
CmbdSystem popi = xmlSystem.subsystems[0].system;
mbdSystem.GetRunnable();
// CmbdSystem popi ; popi a été modifé

Mon objet xmlSystem est également modifié après modification de mon objet mbdSystem.

Voici ce que fait la méthode GetRunnable :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public  void GetRunnable() 
        {
            foreach (CmbdSubSystem sub in subsystems)
            {
                CmbdSystem system = FindSystem(sub);
                if (system.IsaRunnable())
                {
                    runnables.Add(system);
                }
            }
        }

D'où ça peut venir ?
Merci

[alexxxx69]

Salut,

Sans savoir comment tu attribut ton objet "mbdSystem" c'est dur de t'apporter une réponse mais si quand tu modifies une instance d'un objet, une autre instance est aussi modifiée, cela signifie tout simplement que les 2 instances sont les mêmes.

Après dans ta fonction GetRunnable tu ne modifies rien tu ne fais qu'ajouter ton instance "system" dans une liste...

Ajoute plus de code pour qu'on puisse comprendre ce qui est modifié et où

[didio]

Tout d'abord voici la classe CmbdSystem 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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
public class CmbdSystem
{
    public string systemName { get; set; }
    public List<CmbdSubSystem> subsystems { get; set; }
    public List<CmbdSystem> runnables { get; set; }
 
    public CmbdSystem()
    {
        subsystems = new List<CmbdSubSystem>(); // susbsystems (that contains runnables) of the system selected
        systemName = null;
        runnables = new List<CmbdSystem>();
    }
 
    private CmbdSystem FindSystem(CmbdSubSystem subSystem)
    {
        return subSystem.system;
    }
 
    public bool IsaRunnable(){}
 
    public void GetRunnable()// utilisé afin de remplir la liste de runnable  de l'objet.
    {
        foreach (CmbdSubSystem sub in subsystems)
        {
            CmbdSystem system = FindSystem(sub);
            if (system.IsaRunnable())
            {
                runnables.Add(system);
            }
        }
    }
 
}

J'utilise un formulaire de type Windowsform et les deux attributs mbdSystem et xmlSystem appartiennent à la classe Windowsform.
C'est à travers un bouton que ces attributs sont "chargés".

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
 
public FormImport() // Constructeur
        {
            mbdParse =  new CmbdParse();
            InitializeComponent();
            xmlSystem = null;
            mbdSystem = null;
        }
 
private void buttonGetModel_Click(object sender, EventArgs e)
        {
            listRunnable.Height = 17;
            listRunnable.Items.Clear();
            if (mbdParse.UnzipSlxFile(slxPathBox.Text))
            {
                addTaggedValue(currentElement, tag2, "SlxPath", mbdParse.slxPath);
                slxPathBox.Text = mbdParse.slxPath;
                xmlSystem = mbdParse.ParseFile();
                if (xmlSystem == null)
                {
                    MessageBox.Show("Architecture error in the Xml File, please check the file");
                    import_button.Enabled = false;
                }
                else
                {
                    mbdParse.DeleteFile();
                    mbdParse.ClearMbd_Systems();
                    mbdParse.GetMbd(xmlSystem, mbdName);// remplis une liste<CmbSystem>
                    if (mbdParse.mbd_Systems.Count == 1)
                    {        
                        mbdSystem = mbdParse.mbd_Systems[0];
                        GetOutDataTypeStr(xmlSystem, mbdSystem);
                        if (!mbdSystem.IsaRunnable())
                        {
                            CmbdSystem popoo = xmlSystem.subsystems[0].system;
                            mbdSystem.GetRunnable();
                            foreach (CmbdSystem runnable in mbdSystem.runnables)
                            {
                                GetOutDataTypeStr(mbdSystem, runnable);
                            }
                            int count = mbdSystem.runnables.Count;
                            listRunnable.Height += (count - 1) * 13; 
                            foreach (CmbdSystem runnable in mbdSystem.runnables)
                            {
                                listRunnable.Items.Add(runnable.systemName);
                            }
                        }
                        import_button.Enabled = true;
                    }
 
                }
            }
 
        }

J'ai également l'impression que xmlSystem et mbdSystem sont considérés comme étant la même instance mais ils ne sont pourtant pas mis en relation l'un l'autre.

[didio]

Je pense avoir capté :

Lorsque je modifie l'un des deux objets l'autre est également modifié.

Ma classe system est composé de sous-systèmes eux même composé de system (un peu comme un arbre).

L'objet xmlSystem représente le système entier tandis que mbdSystem est une "portion" de ce système. J
'imagine que malgré le fait de "récupérer" ou affecter cette portion à mon objet mbdSystem. Toute modification faite sur l'un de ses deux objets entraîne la modification de l'autre.

Je n'ai pas trouvé l'adresse de ces deux objets mais il doit s'agir de la même ?

Merci

[Pol63]

est-ce que déjà tu es au courant de ça :

Code :

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

1
2
3
4
5
6
7
var p = new Person();
p.Name = "a";
 
var p2 = p;
p2.Name = "b";
 
var s = p.Name; // ici s vaudra "b", pas "a"

sinon va lire des cours de POO avant de coder.

[didio]

est-ce que déjà tu es au courant de ça :

Code :

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

1
2
3
4
5
6
7
var p = new Person();
p.Name = "a";
 
var p2 = p;
p2.Name = "b";
 
var s = p.Name; // ici s vaudra "b", pas "a"

sinon va lire des cours de POO avant de coder.

Bonjour

C'est exactement ce que je cherche à éviter, toujours selon certains cours de POO
il m'a été conseillé de déclarer une nouvelle instance avec les attributs de P (dans cette exemple).
Et ça n'a pas marché.