Discussion :

[Christophe Charron]

Bonjour,

je débute totalement en java et j'ai un souci pour intercepter une exception sur une collection inexistante. Je voudrais donc savoir si je procède correctement.

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
import android.net.*;
import java.lang.Exception;
// Retourne le nombre d'octets téléchargés par l'application
public static String DonneTraficTéléchargement(){
	String result;
	try{
		TrafficStats tStats = new TrafficStats();
		result = Long.toString(tStats.getUidRxBytes(android.os.Process.myUid()));
		if (result.equals("-1")){
			return "0";
		}
		else{
			return result;
		}
	} 
	catch(Exception e) {
		return "0";
	} 
}

TrafficStats n'étant apparue qu'en version 2.3 d'Androïd, je plante légitimement en faisant un test sur Androïd 2.1 update.
Mais je plante bestialement !! Je voudrais trapper l'erreur et passer outre(ne pas sortir brutalement.

D'avance, merci pour vos éclaircissements.

[thelvin]

- "Exception sur collection" ? C'est censé vouloir dire quelque chose ?

- Ici le problème est que tu essaies d'utiliser une classe qui n'existe pas dans ta version de Java. En chargeant ta classe il doit d'abord charger la classe TrafficStats, qui n'existe pas, donc le chargement de ta classe échoue.
Si la classe ne se charge pas, il n'y a pas la moindre chance d'appeler sa méthode DonneTraficTéléchargement()*, et donc ce n'est pas d'entourer ça de try / catch qui va résoudre quoi que ce soit.

Je ne sais pas pour Android, mais en Java classique, la solution c'est la réflexivité. Plutôt que d'utiliser au compile-time la classe TrafficStats, chercher si elle existe au runtime.
Utiliser

Code :

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

Class<?> trafficStatClass = Class.forName("android.net.TrafficStats")

pour vérifier si elle existe et si oui, mettre la main sur sa Class.
À partir de là, je suppose que tu sais lire la JavaDoc de Class et que tu as pas besoin que je t'explique ce que ça fait.

Ensuite tu récupères par réflexivité la méthode à appeler sur cette classe :

Code :

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

Method uidRxBytesMethod = trafficStatClass.getMethod("getUidRxBytes", Integer.TYPE);

puis tu appelles cette méthode. Elle est statique, donc on peut donner null à la place de l'instance sur laquelle l'appliquer.

Code :

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

uidRxBytesMethod.invoke(null, uid);

Ce qui renvoie un objet Long, qui, si j'ai bien compris, vaudra -1 si l'opération n'est pas gérée.

Bien sûr, plusieurs choses peuvent échouer en tentant de faire ça, notamment la classe TrafficStats qui n'existe pas. Du coup c'est effectivement une bonne idée d'entourer tout ça de try / catch, cette fois-ci, puisque le bloc try sera parcouru, ça marchera.

* au fait, un nom de méthode devrait commencer par une minuscule. Question d'usage et de compatibilité avec les systèmes à introspection. (Et je conseille pas spécialement de mettre des accents, mais je suppose que ça c'est pour l'exemple.)

[Uther]

Détecter se genre d'exception est assez sale.

Je ne connais pas le SDK d'android mais il doit forcément y avoir une méthode pour récupérer le numéro de version et ne pas utiliser cette fonctionnalité le cas échéant.
Ça serait beaucoup plus propre.

[nicroman]

Je trouve ca très propre, c'est de la pure reflexivité....

Class<?> trafficStatClass = Class.forName("android.net.TrafficStats");

lancera une exception si la classe n'est pas disponible... basta...
apres, les autres exceptions ne devraient pas être catchées....

[Uther]

Ma réponse était destinée à "Christophe Charron" pas à "thelvin". Nos réponses se sont croisées.
La méthode de thelvin est en effet bonne, bien que j'essaie d'éviter au maximum l'utilisation de l'introspection.

[Christophe Charron]

Bonjour,

pardon pour ma réponse tardive, par rapport à ta hâtive solution.
C'est exactement ce que je cherchais. Je lève bien une exception si la classe n'existe pas, exception que je traite.

- "Exception sur collection" ? C'est censé vouloir dire quelque chose ?

- Ici le problème est que tu essaies d'utiliser une classe qui n'existe pas dans ta version de Java. En chargeant ta classe il doit d'abord charger la classe TrafficStats, qui n'existe pas, donc le chargement de ta classe échoue.
Si la classe ne se charge pas, il n'y a pas la moindre chance d'appeler sa méthode DonneTraficTéléchargement()*, et donc ce n'est pas d'entourer ça de try / catch qui va résoudre quoi que ce soit.

Je ne sais pas pour Android, mais en Java classique, la solution c'est la réflexivité. Plutôt que d'utiliser au compile-time la classe TrafficStats, chercher si elle existe au runtime.
Utiliser

Code :

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

Class<?> trafficStatClass = Class.forName("android.net.TrafficStats")

pour vérifier si elle existe et si oui, mettre la main sur sa Class.
À partir de là, je suppose que tu sais lire la JavaDoc de Class et que tu as pas besoin que je t'explique ce que ça fait.

Ensuite tu récupères par réflexivité la méthode à appeler sur cette classe :

Code :

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

Method uidRxBytesMethod = trafficStatClass.getMethod("getUidRxBytes", Integer.TYPE);

puis tu appelles cette méthode. Elle est statique, donc on peut donner null à la place de l'instance sur laquelle l'appliquer.

Code :

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

uidRxBytesMethod.invoke(null, uid);

Ce qui renvoie un objet Long, qui, si j'ai bien compris, vaudra -1 si l'opération n'est pas gérée.

Bien sûr, plusieurs choses peuvent échouer en tentant de faire ça, notamment la classe TrafficStats qui n'existe pas. Du coup c'est effectivement une bonne idée d'entourer tout ça de try / catch, cette fois-ci, puisque le bloc try sera parcouru, ça marchera.

* au fait, un nom de méthode devrait commencer par une minuscule. Question d'usage et de compatibilité avec les systèmes à introspection. (Et je conseille pas spécialement de mettre des accents, mais je suppose que ça c'est pour l'exemple.)

[Christophe Charron]

Bonjour,

pour ma culture personnelle, en quoi lever une exception est-il plus "propre" que tester les multiples versions ? Je considère ce choix plus comme un mode de programmation plus générique, non ?

Ma réponse était destinée à "Christophe Charron" pas à "thelvin". Nos réponses se sont croisées.
La méthode de thelvin est en effet bonne, bien que j'essaie d'éviter au maximum l'utilisation de l'introspection.