Try catch finally, flux best practice

**LittleBean** · 13/08/2009, 11h20

Bonjour à tous,

Mon problème est le suivant : j'ai deux flux +du code métier générant une exception, qqchose comme ça :

1 - création flux 1
2 - création flux 2
3 - code métier générant une "ApplicationException"
4 - fermeture flux 1
5 - fermeture flux 2

Comment est ce que je peux faire pour fermer ça proprement sans faire d'imbrication de try catch finally ??

Passer par une sous méthode?

un peu de code pour illustrer :

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
public class AEAE {
 
	/**
         * @param args
         * @throws Exception 
         */
 
	public static void main(String[] args) throws Exception {
		// TODO Auto-generated method stub
 
 
		System.out.println(1);
		try {
			System.out.println(2);
			ex();
			System.out.println(3);
		} catch (Exception e) {
			System.out.println(4);
		} finally {
			System.out.println(5);
			close();
			System.out.println(5.1);
			close();
			System.out.println(5.2);
		}
 
		System.out.println(6);
 
	}
 
 
	public static String ex() throws Exception{
		throw new RuntimeException();
	}
	public static String close() throws Exception{
		throw new RuntimeException();
	}
}

res :

1
2
4
5
Exception in thread "main" java.lang.RuntimeException
at AEAE.close(AEAE.java:37)
at AEAE.main(AEAE.java:22)

Puis je avoir 5.1 et 5.2 sans rajouter de try?

merci

**Yomhgui** · 13/08/2009, 11h44

Qu'est-ce qui te pose problème dans le fait de faire un try {...} finally {...} ?

Point important, tu n'as pas besoin de mettre un bloc catch:

Code :

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

1
2
3
4
5
6
7
 
try {
    stream.open();
    callBusinessCode(stream);
} finally {
    stream.close();
}

Le comportement sera que, en cas d'exception dans le bloc try, le contexte de l'exception sera sauvegardé et le branchement sera fait dans le bloc finally qui sera exécuté. Ensuite, le contexte de l'exception sera réutilisé pour relancer l'exception.

Est-ce que ça ne résoudrait pas ton dilemme ?

**adiGuba** · 13/08/2009, 11h57

Salut,

A mon avis le plus simple et le plus sûr est d'utiliser un try/finally par flux, et éventuellement un try/catch global pour la gestion des exceptions...

Quelque chose comme cela :

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
	try {
		InputStream in = new FileInputStream("in");
		try {
			OutputStream out = new FileOutputStream("out");
			try {
				// TRAITEMENTS
			} finally {
				out.close();
			}
		} finally {
			in.close();
		}
	} catch (IOException e) {
		e.printStackTrace();
	}