Discussion :

[Invité]

Bonjour,

Je ne suis pas vraiment sur de mon raisonnement et ma façon de procéder, pouvez-vous me critiquer, conseiller ?

Dans une application swing, j'ai une classe Ligne avec ces getters et setters et autres méthodes.
Je dois représenter cette ligne visuellement, j'ai donc crée une classe LignePanel qui étend JPanel et qui comprend des JComboboxs, JtextFields, etc..

LignePanel prend en paramètre du constructeur un objet Ligne.
A l'aide des JComboBoxs et autres, je sélectionen les valeurs que je désire.
Ensuite je veux pouvoir sauvegaredr cette ligne si les valeurs sont correctes.

( -> c'est ici que je ne suis pas trop à mon aise..)
J'ai crée trois méthodes dans LignePanel :
* checkValues : vérifie mes valeurs, et place un boolean a true si tout est bon
* push : récupère chauqe valeurs et appelle le setter de Ligne associé.
* save : enregistre les valeurs placées dans Ligne

Voilà ..
Je ne sais pas si c'est une bonne façon de faire.. Ca me semble un peu fouilli.


Pouvez-vous me conseiller ? Merci

[xavlours]

Bonjour,

je ne trouve pas ça mauvais. La preuve, c'est que tu l'as expliqué sans poster de code, en quelques lignes, et de manière claire.

Si tu veux quand même changer, je dirais que dans une approche où les données sont vraiment séparées et indépendantes de l'affichage, la classe de données devrait être capable de vérifier elle même la validité des données, et donc que la méthode checkValues devrait se trouver dans la classe Ligne (et être appelée par un/des setters).

Cela augmenterait la modularité dans le sens où quelqu'un qui veut redéfinir l'affichage n'est pas sensé savoir recoder la méthode checkValues. Il doit savoir appeler un setter, et lire la doc pour comprendre le mécanisme qui refuse une valeur impossible.

[Invité]

Ok, merci beaucoup.
J'avait aussi pensé à mettre checkValues dans la classe des données mais cela imposait de faire un set de 25 valeurs avant ...
( ce que je trouvais un peu lourd.) de plus checkValues met en évidence les zones qui ne sont pas valides ( met le background en jaune ) mais ma classe de données n'est pas censée connaitre la "vue"..
Et je ne peux pas vérifer un valeur à la fois car certaines sont dépendantes d'autres..

[xavlours]

Si tu veux vraiment avoir un beau code, tu risques d'y passer plus de temps que nécessaire.
En effet, d'après ce que tu me dis, il faudrait revoir la méthode checkValues pour qu'elle soit tolérante si toutes les valeurs ne sont pas encore rentrées.
De plus, il te faudrait une méthode isValid(), et un système de Listeners pour que le modèle prévienne l'affichage que le statut isValid() a changé.

A toi de voir. Si ton code a peu de chances d'être réutilisé, c'est peut-être pas la peine.