Discussion :

[cyrille37]

Salut

J'ai une enum et je souhaiterais affecter une variable de ce type à partir d'une valeur int.
A part un switch je n'ai pas trouvé d'autre syntaxe. Il doit bien y avoir mieux ?!
Dans la doc il est interdit d'utiliser de constructeur public pour les enums.

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
 
public static enum ProtocoleVersion
{
 VERSION_ONE(1), VERSION_TWO(2);
 int _version ;
 ProtocoleVersion(int version)
 {
  _version = version ;
 }
}
 
private ProtocoleVersion _protocoleVersion ;
 
public void setProtocole( int protocoleVersion )
{
 switch( protocoleVersion )
 {
  case 1:
   _protocoleVersion = ProtocoleVersion.VERSION_ONE ;
   break ;
  case 2:
   _protocoleVersion = ProtocoleVersion.VERSION_TWO ;
   break ;
  default:
   throw new Exception("Invalid protocole version");
 }
}

Je trouve pas comment faire mieux que le contenu de la méthode void setProtocole( int protocoleVersion ).

Merci pour vos idées,
cyrille

[jhaythem]

Salut

J'ai une enum et je souhaiterais affecter une variable de ce type à partir d'une valeur int.
A part un switch je n'ai pas trouvé d'autre syntaxe. Il doit bien y avoir mieux ?!
Dans la doc il est interdit d'utiliser de constructeur public pour les enums.

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
 
public static enum ProtocoleVersion
{
 VERSION_ONE(1), VERSION_TWO(2);
 int _version ;
 ProtocoleVersion(int version)
 {
  _version = version ;
 }
}
 
private ProtocoleVersion _protocoleVersion ;
 
public void setProtocole( int protocoleVersion )
{
 switch( protocoleVersion )
 {
  case 1:
   _protocoleVersion = ProtocoleVersion.VERSION_ONE ;
   break ;
  case 2:
   _protocoleVersion = ProtocoleVersion.VERSION_TWO ;
   break ;
  default:
   throw new Exception("Invalid protocole version");
 }
}

Je trouve pas comment faire mieux que le contenu de la méthode void setProtocole( int protocoleVersion ).

Merci pour vos idées,
cyrille

pourquoi un switch te derange autant?

[cyrille37]

Bon j'ai ajouté une méthode create(int) dan l'enum et là je suis satisfait ;-)

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
public static enum ProtocoleVersion
{
 VERSION_ONE(1), VERSION_TWO(2);
 int _version ;
 ProtocoleVersion(int version)
 {
  _version = version ;
 }
 public static ProtocoleVersion create( int protocoleVersion )
 {
  switch( protocoleVersion )
  {
   case 1:
    return ProtocoleVersion.VERSION_ONE ;
   case 2:
    return ProtocoleVersion.VERSION_TWO ;
  }
  throw new Exception("Invalid protocole version");
 }
}
 
private ProtocoleVersion _protocoleVersion ;
 
public void setProtocole( int protocoleVersion ) throws Exception
{
   _protocoleVersion = ProtocoleVersion.create(protocoleVersion) ;
 }
}

[adiGuba]

Salut,

C'est normal que tu ne puisses pas utiliser le constructeur d'enum puisque les valeurs des enums sont obligatoirement unique : tu ne peux pas dupliquer des valeurs !

Les enums ont implicitement une numéro d'ordre correspondant à leurs ordre de déclaration, que tu peux éventuellement utiliser ici (même s'il commence à zéro) :

Code :

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

1
2
3
4
5
6
7
enum ProtocoleVersion {
	VERSION_ONE, VERSION_TWO;
 
	public static ProtocoleVersion create(int protocoleVersion) throws ArrayIndexOutOfBoundsException {
		return ProtocoleVersion.values()[protocoleVersion-1];
	}
}

Sinon si tu veux vraiment utiliser un attributs (par exemple si tes numéro de versions ne se suivent pas), je te conseillerais déjà d'utiliser un attribut final (pour éviter toute mauvaise manip dans le code de ton enum). Ensuite il serait plus simple de remplacer le switch par une boucle sur le tableau des valeurs :

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
enum ProtocoleVersion {
	VERSION_ONE(1), VERSION_TWO(2);
	final int _version;
 
	ProtocoleVersion(int version) {
		_version = version;
	}
 
	public static ProtocoleVersion create(int protocoleVersion) throws Exception {
		for (ProtocoleVersion pv : ProtocoleVersion.values()) {
			if (pv._version == protocoleVersion) {
				return pv;
			}
		}
		throw new Exception("Invalid protocole version");
	}
}

Le code n'a pas à être modifié si tu rajoutes une valeur dans l'enum

a++

[cyrille37]

Super !
C'est exactement une idée comme celle-ci que je cherchais.
thank you very much.
Cyrille

[bouye]

Reste que je ne pige toujours pas l'interret de cette maniere de faire/conception :
- d'une part la valeur de l'enum est unique et donc suffit pour un switch.
- d'autre part si jamais ce n'etait pas suffisant la methode ordinal() retourne l'indice de la valeur de l'enum et donc aussi suffit largement pour un switch.

La methode indiquee par adiGuba est effectivement la bonne demarche a suivre mais dans ce cas autant directement surcharger la methode valueOf().

[cyrille37]

dans ce cas autant directement surcharger la methode valueOf().

Merci !
En voilà de bonnes idées.
Décidement je ne regrète pas d'avoir posé la question ;-)

Merci
Cyrille

[adiGuba]

Reste que je ne pige toujours pas l'interret de cette maniere de faire/conception :
- d'une part la valeur de l'enum est unique et donc suffit pour un switch.
- d'autre part si jamais ce n'etait pas suffisant la methode ordinal() retourne l'indice de la valeur de l'enum et donc aussi suffit largement pour un switch.

L'utilisation d'un attribut n'a pas pour objectif de remplacer ordinal(), mais plutôt d'associer une valeur à l'enum.
En comparaison avec ordinal() cela apporte plusieurs avantages :

• On peut utiliser des valeurs non-non-consécutives et pas forcément ordonnée.
• On peut utiliser n'importe quel type de donnée.
• On peut dupliquer des valeurs (deux enums différentes pourraient très bien avoir la même valeur logique)

cela permet de gérer cela au niveau de l'enum et pas pas lors de son utilisation...


a++