Discussion :

[Invité]

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 :

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

[ngpub]

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 :

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
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 ???
            }
       }
}

[Invité]

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

mais ça me tracasse toujours ce "try catch" dans un "catch"

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

[adiGuba]

Salut,


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

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

[Invité]

ça me plait un peu plus comme solution
merci !