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...