Discussion :

[LGnord]

Bonjour à tous,

J'ai une classe provider. Tous les éléments qu'elle peut fournir sont dans des des champs static. Je voudrais itérer sur tous ces éléments. Comment faire?

Voici quelques détails:

D'abord la classe Provider:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
class Provider {
 
  static element un = ...;
 
  static element deux = ...;
 
  .... quelques milliers d'éléments
 
}

Ensuite, ce que je voudrais:

Code :

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

1
2
 
Iterable<Element> all();

Une première solution:

Code :

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

1
2
 
static Element[] all = {un, deux,...}

Mais d'un coté c'est assez chiant d'ajouter un à un tous les éléments dans le tableau. Un autre inconvénient est qu'à chaque ajout d'élément il faut mettre le tableau all à jour.

Je peux aussi faire de la réfection mais je n'y connais rien. De plus, j'ai un mauvais pré-sentiment.

La solution est-elle de revoir tout le design de la class Provider? Avez-vous d'autres idées?

Merci de votre aide.

[thelvin]

La solution est-elle de revoir tout le design de la class Provider?

Oui.

Mais en mesure de secours, l'introspection marcherait très bien. Tu peux oublier ton mauvais pressentiment. Oui, c'est pas très propre, et oui, du coup, tu auras les difficultés que tu n'aurais pas avec une méthode propre. Mais ça peut dépanner.

[tchize_]

Pourquoi tous ces éléments sont statiques?
Pourquoi ne pas avoir simplement mis un tableau dans ta classe provider. Puisque all() retourne une liste, tu peux les traiter indifférement sous cette forme, donc leur nom n'a aucune importance, donc un tableau ou un liste sont bien plus pratiques.

[atlas86]

Bonjour,
Si tu veux procéder de telle façon, je suis avec thelvin pour l'introspection. Mais je te propose de passer par un fichier ".proporties"(provider.properties) qui est facile à parcourir, c'est la solution alternative pour une classe qui contient que des méthodes statiques.

[LGnord]

Merci pour votre aide.

En une demi-journée, la solution avec l’introspection a été implémentée. Le sur-coût du mauvais design est donc négligeable.

Merci pour l'idée du .properties. Je vais explorer cette piste.