Discussion :

[spyke brain]

Bonjour,
j'aimerais faire un case dans mon enum hors le type de l'enum n'etant pas un string n'est pas reconnu par mon case.
Y aurait-il un autre moyen ?
Voici le bout de code :

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 enum RouteType {
		DEFAULT(null),
		AVOID_HIGHWAYS("highways"),
		AVOID_TOLLS("tolls"),
		AVOID_FERRIES("ferries"),
		PUBLIC_TRANSIT("transit"),
		WALKING("walking"),
		BICYCLING("bicycling");
		private String param;
		private RouteType(String param) { this.param = param; }
		public String getParam() { return param; }
		public String toParam() { 
			switch(this.param)
			{
			case null:
				return "";
				break;
			case AVOID_HIGHWAYS: case AVOID_TOLLS: case AVOID_FERRIES:
				return new StringBuilder("&avoid=").append(this.param).toString();
				break;
			case PUBLIC_TRANSIT: case WALKING: case BICYCLING:
				return new StringBuilder("&mode=").append(this.param).toString();
				break;
				default:
					break;
			}
		}
	}

PS : c'est pour utiliser le web service de google maps

[adiGuba]

Salut,


Le switch est de type String (this.param) alors que les case contiennent des enums RouteType...
Un switch(this) devrait faire l'affaire...


Sinon pourquoi utiliser StringBuilder ici ? Cela alourdit inutilement le code...

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 enum RouteType {
		DEFAULT(null),
		AVOID_HIGHWAYS("highways"),
		AVOID_TOLLS("tolls"),
		AVOID_FERRIES("ferries"),
		PUBLIC_TRANSIT("transit"),
		WALKING("walking"),
		BICYCLING("bicycling");
		private String param;
		private RouteType(String param) { this.param = param; }
		public String getParam() { return param; }
		public String toParam() { 
			switch(this.param)
			{
			case DEFAULT:
				break;
			case AVOID_HIGHWAYS: case AVOID_TOLLS: case AVOID_FERRIES:
				return "&avoid=" + this.param;
				break;
			case PUBLIC_TRANSIT: case WALKING: case BICYCLING:
				return "&mode=" + this.param;
				break;
			default:
				break;
			}
			return "";
		}
	}

a++

[spyke brain]

Un switch(this) me ferait changer le type de retour de ma méthode.

[tchize_]

y a pas de raison!

[spyke brain]

et pourtant..

[thelvin]

Amusant.

Tu peux nous montrer ton code après que tu aies été obligé de changer le type de retour ? Je suis curieux de savoir comment tu fais pour en arriver là, sans vouloir en arriver là et sans raison d'en arriver là.

[spyke brain]

Enfaite si jamais j'ecrit switch(this)
comme ici :

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
public String toParam() { 
			switch(this)
			{
			case DEFAULT:
				return "";
				break;
			case AVOID_HIGHWAYS: case AVOID_TOLLS: case AVOID_FERRIES:
				return new StringBuilder("&avoid=").append(this.param).toString();
				break;
			case PUBLIC_TRANSIT: case WALKING: case BICYCLING:
				return new StringBuilder("&mode=").append(this.param).toString();
				break;
				default:
					break;
			}
		}

Eclipse me demande de changer le type de retour de ma fonction.

[thelvin]

Tiens c'est marrant moi il me dit beaucoup de choses, mais rien qu'on pourrait croire avoir un rapport avec le type de retour de la méthode.
On peut voir les messages d'erreur et lignes concernées ?


... 'Fin bon.

• Pas de case null. Remplace par default si tu veux.
• Pas d'instruction après un return : les breaks sont inutiles puisque la méthode est déjà finie.
• La méthode doit forcément se terminer sur un return. Tu n'as pas le droit d'atteindre la fin du switch et ne rien faire ensuite.

... Et pas le moindre changement de type de retour.

[spyke brain]

ligne 1 : This method must return a result of type String
et au niveau de chaque break : Unreachable code

désolé je ne suis pas un pro en java..

[thelvin]

ligne 1 : This method must return a result of type String

Donc un message qui te dit que tu n'as pas le droit de faire autre chose que renvoyer quelque chose du type de retour, String.
Il te le dit parce que ta méthode ne se termine pas toujours par un return, et donc ne fournit pas toujours de String à renvoyer. C'est interdit. Tu dois toujours atteindre un return d'une manière ou d'une autre.

et au niveau de chaque break : Unreachable code

Je vois pas le rapport avec le type de retour.
Il dit que tu as du code inatteignable. Il ne faut pas avoir de code inatteignable.

[spyke brain]

Ok j'ai compris ! C'est tout bete finalement.. Merci beaucoup.

ET pour le break, je ne savais pas qu'on pouvait ne pas les mettre sur un switch.

Merci beaucoup, tout marche !

[tchize_]

ET pour le break, je ne savais pas qu'on pouvait ne pas les mettre sur un switch.

On peut, et on doit mettre des break dans un switch, en général. Mais c'est une instruction comme une autre. Et on ne peux pas mettre d'instruction après un return