Discussion :

[djidane39]

Voilà mon problème:
J'ai une classe State avec plusieurs champs static

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
/** disconnected state */
	public static final int DISCONNECTED = 0;
 
	/** waiting connection confirmation state */
	public static final int WAITING_CONNECTION_CONFIRMATION = 1;
 
	/** connected with local list state */
	public static final int CONNECTED_LOCAL_LIST = 2;
 
	/** connected state */
	public static final int CONNECTED = 3;
 
	/** select a file to download state */
	public static final int P2P_SELECTED = 4;

J'aimerai afficher un de ces champs non pas avec la valeur de l'entier, mais avec le nom du champs qui correspond à une valeur donnée. par exemple si je donne la valeur int state = 3, j'aimerais qu'il me retourne la chaine "CONNECTED". Voici l'entete de ma fonction:

Code :

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

public static String getStringState(int state)

Je pense qu'il faut utiliser l'introspection ou la reflexivité, en regardant les tuto et la faq, c'est ce qui s'en rapproche le plus. Cependant je n'ai pas trouvé comment faire -_-'

[krachik]

bonjour
sinon tu peux faire ça

Code :

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

1
2
3
4
 
public static String getStringState(int state){
    return (state==CONNECTED?"CONNECTED":" ";
}

mais vu que CONNECTED soit un nom de variable mais une constante ........................

[djidane39]

oui je peu faire comme ça, mais vu que mes etat vont peu être changer, je voulais ne pas avoir à tout changer mon Switch case -_-'
il me faudrait un truc du style

Code :

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

String res = getNomDuChampsStaticQuiALaValeur(int state)

[djidane39]

en fait voilà ma solution actuelle, version "little player" qui n'est pas évolutif:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
public static String getStringState(int state) {
		String stringState = "";
		switch (state) {
			case DISCONNECTED:
				stringState = "DISCONNECTED";
			case WAITING_CONNECTION_CONFIRMATION:
				stringState = "WAITING_CONNECTION_CONFIRMATION";
			case CONNECTED_LOCAL_LIST:
				stringState = "CONNECTED_LOCAL_LIST";
			case CONNECTED:
				stringState = "CONNECTED";
			case P2P_SELECTED:
				stringState = "P2P_SELECTED";
			case DOWNLOADING:
				stringState = "DOWNLOADING";
			case PDF_SELECTED:
				stringState = "PDF_SELECTED";
			case VIEWING:
				stringState = "VIEWING";
			case WAITING_DISCONNECTION_CONFIRMATION:
				stringState = "WAITING_DISCONNECTION_CONFIRMATION";
		}
 
		return stringState;
	}

il me faudrait quelquechose de vraiment indépendant vis-à-vis des nom de constante qui risque de changer

[krachik]

Dans un cadre evolutif ou de mise à jour tu pourrais aussi creer un tableau dynamique pour stocker les etats ou un fichier pour les stoker

[bouye]

Note : il manque des break; dans ton switch.

Suivant la version que tu as de Java, ce sera beaucoup plus pratique de faire ca avec des enum :

Code :

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

1
2
3
enum State {
DISCONNECTED, WAITING_CONNECTION_CONFIRMATION, WAITING_CONNECTION_CONFIRMATION, CONNECTED_LOCAL_LIST, CONNECTED_LOCAL_LIST, CONNECTED, P2P_SELECTED, DOWNLOADING, PDF_SELECTED, VIEWING, WAITING_DISCONNECTION_CONFIRMATION;
}

Puisque la method toString() de l'enum retourne directement la chaine correspondante au nom de la variable/valeur, qu'on peut retrouver un entier via la methode ordinal() et que ca marche directement dans un switch.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
State state = ...
 
String stringState = state.toString();
 
switch (state) {
  case DISCONNECTED :
     [...]
  break;
  case WAITING_CONNECTION_CONFIRMATION:
     [...]
  break;
  [...]
}

Sinon pour decoupler completement le texte a afficher de la valeur il suffit de mettre tout ca dans un fichier de ressources ; par exemple :

Code :

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

1
2
3
4
ResourceBundle bundle = ResourceBundle.getBundle("Labels");
String stringState = bundle.getString("LABEL_STATE_"+state);
// Ou qq chose du genre :
String stringState = bundle.getString("LABEL_STATE_"+state.ordinal());

Ca peut aussi marche avec tes entiers. Ou tu peux mettre les chaines a afficher dans une Map initialisee statiquement (ou pas) (la aussi ca marche avec tes entiers de maniere similaire), ajouter une methode a l'enum, surcharger toString(), etc... Bref, il y a plein de solutions...

[adiGuba]

Salut,


Les enum de Java 5 serait bien entendu la meilleure solution


Sinon on pourrait se baser sur les Jakarta Commons Lang qui propose un Design Pattern pour cela via la classe org.apache.commons.lang.enums.Enum...


Ou bien on peut directement utiliser une classe spéciale, par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
// Classe finale pour qu'elle ne soit pas étendu
public final class State {
 
	// Attributs des valeurs de l'énums (nom + valeur)
	private final String name;
	private final int value;
 
	// Constructeur privé (pour interdire la création de nouvelle instance)
	private State(String name, int value) {
		this.name = name;
		this.value = value;
	}
 
 
	public String name() {
		return this.name;
	}
 
	public int value() {
		return this.value;
	}
 
 
	/** disconnected state */
	public static final State DISCONNECTED = new State("DISCONNECTED", 0);
 
	/** waiting connection confirmation state */
	public static final State WAITING_CONNECTION_CONFIRMATION = new State("WAITING_CONNECTION_CONFIRMATION", 1);
 
	/** connected with local list state */
	public static final State CONNECTED_LOCAL_LIST  = new State("CONNECTED_LOCAL_LIST", 2);
 
	/** connected state */
	public static final State CONNECTED  = new State("CONNECTED", 3);
 
	/** select a file to download state */
	public static final State P2P_SELECTED  = new State("P2P_SELECTED", 4);
}

Le problème principale vient du fait que les switch() ne sont plus possible


a++

PS : a noter également qu'en cas de sérialisation il faut rajouter en plus des traitements particulier pour gérer l'unicité des valeurs...

[djidane39]

bon, ben j'ai plus qu'à choisir ^_^
en tout cas merci à vous tous!
Question bonus: je crois que en 1.6 les switch sur les enum ça passe???
sinon je crois que je vais choisir les enum, qui me paraisse le plus simple, le moins lourd et le plus efficace. Par contre pour la sérialisation en effet ça risque de me poser un problème.... Je vais étudié ça

[adiGuba]

Question bonus: je crois que en 1.6 les switch sur les enum ça passe???

Oui ! Les switch passe sur les enums même en 1.5... C'est juste sur les classes persos que cela ne marche pas...

sinon je crois que je vais choisir les enum, qui me paraisse le plus simple, le moins lourd et le plus efficace. Par contre pour la sérialisation en effet ça risque de me poser un problème.... Je vais étudié ça

Si tu es en Java > 1.5 alors il n'y a pas à réfléchir trop longtemps, les enums sont fait pour cela et gère très bien tout ceci... y compris la sérialisation !

a++