Problème de référence

Version imprimable

12/01/2007, 22h48
babozfr

Problème de référence

Bonjour,
Voilà je commence depuis peu à développer en C++ .net (mais depuis 2 ans en C++) et voilà j'ai un petit souci.
Je développe un convertisseur de carnet d'adresse MAC vers carnet d'adresse PC(pour ceux que ça intéresse). Je scrute mon fichier VCF qui contient tous mes contact. Je créé un objet "personne" qui va contenir les données de la personne en cours. A la fin des données concernant cette personne, j'ajoute l'objet "personne" dans une liste soit list->Add(personne) (aussi simplement qu'en java d'ailleurs).
A chaque nouvelle personne, je réutilise mon objet personne que j'ai créé précédemment tout en le réinitialisant à chaque personne rencontrée (je ne désalloue pas afin de faire du recyclage d'objet).
Le souci est que lorsque j'affiche l'ensemble des contacts mémorisés (ici j'en ai 66), les 66contacts ont le même nom.

J'ai l'impression que le list->add(objet) ne fait que passer l'adresse et ne copie pas l'objet personne.
Que dois je faire un constructeur de copie ? (ça me pose quelques soucis en .net)...
Merci de m'éclaircir
12/01/2007, 23h08
nico-pyright(c)

Citation:

Envoyé par babozfr

Bonjour,
Voilà je commence depuis peu à développer en C++ .net (mais depuis 2 ans en C++) et voilà j'ai un petit souci.
Je développe un convertisseur de carnet d'adresse MAC vers carnet d'adresse PC(pour ceux que ça intéresse). Je scrute mon fichier VCF qui contient tous mes contact. Je créé un objet "personne" qui va contenir les données de la personne en cours. A la fin des données concernant cette personne, j'ajoute l'objet "personne" dans une liste soit list->Add(personne) (aussi simplement qu'en java d'ailleurs).
A chaque nouvelle personne, je réutilise mon objet personne que j'ai créé précédemment tout en le réinitialisant à chaque personne rencontrée (je ne désalloue pas afin de faire du recyclage d'objet).
Le souci est que lorsque j'affiche l'ensemble des contacts mémorisés (ici j'en ai 66), les 66contacts ont le même nom.

J'ai l'impression que le list->add(objet) ne fait que passer l'adresse et ne copie pas l'objet personne.
Que dois je faire un constructeur de copie ? (ça me pose quelques soucis en .net)...
Merci de m'éclaircir

montre nous ton code

Il est sale :)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
CReadVcf::CReadVcf(TextReader ^ stream){
    String ^ line;
    int count = 0;
    bool readable = 0;
    this->fileStream = stream;
    personne = gcnew CObjetVcf();
    array<String ^, 1> ^ adr = gcnew array<String ^>(3);
    contacts = gcnew System::Collections::Generic::List<CObjetVcf^>();
 
    while(line = stream->ReadLine()){        
        if(line->StartsWith("BEGIN:VCARD")){
            //on saute la version
            stream->ReadLine();
            readable = 0;
        }
        if(line->StartsWith("END:VCARD")){
            readable = 0;
            count++;
            contacts->Add(personne);
            personne->Re set();            
        }

Objet personne :

.h

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
#pragma once
using namespace System;
using namespace System::Windows::Forms;
 
 
ref class CObjetVcf{
 
private: 
    String ^ nom, ^ prenom, ^mailWork, ^mailHome;
    array<String^>^adresseWork,^adresseHome;
    int age;
    String^ telFixe,^telMobile,^telTravail;
 
public :
    CObjetVcf(); //constructeur
    CObjetVcf CObjetVcf(const CObjetVcf ^o); //constructeur de copie
    String^ getTelFixe();
    String^ getTelMobile();
    String^ getTelTravail();
    int getAge();
    String^ getNom();
    String^ getPrenom();
    array<String^>^ getAdresseWork();
 
    void setNom(String ^);
    void setPrenom(String ^);
    void setAdresseWork(array<String^>^);
    void setAdresseHome(array<String^>^);
    void setMailWork(String ^);
    void setMailHome(String ^);
 
    void convertToLdiff();
    String^ Show();
    void Reset();
};

.cpp

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
#include "stdafx.h"
#include "CObjetVcf.h"
 
CObjetVcf::CObjetVcf(){
    Reset();
}
 
CObjetVcf CObjetVcf::CObjetVcf(const CObjetVcf ^o){
    this->nom = o->nom;
    this->prenome
        //String ^ nom, ^ prenom, ^mailWork, ^mailHome;
    //array<String^>^adresseWork,^adresseHome;
    //int age;
    //String^ telFixe,^telMobile,^telTravail;
}
 
void CObjetVcf::Reset(){
    nom = "";
    prenom = "";
    telFixe = "";
    telMobile = "";
    telTravail = "";
    age = 0;
}
 
String^ CObjetVcf::getTelFixe(){ return this->telFixe;}
String^ CObjetVcf::getTelMobile(){ return this->telMobile;}
String^ CObjetVcf::getTelTravail(){ return this->telTravail;}
int CObjetVcf::getAge(){ return this->age;}
String^ CObjetVcf::getNom(){ return this->nom; }
String^ CObjetVcf::getPrenom(){ return this->prenom;}
array<String^>^ CObjetVcf::getAdresseWork(){ return this->adresseWork;}
void CObjetVcf::setNom(String ^ name){ this->nom = name;}
void CObjetVcf::setPrenom(String ^ surname){ this->prenom = surname;}
void CObjetVcf::setAdresseWork(array<String^>^adr){ this->adresseWork = adr;}
void CObjetVcf::setAdresseHome(array<String^>^adr){ this->adresseHome = adr;}
void CObjetVcf::setMailWork(String^ mail){this->mailWork=mail;}
void CObjetVcf::setMailHome(String^ mail){this->mailHome=mail;}
 
void CObjetVcf::convertToLdiff()
{
 
}
 
String^ CObjetVcf::Show(){
    String^contenu="";
    contenu = "Nom : "+this->nom+"\nPrénom : "+this->prenom+"\nAdresse Travail : "+this->adresseWork[0]+"\n"+this->adresseWork[1]+" "+this->adresseWork[2]+"\nMail : "+this->mailWork+"\n --------\n";
    return contenu;
}

Merrciiiii :)

13/01/2007, 08h52
nico-pyright(c)

j'ai pas trop cherché plus loin, mais dans ta boucle, tu utilises toujours le meme objet personne
tu devrais en recréer un à chaque itération et le remplir de valeur
13/01/2007, 09h02
babozfr

justement je fais exprès d'utiliser le même objet car ça évite d'allouer / désallouer la mémoire, ce qui prend plus de temps...
on nous a appris à "recycler les objets" il parait que ça optimise les prog.
Merci
13/01/2007, 10h15
babozfr

je viens d'essayer la même chose en allouant et désallouant l'objet à chaque boucle. Le programme se lance mais une erreur est lancée. Je confirme ce que je pense, la liste ne copie pas mon objet mais pointe dessus ce qui est gênant lorsque je delete l'objet et que j'affiche le contenu de ma liste.
Avec vous une solution ? Je suis bloqué...
Si j'utilise la STL ça marcherait mieux ? J'ai des doutes sur l'outil list de .net... je n'ai jamais rencontré ce problème en utilisant les STL.
Merci
13/01/2007, 11h06
nico-pyright(c)

il faut que tu alloues un nouvel objet à chaque fois

tu dois avoir un problème dans ton code

et ne t'inquietes pas pour la mémoire, le GC s'en occupe
14/01/2007, 22h20
babozfr
J'ai trouvé la solution à ma réponse. C'était bien un problème de référence.
Il faut donc "cloner" mon objet (je pense que ça correspond à faire une copie) avant de l'ajouter à la liste.
Donc pas besoin de recréer un objet à chaque boucle de ce fait.

voilà le code :

Code:

Object^ Clone(){return this->MemberwiseClone();};
Code:

1 2 3 4 System::Collections::Generic::List<Object^> contacts = gcnew System::Collections::Generic::List<Object^>(); contacts->Add(personne->Clone());
Bonne soirée