Discussion :

[nmarf]

Bonjour à tous,

J'ai une CList de CLogManager (classe maison) définie comme il suit :

Code :

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

1
2
 
CList<CLogManager, CLogManager&> m_logs;

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
 
class CLogManager
{
public:
[...]
    void SetDividers(double fCrossWebDivider, double fDownWebDivider);
[...]
 
private:
[...]
    double m_fDownWebDivider;
    double m_fCrossWebDivider;
};
 
 
void CLogManager::SetDividers(double fCrossWebDivider, double fDownWebDivider)
{
    m_fDownWebDivider = fDownWebDivider;
    m_fCrossWebDivider = fCrossWebDivider;
}

J'essaie de modifier les deux doubles de chaque élément de la liste par la fonction SetDividers() mais aucune modification n'est prise en compte.
Je fais comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
void CMultipleLogsManager::SetDividers(double fCrossWebDivider, double fDownWebDivider)
{    
    POSITION pos = m_logs.GetHeadPosition();
 
    while (pos != NULL)
    {
        CLogManager& logTmp = m_logs.GetNext(pos);
        logTmp.SetDividers(fCrossWebDivider,fDownWebDivider);
    }
}

Il semblerait que le problème soit logTmp qui finalement n'est qu'une copie de l'élément de la liste, mais alors, comment récupérer un pointeur sur chaque élément pour pouvoir faire les modifs ?

[farscape]

salut,
bizarre ton probleme:

CList::GetNext

TYPE& GetNext( POSITION& rPosition );
TYPE GetNext( POSITION& rPosition ) const;
Return Value
If the list is const, GetNext returns a copy of the element at the head of the list. This allows the function to be used only on the right side of an assignment statement and protects the list from modification.
If the list is not const, GetNext returns a reference to an element of the list. This allows the function to be used on either side of an assignment statement and thus allows the list entries to be modified.

[nmarf]

Oui, j'ai vu ça aussi dans la MFC.
Ma CList n'est pas constante, des éléments sont rajoutés et déplacés dans d'autres fonctions.

[r0d]

Personellement, je préfère utiliser des pointeurs dans mes listes. Du style:

Code :

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

CList<CLogManager*, CLogManager*> m_logs;

L'avantage c'est que justement, il n'y a pas d'ambiguité sur pointeur/non pointeur. On sait qu'il s'agit de pointeur. Et puis on n'est pas obligé de créer un constructeur de recopie et un operateur =.
Enfin, c'est mon point de vue, peut-être y a-t-il des implications que je ne vois pas.

[nmarf]

r0d > je vais essayer de modifier avec des pointeurs, sinon je changerai toutes les CList de ce code qui n'est pas le mien par des vector<> et des list<> que je préfère largement.