Discussion :

[_LittleFlea_]

Bonjour à tous,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public int loadCounter(String filename) {
  FileReader reader = null;
  int counter;
  try {
    reader = new FileReader(filename);
    ...
    return counter;
  } catch(FileNotFoundException e) {
    throw new NotFoudnException("File " + filename, e);
  } finally {
    if( reader != null ) reader.close();
  }
}

est-il vraiment nécessaire de mettre les instructions "if( reader != null ) reader.close();" dans un bloc finally ?
si j'ai bien compris si une exception est levée dans le try, alors on execute les instructions du catch correspondant à l'exception, puis les instructions suivantes sont executées.

Je pense donc qu'ici le finally est obligatoire à cause du throw, ai-je raison ?

merci à vous

[professeur shadoko]

autre organisation possible : double try

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
 
public int loadCounter(String filename) {
  FileReader reader = null;
  int counter;
  try {
    reader = new FileReader(filename);
           try {
            ... /// code qui exploite reader
               return counter ;
            } finally {
                 reader.close();
            }  
  } catch(IOException e) {
    ///
  } 
 
}

[Jidefix]

Le finally est effectivement obligatoire à cause du throw, mais aussi à la fin de ton try tu fais un "return".
Donc si tu retires le finally et s'il n'y a pas d'exception, tu va aller jusqu'au bout de ton try, puis arrêter ta méthode, et ne passera donc pas dans ton reader.close();

[_LittleFlea_]

Le finally est effectivement obligatoire à cause du throw, mais aussi à la fin de ton try tu fais un "return".
Donc si tu retires le finally et s'il n'y a pas d'exception, tu va aller jusqu'au bout de ton try, puis arrêter ta méthode, et ne passera donc pas dans ton reader.close();

ok merci.

Et dans ce code là :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public int loadCounter(String filename) {
  FileReader reader = null;
  int counter;
 
  try {
   reader = new FileReader(filename);
   ...
  } catch(FileNotFoundException e) {
    // Renvoyer l'exception dans une exception implicite
    throw TechnicalException.rethrow("Cannot open counter file " + filename, e); 
  } finally {
    if( reader != null ) reader.close();
  }
  return counter;

le "return counter;" ne sera pas execute à cause du throw dans le catch ?
mais s'il n'y avait pas eu de throw dans le catch, "return counter;" aurait été alors executé (après les instructions du catch et du finally) ?
ai-je bien compris ?

[Jidefix]

le "return counter;" ne sera pas execute à cause du throw dans le catch ?
mais s'il n'y avait pas eu de throw dans le catch, "return counter;" aurait été alors executé (après les instructions du catch et du finally) ?
ai-je bien compris ?

1)oui puisque ta fonction va planter au niveau du rethrow qui n'est pas catché => la fonction exécute le/les éventuel(s) finally puis se termine avec l'exception

2)oui puisque l'exception aurait été catchée, tu serais sorti sans encombre du catch, aurait exécuté le finally, puis serait ensuite passé sur le return.

3)visiblement oui

[tchize_]

oui


en pratique, ce genre de code sensible (libérer les ressources système) sont mise systématiquement dans un bloc finally pour plusieurs raison

-> le bloc est garanti d'être exécuté (pas nécessairement dans son entièreté, ca dépend du contenu)
-> il est plus facile de mettre ça dans une bloc finally que de calculer le code principal pour etre sur que c'est toujours exécuté (un programmeur, c'est fainéant pas nature)
-> Il est possible que des exceptions runtime non prévues par le codeur remontent (nullpointerexception dans un traitement, unsupportedoperation ou autres)

[_LittleFlea_]

Merci à tous pour vos réponses.

Et si j'ai un programme de ce style :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
try{
   quelquechose = open(..);
 
   try{}
   catch(Exception e){
      throw e;
   }
 
}
catch{}
finally{quelquechose.close();}

quelquechose.close() sera-t'il executé si e est levée ?