Discussion :

[nadsky]

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 :

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

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

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

[adiGuba]

Salut,


Surveilles bien tes accolades et l'indentation :

Code :

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

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 :

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
		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());
		}

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

[nadsky]

C'est exactement cela

merci beaucoup pour la rapidité et la clarté de ta réponse

[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

[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 :

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

1
2
3
if (true) throw new Exception();
 
System.out.println("Hello World");

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

[DevTeam_]

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