Fermeture connexion TCP et exception

Version imprimable

J'ai une application qui utilise une connexion TCP.
Je souhaite que : lorsqu'il y a une erreur au niveau de la connexion, celle ci soit fermée.

Mais je me retrouve avec un code assez aberrant, avec des try catch imbriqués :

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
 
    public void run()
    {
        try
        {
            while (true) //boucle de réception
            {
                Action action = this.connection.receiveAction(); //réception
 
                if (action != null)
                {
                    this.action(action); //traitement de l'objet reçu
                }
            }
        }
        catch (IOException e) //si jamais ça plante
        {
            e.printStackTrace();
            try
            {
                this.connection.close(); //on ferme la connexion
            }
            catch (IOException e1)
            {
                e1.printStackTrace(); //plantage de la fermeture de connexion ???
            }
        }
    }

(mon objet "connection" encapsule une connexion TCP)

Connaissez vous un moyen élégant de traiter ce genre de cas ?

Merci !

C'est moins un problème d'élégance que de nécessité.
Quoi qu'il en soit, c'est mieux de faire comme ça :

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
public void run()
{
        try
        {
            while (true) //boucle de réception
            {
                Action action = this.connection.receiveAction(); //réception
 
                if (action != null)
                {
                    this.action(action); //traitement de l'objet reçu
                }
            }
        }
        catch (IOException e) //si jamais ça plante
        {
        	e.printStackTrace();
        }
        finally
        {
            //Pour être sûr de toujours fermer la connection.
            try
            {
                this.connection.close(); //on ferme la connexion
            }
            catch (IOException e1)
            {
                e1.printStackTrace(); //plantage de la fermeture de connexion ???
            }
       }
}

02/12/2009, 01h58
Invité

ok merci !
je vais revoir mon code en fonction de tes conseils

mais ça me tracasse toujours ce "try catch" dans un "catch"
02/12/2009, 02h01
ngpub

C'est effectivement un peu lourd mais dans un cas comme ça si on veut gérer les choses correctement il est difficile de faire autrement.

Salut,

On peut utiliser un try/finally dans un try/catch, ce qui permet d'avoir une gestion unique des exceptions :

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
	public void run()
	{
	        try
	        {
	        	// INITIALISATION
	        	try {
		            while (true) //boucle de réception
		            {
		                Action action = this.connection.receiveAction(); //réception
 
		                if (action != null)
		                {
		                    this.action(action); //traitement de l'objet reçu
		                }
		            }
	        	} finally {
	        		this.connection.close();
	        	}
	        }
	        catch (IOException e) //si jamais ça plante
	        {
	        	e.printStackTrace();
	        }
	}

De plus en créant/ouvrant la connexion à l'emplacement indiqué par le commentaire "INITIALISATION", tu peux limiter son scope et donc éviter toute consommation abusive de la mémoire...

a++

02/12/2009, 11h41
Invité

ça me plait un peu plus comme solution :)
merci !