Discussion :

[titpuce]

Bonjour
Une question de débutant, mais débutant qui a regardé la FAQ and co avant de poster....
Je fais évoluer une application java, j'ai les mains liées sur un certain nombre de points.
Cet application sert à générer des motifs géométriques.
Un motif a pour caractéristique : une forme, une surface, un jeu de couleur spécifique par forme.
J'ai rajouté un type de motif, et donc rajouté un jeu de couleurs. Le nom des couleurs étant stocké sous forme de tableau de String, j'ai créé un tableau pour chaque forme. (on va les appeler couleur_carre; couleur_rond)
Jusque là, comme il n'y avait qu'une couleur, le code était simple. Par exemple pour afficher la couleur du motif dans une fenetre :

Code :

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

pC.add(new ALabel(Couleur[motif.couleur]));

Vu que maintenant il y a deux listes de couleur en fonction du motif, dois-je reprendre tout le code pour pouvoir indiquer la bonne liste en fonction de la couleur du motif de cette manière ?

Code :

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

1
2
if (motif.couleur<5) pC.add(new ALabel(Couleur_carre[motif.couleur]));
else pC.add(new ALabel(Couleur_rond[motif.couleur]));

ou existe-t-il une manière nettement plus élégante de faire qui me permettrait de ne m'occuper de la valeur de motif.couleur seulement au moment où je consulte mon tableau ?

[franck06]

Bonjour
Une question de débutant, mais débutant qui a regardé la FAQ and co avant de poster....
Je fais évoluer une application java, j'ai les mains liées sur un certain nombre de points.
Cet application sert à générer des motifs géométriques.
Un motif a pour caractéristique : une forme, une surface, un jeu de couleur spécifique par forme.
J'ai rajouté un type de motif, et donc rajouté un jeu de couleurs. Le nom des couleurs étant stocké sous forme de tableau de String, j'ai créé un tableau pour chaque forme. (on va les appeler couleur_carre; couleur_rond)
Jusque là, comme il n'y avait qu'une couleur, le code était simple. Par exemple pour afficher la couleur du motif dans une fenetre :

Code :

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

pC.add(new ALabel(Couleur[motif.couleur]));

Vu que maintenant il y a deux listes de couleur en fonction du motif, dois-je reprendre tout le code pour pouvoir indiquer la bonne liste en fonction de la couleur du motif de cette manière ?

Code :

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

1
2
if (motif.couleur<5) pC.add(new ALabel(Couleur_carre[motif.couleur]));
else pC.add(new ALabel(Couleur_rond[motif.couleur]));

ou existe-t-il une manière nettement plus élégante de faire qui me permettrait de ne m'occuper de la valeur de motif.couleur seulement au moment où je consulte mon tableau ?

un motif n'est pas modélisé par une classe ??

[titpuce]

Bonjour Franck,
les couleurs ne sont pas définies dans la classe motif, mais dans une interface. Sinon il me suffirait de faire une méthode et le pb serait réglé.
Dans mon appli un motif utilise un groupe de couleur, un composant aussi.
La classe motif sert à définir un motif qui est affiché.
La classe composant sert à définir un composant qui est défini dans un fichier plat.
Un composant contient des motifs et autres structures.

[gifffftane]

Peut être peux-tu reprendre le constructeur de ALabel de la façon suivante approximativement :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
ALabel(Couleur couleur)
{
 this.couleur = couleur;
 if (couleur.getMotifCouleur() < 5)
   this.couleur = Couleur_carre[motif.couleur];
 else
   this.couleur = Couleur_rond[motif.couleur];
...
}

??

[titpuce]

merci gifftane.
Mais ta solution est simplement une réécriture de ce que j'ai mis plus haut non ?
En fait ma liste de couleurs n'est pas appelée que pour les menus. Si j'affiche un motif, la liste de couleur suit, donc si je comprends bien ta méthode, pour chaque type d'objet, je me retrouve à redéfinir le choix des couleurs ?

[gifffftane]

J'ai quelque mal à saisir l'ensemble du contexte. Je ne sais pas d'où sortent ces menus dont tu me viens de me parler. J'avais l'impression de ALabel, donc, peut être, de labels ??

La solution que je suggère intervient dans le constructeur de cet ALabel, de façon à définir la couleur, selon, effectivement, ton écriture (c'était me semble-t-il son intérêt). Simplement, comme c'est dans le constructeur, tu n'as plus besoin de l'écrire partout où tu fais appel à la création de cet objet, comme c'était le cas, avais-je l'impression, avant. Mais je dois admettre que ce ne sont que pures suppositions.