Discussion :

[Kaluza]

Bonjour.

Je viens d'implémenter quelque chose en Qt qui me satisfait pleinement, mais je ne sais pas trop si c'est dans l'esprit C++ et si ça ne peut pas poser de problème en mémoire.

J'ai un widget perso QWidget MonWidget, qui a une variable privée : QGraphicsScene *_graphicsScene;
Comme c'est un composant essentiel de mon widget, je ne voudrais pas avoir à réimplémenter tous les set/get QGraphicsScene dans QWidget MonWidget.

J'ai donc créé une fonction :

Code :

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

1
2
3
4
QGraphicsScene* MonWidget::getScene()
{
    return _graphicsScene;
}

Depuis l'extérieur, du widget, je peux donc faire par exemple :

Code :

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

monWidget.getScene()->setBackgroundBrush(QColor(255,0,0));

qui agit directement sur _graphicsScene (variable privée).

Comme c'est la première fois que je fais ce genre de choses, je voudrai être sûr que c'est "C++ compliant"/"Qt compliant" et que ça ne pourrait pas causer d'éventuels problèmes...

Merci

[LittleWhite]

Bonjour,

Le code actuel est insuffisant pour comprendre le problème. Peut on avoir un projet que l'on puisse tester ?

[gbdivers]

Bonjour Kaluza

En général, les fonctions get/set brisent l'encapsulation objet et sont donc à éviter.

Mais pour la QGraphicsScene, on peut voir que QGraphicsView propose les fonctions scene() et setScene(). Donc ton code n'est pas éloigné de l'esprit de Qt A mon avis, la raison est que la scène n'est pas considérée comme une variable interne de la vue mais elle peut être partagée entre plusieurs vues (pattern modèle-vue). Par contre, cela implique normalement que la responsabilité de la construction/destruction de la scène n'est pas dans la vue.

En fait, si tu utilises QGraphicsScene, tu ne dois pas hériter de QWidget mais de QGraphicsView (qui hérite lui même de QWidget), qui contient déjà beaucoup de fonction facilitant l'utilisation de la scène (par exemple, appel de la scène lors des mises à jour, envoie des events à la scène, etc.) Repartir de QWidget ressemble beaucoup à "réinventer la roue" (syndrome NIH )

Bon continuation