unreachable code erreur

Version imprimable

Bonjour à tous,

Je travaille actuellement sur une application découpée en couches, et je suis en train d'écrire une méthode de service métier.
Seulement, une erreur m'intrigue et je n'arrive pas à trouver d'où elle provient :

Code:

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
39
40
41
42
43
44
45
46
47
48
public Integer save(ScalarHabilitation scalarHabilitation, ScalarPersonne scalarPersonne,
						ScalarSociete scalarSociete)//, Integer societeId)
	throws ServiceException, RegleGestionException, TechniqueException {
		IPersonneService personneService = ServiceFactory.getInstance().getPersonneService();
		ISocieteService societeService = ServiceFactory.getInstance().getSocieteService();
		Habilitation uneHabilitation = new Habilitation();
 
		// Si l'habilitation n'est pas nouvelle, elle doit être modifiable
		if (scalarHabilitation.getId() != 0) {
			Habilitation habilitationStockee = this.rechercherParId(scalarHabilitation.getId());
			{
			if (!this.isModifiable(habilitationStockee)){
				//, CezarSecuriteFonctionnelleManager.getInfosUser()){//societeId)) {
					}
				throw new AccessForbiddenException("Vous n'êtes pas habilité à modifier cette habilitation");
			}
 
		//Mise à jour de la personne
			if (uneHabilitation.getPersonne() == null) {
				uneHabilitation.setPersonne(new Personne());
			}
			scalarPersonne.saveToValueObject(uneHabilitation.getPersonne());
			personneService.save(uneHabilitation.getPersonne());
 
			// Si l'habilitation est nouvelle, la personne
 
			//Mise à jour de la société
			if (uneHabilitation.getEmployeur() != null) {
				scalarSociete.saveToValueObject(uneHabilitation.getEmployeur());
				societeService.save(uneHabilitation.getEmployeur());
			}
 
 
			//Enregistrement final de l'habilitation
			// Si habilitation nouvelle
			// Recherche compte habilitations valides pour cette personne
			// SI > 0 leve reglegestionexception "deja une en cours"
 
			if (scalarHabilitation.getId() == 0) {
				if (!personneService.isHabilitable(uneHabilitation.getPersonne().getId())) {
					throw new RegleGestionException(
							InitManager.getRegleGestion("rg.habilitationservice.personne_non_habilitable"));
				}
			}
		}
			this.save(uneHabilitation);
			return uneHabilitation.getId();
	}

Ce code me parait correct, seulement voila, ce morceau de code :

Code:

1
2
3
if (uneHabilitation.getPersonne() == null) {
				uneHabilitation.setPersonne(new Personne());
			}

est souligné et m'affiche comme erreur "unreachable code".

Pouvez - vous m'aider, svp??

Merci d'avance

Salut,

Surveilles bien tes accolades et l'indentation :

Code:

1
2
3
4
5
6
7
8
9
10
11
		Habilitation habilitationStockee = this.rechercherParId(scalarHabilitation.getId());
		{
		if (!this.isModifiable(habilitationStockee)){
			//, CezarSecuriteFonctionnelleManager.getInfosUser()){//societeId)) {
				}
			throw new AccessForbiddenException("Vous n'êtes pas habilité à modifier cette habilitation");
			}
				//Mise à jour de la personne
		if (uneHabilitation.getPersonne() == null) {
			uneHabilitation.setPersonne(new Personne());
		}

Une fois bien indenté, ce code devient :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
		Habilitation habilitationStockee = this
				.rechercherParId(scalarHabilitation.getId());
		{
			if (!this.isModifiable(habilitationStockee)) {
				// ,
				// CezarSecuriteFonctionnelleManager.getInfosUser()){//societeId))
				// {
			}
			throw new AccessForbiddenException(
					"Vous n'êtes pas habilité à modifier cette habilitation");
		}
 
		// Mise à jour de la personne
		if (uneHabilitation.getPersonne() == null) {
			uneHabilitation.setPersonne(new Personne());
		}

:arrow: Le throw est en dehors du if et donc exécuté quoi qu'il arrive : le code qui suit ne pourra donc jamais s'exécuter...

a++

19/05/2008, 11h16
nadsky

C'est exactement cela;)

merci beaucoup pour la rapidité et la clarté de ta réponse;);)
28/05/2008, 15h43
DevTeam_

A ce sujet, j'ai une question pour ma culture personnelle : s'agit-il d'une erreur fatale pour le compilateur (par là, je fais le distingo entre une vraie erreur que le compilateur n'accepte pas et ce qui apparait comme erreur dans Eclipse par exemple, mais qui n'est qu'une question de paramétrage de l'IDE) ?

Merci d'avance
28/05/2008, 16h02
adiGuba
Salut,

Oui le compilateur détecte ce type d'erreur et stoppe la compilation, même si on peut passer outre avec un "faux" test :
Code:

1 2 3 if (true) throw new Exception(); System.out.println("Hello World");
:arrow: Même si le Hello World ne pourra jamais être exécuté, le compilateur ne génèrera pas d'erreur car il n'évalue pas la condition.

Notez toutefois que cela peut s'avérer pratique dans certain cas à des fins de tests ;)

a++
28/05/2008, 16h05
DevTeam_

Merci pour la réponse ! Rapide et concis, as usual :king: