Discussion :

[f56bre]

Bonjour,

Je programme une application avec wxWidgets, j'ai donc classiquement une wxMyFrame qui derive de wxFrame et qui contient tous les contrôle (menubar, toolbar, wxAui, wxTreeCtrl, ...). Tous ces controles doivent pouvoir modifier une structure de donnés qui appartient à wxMyFrame et doivent également pouvoir interagir entre eux ( ex : on fait un "ajoute page" à partir du menu et ça modifie l'arborescence de projet). J'ai donc commencé à faire ma table d'évènement dans wxMyFrame, laquelle gère tous les evenements de tous les controles (ainsi une fonction d'evenement sur n'importe quel controle a acces à toutes les donnes ou controles de wxMyFrame). Mon souci est que ma classe wxMyFrame commence à etre enorme et brouillonne, c'est pourquoi je vous demande si vous n'auriez pas une meilleure méthode pour arriver à mes fin en gardant un code clair.

[f56bre]

La solution que j'ai pour le moment : Etant donné que les actions sur le menu peuvent agir sur les fenêtres de ma frame et non l'inverse je pensais à faire un héritage :

wxFrameMenu derive de wxFrame et ajoute la structure de données, la menubar et toolbar ainsi que les evenements correspondants.

wxFrameAui derive de wxFrameMenu et ajoute les fenetres (wxTreeCtrl, wxGLCanvas, etc...) le tout géré avec wxAui. Les evenement sur chaque controle son implémentés ici.

wxMyFrame derivant de wxFrameAui et permettant de créer simplement ma frame.



Ceci diviserais le code en 2 (au niveau de la gestion des événements) et ce serais selon moi plus propre.

Cette solution vous semble t'elle bonne?
Si oui est il possible d'aller plus loin?

merci d'avance

[PRomu@ld]

Cette solution vous semble t'elle bonne?

En tout cas, c'est comme ça que j'aurai fait.

Si oui est il possible d'aller plus loin?

Pour aller plus loin, il faudrait voir si on ne peut pas factoriser un peu plus mais il faudrait voir le code parce que là c'est pas évident

[f56bre]

Pour ce qui est du code, tout ce qu'il y a de plus classique pour l'ajout des menubar et toolbar.
Pour le reste j'ai un wxAuiManager dans ma wxMyFrame auquel j'ajoute un wxTreeCtrl, un wxGLCanvas et un wxNotebook. Je configure chacun des mes controles à ma guise et je les ajoute avec AddPane().

[PRomu@ld]

Et bien je ne vois pas grand chose en fait. Le fait que tu ais beaucoup de code n'est pas très grave, c'est même plutôt normal.

Si tu as eu l'occasion de voir ce que des outils visuels de création d'interface (visual studio, c++builder, ...) , tu pourras remarquer que pour une simple application la quantité de code générée est enorme.

[Alp]

Generalement, il est tres conseille de diviser. Diviser pour mieux reigner. C'est donc dans la meme direction que celle que tu empruntes.
Tu pourrais en fait isoler toutes les sections qui sont separables en des classes distinctes. Pourquoi? Tout d'abord ton code sera plus clair, et tu pourras plus facilement localiser des bugs, etc ... Ensuite, si un jour tu desires modifier une section, cela te sera facile car elle sera deja isolee. Tu n'auras ainsi que peu de code a modifier. Alors que cela avait ete en vrac dans ta classe de frame...

[f56bre]

Merci pour vos réponses,

Effectivement le developpement de mon application va se placer sur le long terme donc je préfère prendre mes dispositions pour ecrire un code clair. Je m'oriente donc vers une personnalisation de tous les composants wx que j'utilise (via l'héritage) et une séparation claire du code.
De plus je prépare le terrain pour un futur travail en équipe et ayant rencontré pas mal d'exemples de code où tout est en vrac dans la frame je vois bien que ce n'est pas viable.

re-merci.