Discussion :

[SheikYerbouti]

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.

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

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

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

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

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

import javazoom.jl.player.Player;

[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

[SheikYerbouti]

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

Code :

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

private javazoom.jl.player.Player player;

Cela peut-il être pris en compte par la reflexion ?

[Alkhan]

tu ne peut pas avoir de référence à la classe directement, donc ton instance doit être un "Object"

Comme ça :

Code :

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

1
2
 
private Object player;

[SheikYerbouti]

Ok, donc il faudra que j'utilise les méthodes de façon dynamique.
Merci, je regarde tout ça.

[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

[SheikYerbouti]

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

Code :

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

private javazoom.jl.player.Player player;

[SheikYerbouti]

Ok, c'est parfait. Merci bien.

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

[SheikYerbouti]

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

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

[Alkhan]

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

Code :

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

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

[adiGuba]

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 :

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

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

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 :

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

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

[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