Discussion :

[Katachana]

Bonjour,

Je souhaite lister des fichiers txt contenu dans un jar et puis ensuite les copier en local.


Pour récupérer la liste des fichier txt pas de souci :

Code :

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

1
2
3
4
5
6
7
8
 
Enumeration<JarEntry> entries = jarfile.entries();
      while(entries.hasMoreElements()) {
          JarEntry entry = (JarEntry)entries.nextElement();
          if(entry.getName().endsWith(".txt")){
              FileUtil.copyFile(entry.getName(),entry.getName());
          }
      }

Ensuite j'ai une erreur dans la méthode copyFile, au niveau de l'inputStream (Le fichier spécifié est introuvable).... C'est surement un probleme de path, mais je n'y vois plus grand chose

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
 
    public static void copyFile (String input, String output){
 
        FileChannel in = null; // canal d'entrée
        FileChannel out = null; // canal de sortie
 
        try {
            // Init
            in = new FileInputStream(input).getChannel();
            out = new FileOutputStream(output).getChannel();
 
            // Copie depuis le in vers le out
            in.transferTo(0, in.size(), out);
          } catch (Exception e) {
            e.printStackTrace(); // n'importe quelle exception
          } finally { // finalement on ferme
            if(in != null) {
              try {
                in.close();
              } catch (IOException e) {}
            }
            if(out != null) {
              try {
                out.close();
              } catch (IOException e) {}
            }
          }
 
    }

Merci d'avance !

[fr1man]

A mon avis ce que tu essaies de faire ne marchera pas.
Il faut plutôt lire ton fichier .txt à partir d'un JarInputStream avec la méthode read(byte[] b, int off, int len) et utiliser getNextJarEntry() .

[adiGuba]

Salut,



Pour commencer quelques remarques sur ta méthode copyFile :

• Le catch(Exception) n'est pas très propre car il pourrait te cacher des erreurs de programmation (nullpointer ou autre). Il est préférable de se limiter aux IOException...
• Personnellement j'aurais même tendance à laisser remonter l'IOException, car il n'y a rien permettant de détecter une erreur de copie lorsqu'on appelle ta méthode.
• Plutôt qu'un grand finally avec des if et des try/catch pas très propre, je te conseillerais d'utiliser un finally contenant uniquement le close, mais de le faire pour chaque flux.

En clair, sans remonter d'exception cela donnerait ceci :

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
    public static void copyFile (String input, String output){
 
    	try {
	        FileChannel in = new FileInputStream(input).getChannel(); // canal d'entrée
	        try {
	        	FileChannel out = new FileOutputStream(output).getChannel(); // canal de sortie
	        	try {
	        		in.transferTo(0, in.size(), out);
	        	} finally {
	        		out.close();
	        	}
	        } finally {
	        	in.close();
	        }
    	} catch (IOException e) {
    		e.printStackTrace();
    	}
    }

Quand à ton problème, il vient du fait que tu considères les éléments d'une archive comme un fichier standard. Or ce n'est pas le cas : il ne s'agit pas d'un fichier présent directement sur le système de fichier (il est inconnu de l'OS), donc tu ne peux pas l'ouvrir avec un FileInputStream.

Du coup tu ne peux même pas utiliser les FileChannel tu dois faire la copie à la main, ce qui n'est pas très compliqué en soit :

Code :

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

1
2
3
4
5
6
7
	public static void copy(InputStream in, OutputStream out) throws IOException {
		byte[] buf = new byte[8192];
		int len;
		while ( (len=in.read(buf)) >= 0 ) {
			out.write(buf, 0, len);
		}
	}

En fait pour accéder au contenu de l'élément d'une archive, tu dois utiliser la méthode getInputStream() du JarFile.

Tu devrais donc avoir une méthode de la sorte :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	public static void copy(JarFile jarFile, JarEntry entry, String outputFileName) throws IOException {
		InputStream in = jarFile.getInputStream(entry);
		try {
			OutputStream out = new FileOutputStream(outputFileName);
			try {
				copy(in, out);
			} finally {
				out.close();
			}
		} finally {
			in.close();
		}
	}

a++

[Katachana]

Merci beaucoup !!