Class not found : intercepter l'erreur

Version imprimable

29/01/2009, 10h01
SheikYerbouti

Class not found : intercepter l'erreur

Bonjour,
Dans une application, j'autorise la lectur de fichiers sons au format MP3 via l'ajout de JLayer 1.0.
Cela sous-entend l'ajout de jl1.0.jar.
Cependant, certains utilisateurs, non intéressés, n'installeront pas ce JAR supplémentaire.
A l'exécution de l'appli, on récupère donc un Class not found exception et l'appli plante (même si la fonction MP3 n'est pas utilisée).
Existe t-il un moyen d'intercepter cette erreur et de permettre à l'application de continuer ?

D'avance meci.
29/01/2009, 10h21
Alkhan
bonjour,

étant donné que l'application ne possède pas forcément le jl1.0.jar, j'imagine que tu doit passé par un Class.forName.
Cependant vu ta question j'ai un doute.

sinon tu dois avoir ca :
Code:

1 2 3 4 5 6 try { Class.forName("Classquinestpasdanslepath"); } catch (ClassNotFoundException e) { System.out.println("Class non trouvée"); }
Si tu n'est pas dans ce cas la (par exemple si tu instancies directement les classe du jl1.0.jar) c'est que tu as un problème de conception, car dans la phase de développement, tu doit prendre en compte le fait que potentiellement ce jar n'est pas dans le path.
Dans ce cas, soit tu désactive les appels aux fonctionnalités qui utilise ces classes, soit tu fais de la réflexion pour appeler ces classes (l'exemple avec le class.forName ci dessus rentre dans ce cadre).
29/01/2009, 10h31
SheikYerbouti

Le problème est que je ne tente même pas pas d'accéder à ces fonctions. C'est dès le chargement de l'appli que le message surgit.
Cela serait-il du à l'import ?

Code:

import javazoom.jl.player.Player;
29/01/2009, 10h39
Alkhan

oui c'est bien ce que je pensais, c'est pour cela que tu dois plutôt envisager de faire de la reflexion et ne pas import et instancier directement ces classes, ou sinon tu imposes le fait que ce jar doit être dans le classPath.

Si tu ne connais pas la réflexion va voir ici
29/01/2009, 10h49
SheikYerbouti

Merci pour le tuyau.
Cependant, j'ai besoin d'une variable globale à la classe:

Code:

private javazoom.jl.player.Player player;

Cela peut-il être pris en compte par la reflexion ?
29/01/2009, 11h14
Alkhan
tu ne peut pas avoir de référence à la classe directement, donc ton instance doit être un "Object"

Comme ça :
Code:

1 2 private Object player;
29/01/2009, 11h17
SheikYerbouti

Ok, donc il faudra que j'utilise les méthodes de façon dynamique.
Merci, je regarde tout ça. 8-)
29/01/2009, 11h45
Invité

Beeeeerk. :?
Utiliser la réflexion dans ce cas ça s'apparente à écraser une mouche avec une enclume.

Normalement si toutes les références (import, new, etc.) au contenu de jl1.0.jar sont à l'intérieur de classes qui ne sont jamais appelées (parce que l'utilisateur ne se sert pas de la fonctionnalité), tu ne devrais pas obtenir d'exception ClassNotFound (je suis pas sûr à 100%, mais ça me semblerait logique).

Personnellement je préconiserais que tu revoies un peu ton architecture parce que si tu utilises la réflexion ici tu vas cumuler réflexion (qui ajoute beaucoup de complexité et de confusion au code) + architecture potentiellement mal conçue et ton appli va devenir impossible à maintenir.

Après on a pas tous les mêmes priorités dans le développement. Si la refonte de l'architecture s'avère trop lourde et que t'as des délais à tenir...

Mais quand même spa joli :P
29/01/2009, 12h05
SheikYerbouti

Il suffit de la déclaration d'une variable dans la classe pour générer l'exception:

Code:

private javazoom.jl.player.Player player;
29/01/2009, 13h22
SheikYerbouti

Ok, c'est parfait. Merci bien. ;)
29/01/2009, 14h15
Invité

Je sais bien qu'une simple déclaration de variable provoque l'exception, mais si ta déclaration de variable et tes appels de méthode sont toutes dans des classes bien isolées qui ne seront utilisées que par des utilisateurs disposant du bon jar normalement tu ne devrais avoir aucune exception.
29/01/2009, 14h19
SheikYerbouti

Citation:

Envoyé par kei2906

... classes bien isolées qui ne seront utilisées que par des utilisateurs disposant du bon jar...

Et comment je sais que le JAR a bien été installé ? puisque c'est justement le problème...
29/01/2009, 14h49
Invité

Si j'ai bien compris tu as deux cas de figure :

1) Certains utilisateurs utilisent toute l'application. Ceux-là installent le jar de jl. Ils savent que leur jar est bien installé parce que quand ils utiliseront la fonctionnalité... elle fonctionnera correctement (sans générer d'exception).

2) Certains utilisateurs décident qu'ils n'auront pas besoin de la lecture de sons, donc pas besoin du jar. Ceux-là ne l'installent pas et comme ils n'utiliseront pas la fonctionnalité correspondante, les classes de ton application qui font appel au contenu de ce jar ne seront (a priori, si c'est bien compartimenté) jamais instanciées et ne lèveront jamais de ClassNotFoundException.

Un solution pas très élégante mais très efficace est de faire :
Code:

1 2 3 4 5 6 try { Class.forName("javazoom.jl.player.Player"); } catch (ClassNotFoundException e) { // le jar n'est pas dans le path }

Salut,

Je vote également pour revoir un peu la conception...

Si la classe javazoom.jl.player.Player est optionnel tu ne devrais pas la manipuler directement, car sinon cela peut t'obliger à devoir gérer plusieurs cas particulier...

Perso je me définirais une interface représentant mon player Audio, par exemple :

Code:

1
2
3
4
5
6
7
interface AudioPlayer {
 
	public void play();
 
	public void close();
 
}

:arrow: C'est cette interface que j'utiliserais alors dans mon code.

Ainsi pour utiliser Javazoom il me suffirait de créer une classe qui déléguerait les appels :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
import javazoom.jl.player.Player;
 
class JavaZoomAudioPlayer implements AudioPlayer {
 
	final Player player;
 
	public JavaZoomAudioPlayer(InputStream source) {
		this.player = new Player(source);
	}
 
	@Override
	public void close() {
		this.player.close();
	}
 
	@Override
	public void play() {
		this.player.play();
	}
}

Bien entendu tu ne pourras utiliser cette classe QUE dans le cas où le jar de Javazoom est bien chargé, c'est pourquoi tu ne devras pas instancier cette classe directement, mais passer par une fabrique qui gérerai tout cela, par exemple :

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
class Players {
 
	/**
         * Une classe "dummy" qui ne fait rien, et qui sera utilisé
         * dans les cas où l'on ne trouve aucun player valide :
         */
	private static final AudioPlayer DUMMY = new AudioPlayer() {
		@Override
		public void close() {
			// empty
		}
 
		@Override
		public void play() {
			// empty
		}		
	};
 
 
	/** Indique la présence de Javazoom dans le classpath : */
	private static final boolean JAVAZOOM_PRESENT = javazoomPresent();
 
 
	/**
         * Vérifie la présence de Javazoom dans le classpath,
         * en recherchant la classe javazoom.jl.player.Player
         * @return true si la classe est présente
         */
	private static boolean javazoomPresent() {
		try {
			Class.forName("javazoom.jl.player.Player");
			return true;
		} catch(Exception e) {
			// Log ?
			return false;
		}
	}
 
	/**
         * Retourne le player adapté selon les cas :
         */
	public static AudioPlayer getPlayer(String fileName) throws IOException {
		// Si javazoom est bien présent :
		if (JAVAZOOM_PRESENT) {
			// On utilise le player de JavaZoom
			return new JavaZoomAudioPlayer(new FileInputStream(fileName));
		}
		// Sinon on renvoi une implémentation vide :
		return DUMMY;
	}
}

a++

29/01/2009, 17h24
SheikYerbouti

J'ai mis en place la reflexion pour tester si la classe peut être chargée et également pour exécuter les méthodes - method.invoke() - et cela fonctionne parfaitement. Si le JAR n'est pas indiqué dans le class path, je peux afficher dans la console un sympathique message sans planter l'appli ;)