Discussion :

[LittleWhite]

Bonjour,

J'ai vraiment une question idiote, qui de plus me traumatise depuis le défi Qt.

Avant, pour chaque classe qui hérite de Qt, si j'avais besoin d'un bouton dans ma classe, je définissais un pointeur sur un QButton dans la classe, que j'allais contruire ( new QButton ) dans le constructeur ( et détruire proprement dans le destructeur ).
Le problème, c'est que dans le C++ on dit qu'il faut évité d'utiliser les pointeurs.
Deuxième point, c'est que dans certains ( Inclusion d'un element dans un QGridLayout ( ou autre ) ), les éléments sont détruit par Qt automatiquement.

Maintenant, je veux faire en sorte de ne pas utiliser de pointeur \o/.

Mais pour le code suivant:

class.hpp

Code :

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

1
2
3
4
5
6
7
8
 
class Bar : public QWidget
{
     private:
          QButton button;
     public:
          Bar(void);
};

class.cpp

Code :

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

1
2
3
4
5
 
Bar :: Bar(void)
{
     button = QButton(this);
}

Une chose me turlupine. Il va me construire deux fois le QButton n'est ce pas ( constructeur vide, puis constructeur avec this comme paramètre )?
Si on construit deux fois le QButton, je trouve cela un peut inutile. Ou alors, j'ai vraiment perdu une notion du C++.

Pourriez vous avoir l'obligeance de m'éclaircir les idées, et me dire quel est la bonne pratique à utiliser ?

Merci beaucoup

[ymoreau]

Moi je crée mes objets en composition sans pointeur dans la mesure du possible. Pour ce qui est de la création de 2 objets, ça dépend de l'implémentation de l'opérateur égal ! C'est donc propre à chaque classe, je sais que dans Qt il y a un système qui optimise ce genre de choses en utilisant de manière transparente des références partageant les mêmes données pour éviter les copies inutiles, mais je ne pourrais pas t'en dire plus sur le sujet.

En tout cas, dans ton code, il y a une manière C++ et simple d'éviter la copie : utiliser la "vraie" initialisation :

Code :

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

1
2
3
Bar :: Bar(void) : button(this)
{
}

Cela donnera directement l'argument au moment de la création de ton bouton, alors que le code placé dans le constructeur est exécuté juste après sa création.

[gbdivers]

je sais que dans Qt il y a un système qui optimise ce genre de choses en utilisant de manière transparente des références partageant les mêmes données pour éviter les copies inutiles, mais je ne pourrais pas t'en dire plus sur le sujet.

Il s'agit du système COW (copy on write) qui permet de ne pas copier toute les données lors de la duplication d'un objet mais uniquement lors de la première modification de ces données : http://qt.developpez.com/doc/latest/implicit-sharing/

la "vraie" initialisation

Ca s'appelle un liste d'initialisation
A lire : http://cpp.developpez.com/faq/cpp/?p...EUR_liste_init

[LittleWhite]

Vous avez raison, la liste d'initialisation est la meilleure methode, mais je ne suis pas completement sur que cela fonctionne tout le temps (par exemple, des parametres calculer selon les autres variables, ou je ne sais quoi).
Et puis, cela va me faire de longue liste d'initialisation

Pour la Copy On Write, je ne pense pas que ce soit ce qui correspond a ma question. Mais merci de m'apprendre toujours plus.

[gbdivers]

Pour le COW, c'était pour préciser les propos de YoniBlond, pas répondre à la question

Et puis, cela va me faire de longue liste d'initialisation

pas très grave. Et vaux mieux ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
Bar::Bar(void) :
    button1(this),
    button2(this),
    button3(this),
    button4(this),
    button5(this),
    button6(this),
    button7(this)
{
}

que de payer le "coût" d'une copie...

Imagine qu'a la place d'un simple bouton, tu as une très grosse classe (par exemple contenant plusieurs std::vector de taille 10000000) : les appels répétés à l'opérateur de copie peut être très couteux.

PS: je viens de remarquer que QVector utilise le COW, au contraire de std::vector... une bonne raison d'utiliser QVector par défaut
PS2 : pour un simple QPushButton, ce n'est pas très grave de faire une copie. QPushButton n'utilise pas le COW mais utilise QPixmapCache, ce qui évite de copier plusieurs fois le pixmap (http://qt-quarterly.developpez.com/qq-12/qpixmapcache/)

[yan]

Salut.
J'ai lue rapidement...
Mais
1- les QObject ne sont pas et ne doivent pas être copiable!!! C'est pas prévue pour. C'est pour cela que l'operateur = est private.
2- les widget sont reparenté automatique => forte probabilité de problème mémoire si on utilise pas des pointeurs.

http://qt.developpez.com/faq/?page=g...ire#memoire-qt
http://qt.developpez.com/faq/index.p...qobject-membre

Sans le cas de Qt, les widget oblige l'utilisation de pointeur. Mais en contre partie, la mémoire au automatiquement géré (GC interne).
Il y as quelques pointeur intelligent intéressant comme QPointer ou QWeakPointer qui peuvent être utile.