Discussion :

[Djobird]

Bonjour, voila quelques temps que je me pose cette question, et j'en viens à me demander si je sais vraiment utiliser finally : Quel est l'intéret du bloc finally ?

Quelle différence entre faire

Code :

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

1
2
3
4
5
6
7
8
9
10
 
try {
       inputStream.open();
} catch(Exception e) {
       e.printStackTrace();
} 
finally  {
      inputStream.close();
} 
blabla

Et faire :

Code :

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

1
2
3
4
5
6
7
8
9
 
try {
       inputStream.open();
} catch(Exception e) {
       e.printStackTrace();
} 
 
inputStream.close();
blabla;

C'est un simple problème de clarté de code, ou j'ai raté un truc ?

[Kanithael]

Comme tu as du le lire ailleurs, le code contenu dans finally sera toujours exécuté, même si il y a une exception. Ce n'est pas le cas du reste de la méthode !!

Dans ton premier exemple, ton inputStream sera fermé (bon exemple, c'est typiquement un cas d'utilisation), mais blabla ne sera pas exécuté.

Dans ton second exemple, ni ton inputStream, ni blabla ne sera exécuté, ce qui pourra laisser un flux vers un fichier ouvert (par exemple), et donc en interdire un accès ultérieur.

[Djobird]

Bah si, le reste sera exécuté, vu que c'est en dehors du bloc try ...

[ancrou]

Vu ton catch, le blabla sera executé.
Mais selon ce dont tu mets dans don Catch comme un throw ... le blabla ne sera pas exécuté, sauf si tu le mets dans un bloc Finally.

SUIS PAS SÛR

[Astartee]

Là tu as un catch Exception qui ne fait "rien" (dans le sens où toutes les exceptions sont masquées par cette clause catch, et qu'aucune exception ne sera levée, et que donc l'instruction suivante sera forcément exécutée).
Donc effectivement, dans ce cas précis, les deux sont équivalents.


Autres contextes :

Code :

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

1
2
3
4
5
6
7
8
9
10
try {
  // traitement normal
  // traitement final (ex : fermeture des flux)
}
catch (Exception1 e) {
  // traitement de l'exception Exception1
}
catch (Exception2 e) {
  // traitement de l'exception Exception2
}

En cas de levée d'une exception pendant le traitement normal, le traitement final ne sera jamais exécuté...

Code :

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

1
2
3
4
5
6
7
8
9
10
try {
  // traitement normal
}
catch (Exception1 e) {
  // traitement de l'exception Exception1
}
catch (Exception2 e) {
  // traitement de l'exception Exception2
}
// traitement final (ex : fermeture des flux)

En cas de levée d'une exception Exception1 ou Exception2 pendant le traitement normal, le traitement final sera effectivement exécuté...
MAIS
- si en fait dans le traitement de l'exception Exception1 on relance l'exception (avec un traitement du type e.printStackTrace() ; throw e; qui se contente de logguer l'erreur avant de la relancer) ? le traitement final ne sera jamais exécuté...
- si une exception Exception3 non prévue dans le code est levée ? le traitement final ne sera jamais exécuté...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
try {
  // traitement normal
}
catch (Exception1 e) {
  // traitement de l'exception Exception1
}
catch (Exception2 e) {
  // traitement de l'exception Exception2
}
finally {
  // traitement final (ex : fermeture des flux)
}

En cas de levée d'une exception pendant le traitement normal, le traitement final sera TOUJOURS exécuté même si une exception est lancée dans les clauses catch, même si c'est une exception non prévue dans le code qui intervient.

Code :

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

1
2
3
4
5
6
try {
  // traitement normal
}
finally {
  // traitement final (ex : fermeture des flux)
}

En cas de levée d'une exception pendant le traitement normal, le traitement final sera TOUJOURS exécuté avant de transmettre l'exception au niveau supérieur.


Et enfin

Code :

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

1
2
3
4
5
6
try {
  return 0;
}
finally {
  // traitement final (ex : fermeture des flux)
}

Le traitement final sera TOUJOURS exécuté avant de transmettre la valeur retournée (ici 0) au niveau supérieur

[Djobird]

Aaaaaah ok, il y avait en effet des détails que j'avais raté. Merci beaucoup !

[adiGuba]

Salut,


A noter qu'il est préférable d'utiliser un try/finally dans un bloc try/catch, afin d'avoir une gestion unique des exceptions


Comment libérer proprement les ressources (ou comment utiliser proprement les bloc try/finally) ?


a++

[virulent]

A noter juste que le finally n'est jamais executé si dans le catch il y a un System.exit ......

Et oui toutes les règles ont une exception...

[jamesleouf]

j'ai une autre question:

Je pensais pourtant que le try/catch servait a interrompre l'exécution;
OK le finally va s'exécuter mais pourquoi le reste qui suit s'exécute aussi?
Je pensais pourtant que le but du catch était d'interrompre l'exécution (sauf si dans le bloc de l'exception on lui dit de faire quelque chose pour continuer autrement l'exécution?

[adiGuba]

Ben le catch interrompt l'exécution du bloc try correspondant...

a++

[jamesleouf]

mais la suite est exécutée?

[Astartee]

mais la suite est exécutée?

"la suite" = ce qui est après le bloc try-catch-finally ?
Eh bien ça dépend de si une exception est levée ou non dans ce bloc...

[virulent]

Bonjour jamesleouf:

La suite est exécutée sauf s'il y a propagation de l'erreur....
si dans ton catch tu fais par exmple:

Code :

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

1
2
3
4
5
6
7
8
9
 
 
try{
 
}catch(Exception e){
     throw new Exception();
}
 
System.out.println( );

Ton System.out ne sera jamais exécuté.
Si tu le met dans un finally il le sera.

[jamesleouf]

Merci tout le monde