Discussion :

[manitor]

Bonjour,

J'ai passé un moment à parcourir le forum mais n'ai pas trouvé la réponse. Ma question reprend un peu les derniers échanges de mon dernier post, mais comme cela n'a rien à voir avec le titre, je me permets d'ouvrir une nouvelle discussion.

J'ai une classe:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Class A{
list<int> m_List;
list<int>::iterator m_element;
 
public:
A(list<int>&)
};
 
A::A(list<int>& list){
m_List = list;
m_element=m_List.begin(); 
}

J'ai besoin de m_element parce que je crée ensuite un vector de A, et j'ai besoin de savoir pour chaque liste quel est l'élément courant:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
Class B{
vector<A> m_Vect;
 
public:
B(list<int>&)
};
 
B::B(list<int>& list){
A a(list);
vector<A> vecttemp;
vecttemp.push_back(a);
m_Vect=vecttemp;
}

Avec ce code, m_element de a pointe sur le premier élément de m_List. Quand je fais (dans le constructeur de B) vecttemp.push_back(a), m_element "ne bouge pas", alors que bien entendu, ce qui m'intéresserait, c'est qu'il indique maintenant m_Vect[0].m_List.begin(). Existe-t-il une solution sans devoir le lui écrire explicitement dans le constructeur de B?

Merci beaucoup

[gbdivers]

Bonjour Manitor

m_element => list<int>::iterator (dans un objet A)

_Vect[0].begin() => vector<A>::iterator (dans un objet B)

Je ne comprend pas très bien ce que tu souhaites faire.

[3DArchi]

Salut,
Je rajouterais que le constructeur ne peut pas avoir un retour
:

Code :

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

1
2
3
4
5
6
B::B(list<int>& list){
A a(list);
vector<A> vecttemp;
vecttemp.push_back(a);
return vecttemp; --> Ne compile pas !
}

Je suis comme gbdivers, je n'arrive pas à comprendre ce que tu veux faire. Qui doit se souvenir de quoi ?

[manitor]

Merci pour les réponses...

Oui, bien sûr, pas de return dans un constructeur... Un moment d'égarement, j'ai corrigé mon code.

Je vais essayer d'expliquer un peu mieux. J'ai donc une classe B qui est un vector<A>, où A est une liste d'entier (m_List) avec un iterator (m_element) pointant sur un élément la liste.

Si je construis un objet B, noté b, à partir d'une liste d'entier (c'est le constructeur de B), je passe par le constructeur de A. Dans le constructeur de A, j'ai initialisé m_element comme étant le premier élément de m_List. Si je crée un A, noté a, quand je fais un push_back(a) pour en faire le premier élément de b, il fait une copie de la liste d'entier, ainsi qu'une copie de l'iterator pour l'insérer dans le vector. L'iterator pointe toujours vers a.begin(), alors que j'aimerais qu'il pointe vers le premier élément de la copie de m_List qui vient d'être ajoutée vector, soit b.m_Vect[0].m_List.begin() (deuxième faute de ma première demande que je corrige aussi...).

[3DArchi]

En préambule, je dirais que je n'aime pas trop me trimbaler des itérateurs dans des objets.
Mais pour répondre à ton problème, tu devrais redéfinir le constructeur par copie de A (et l'opérateur = aussi) :

Code :

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

1
2
3
4
5
   A(const A&rhs_)
   :m_List(rhs_.m_List)
   {
      m_element = m_List.begin();
   }

[gbdivers]

Ok, c'est la suite de la discussion : http://www.developpez.net/forums/d80...ment-fonction/

Pour info 3Darchi, c'est moi qui lui ai conseillé de mettre sa liste et son pointeur dans une classe.

Mais le but était de garder un lien vers le dernier élément inséré (qui n'est pas forcement en première position). Dans ce cas, le mieux est d'invalider le pointeur et de ne plus l'utiliser (sinon, il faut rechercher dans la nouvelle liste l'ancien élément pointé, etc. Un peu lourd).

Sinon, explique pourquoi tu veux garder un pointeur sur le dernier élément inséré. Il y a peut être moyen de faire autrement.