Discussion :

[Stobbyo]

Hello,

J'ai un petit problème avec un booléen récalcitrant.
Voici la classe en question :

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
template <typename T>
    class GraphicObject : public System::Linkable<T>
    {
 
    public :
 
        GraphicObject( void );
        GraphicObject( Layer<T> & );
        ~GraphicObject( void );
 
        // -----------
 
        inline bool IsVisible( void ) const {
            return my_visibility;           }
 
        // -----------
 
        void Hide( bool = true );
        void Show( bool = true );
        void SetLayer( Layer<T> &, bool show = true );
 
        // -----------
 
    private :
 
        Layer<T> * my_layer;
        bool my_visibility;
    };

Et le code des constructeurs dans le fichier "inline".

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
template <typename T>
    GraphicObject<T>::GraphicObject( void )
        : System::Linkable<T>(), my_layer(0), my_visibility(false)
    { 
 
    }
 
    template <typename T>
    GraphicObject<T>::GraphicObject( Layer<T> &layer )
        : System::Linkable<T>(), my_layer(&layer), my_visibility(true)
    {
        my_layer->push_back( this );
    }

Quoi qu'il arrive le booléen "my_visibility" est toujours égal à true.

J'ai même essayé de le définir en dehors de la liste d'initialisation, ça ne change rien.

Code :

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

1
2
3
4
5
6
template <typename T>
    GraphicObject<T>::GraphicObject( void )
        : System::Linkable<T>(), my_layer(0)
    { 
        my_visibility = false; // ça ne change rien
    }

Est-ce déjà arrivé à quelqu'un ?

J'utilise le compilateur Visual C++ 2008 (Express Edition).

Si quelqu'un a une idée, merci

[3DArchi]

Code :

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

1
2
void Hide( bool = true );
        void Show( bool = true );

?

[skeud]

Code :

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

my_layer->push_back( this );

Je suis étonné que tu arrive a compiler avec ça, sachant que ta liste contient des objet de type "T" alors que this est de type "GraphicObject *"

Ensuite si tu ne nous passes pas le code ou tu créé ton objet (dans le main) on peut pas vraiment te dire, je pense que ça vient pas de ton objet mais de la manière de construire ton objet dans le main.

[Stobbyo]

Bonjour,

J'ai marqué le sujet avec le "tag" délestage. J'ai trouvé d'où venait le problème.
C'est un programme composé de plusieurs projets, et l'un d'eux n'était pas à jour.

@3DArchi

Le booléen en paramètre n'est pas utile ? Oui c'est vrai

@skeud

C'est vrai, qu'il y aurait eu une erreur de compilation, mais, n'ayant pas fait d'appel à ce constructeur dans le programme, il n'a donc pas été compilé.

Code :

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

my_layer->push_back( static_cast<T*>(this) );

La classe Layer, n'est pas un conteneur de la STL, c'est une liste cyclique.
La méthode push_back, prend un pointeur en paramètre.

Merci et désolé,

[3DArchi]

Le booléen en paramètre n'est pas utile ? Oui c'est vrai

Ce qui m'a choqué c'est que dans les deux cas la valeur par défaut soit true, la même. Du coup, je me demande à quoi sert ce paramètre ? En fait, probablement à rien et qu'il est redondant avec la fonction.

[Stobbyo]

de cette manière, on pouvait écrire :

Code :

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

1
2
object.Hide(false); // équivalent à object.Show()
object.Show(false); // équivalent à object.Hide()

en gros, ça ne sert pas à grand chose lol
ça permet juste d'exprimer les choses différemment.

[3DArchi]

ça permet juste d'exprimer les choses différemment.

AMHA, ça créé de la confusion. Garde juste Hide()/Show() sans paramètre. C'est quand même très clair comme nom de fonction.

[skeud]

AMHA, ça créé de la confusion. Garde juste Hide()/Show() sans paramètre. C'est quand même très clair comme nom de fonction.

"Pourquoi faire simple quand on peut faire compliqué?"

[Stobbyo]

AMHA, ça créé de la confusion. Garde juste Hide()/Show() sans paramètre. C'est quand même très clair comme nom de fonction.

"Pourquoi faire simple quand on peut faire compliqué?"

J'ai enlevé les booléens en paramètres.
Merci pour ces conseils

A plus,