Discussion :

[luna007]

Bonsoir,
J'ai un problème d'ecriture, Je lis à partir d'un flux xml en ligne et je l'ecris sur un fichier xml,
le problème c'est que le fichier de sortie je le trouve à 0 ok pourtant je remarque que le programme arrive à lire le flux.
voilà le code que j'utilise :

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
 
public static boolean createXmlFromUrl(URL source, File dest) throws IOException{
		BufferedInputStream sourceFile=null;
		try {
			URLConnection connection = source.openConnection();
			InputStream is = connection.getInputStream();	
 
			if(is!=null){
				sourceFile= new BufferedInputStream(is);
				dest.createNewFile();
				FileOutputStream fos = new FileOutputStream(dest);
				BufferedOutputStream destinationFile = new BufferedOutputStream(fos);
 
				try {
					try {
						byte buffer[] = new byte[512 * 1024];
						int nbLecture;
						while ((nbLecture = sourceFile.read(buffer)) != -1) {
							destinationFile.write(buffer, 0, nbLecture);
						}
					}catch(Exception ex){
						System.out.println("Erreur d'écriture dans le fichier de destination");
					}
					finally {
						fos.close();
						destinationFile.close();
 
					}
				}catch(Exception ex){
					System.out.println("Erreur de fermeture du fichier de destination");
				}
				finally {
					sourceFile.close();
					fos.close();
					destinationFile.close();
					is.close();
				}
			}//fin if
		} catch (Exception e) {
			System.err.println("Erreur de connexion à la source ");
			if(sourceFile!=null)
				sourceFile.close();
			return false; // Erreur
		}
		return true; 
	}

quelqu'un à une idée ?
Merci d'avance pour l'aide

[adiGuba]

Tu ne fermes pas les flux dans le bon ordre. Et comme tu utilises un BufferedOutputStream certaines données ne sont pas écrites alors que tu as déjà fermé le fichier

Et avec un fichier de taille plus grande il te manquera la fin...



Quelques conseils :

• Les try/finally doivent commencer juste après la création d'un flux, et le finally ne doit contenir qu'un seul et unique appel de méthode : close(). De plus il est alors inutile de fermer le flux dans les blocs catch (le finally est exécuté dans tous les cas).
• Evites de multiplier les variables cela prêtent à confusion et c'est source d'erreur.
  Englobe les création de flux lorsque c'est possible, par exemple :
  

  Code :

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

  BufferedOutputStream out = new BufferedOutputStream(new FileOutputStream(dest));

• Tu as trois niveaux de catch : A quoi cela peut-il bien servir ??? (mis à part à rendre le tout illisible).
  De plus tu ignores complètement le stacktrace et en cas d'erreur tu perds toutes les infos utile au débuggage
• Toujours au niveau des exceptions : il est plus pratique de remonter une exception que de retourner true/false, car cela ne permet pas de traiter correctement les erreurs et encore moins de les comprendre...
• Tes BufferedStream sont inutile puisque tu effectues toi-même une lecture bufférisé... tu pourrais t'en passer !

Perso j'aurais écrit ta méthode de la manière suivante :

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
	public static void createXmlFromUrl(URL source, File dest) throws IOException {
		final InputStream input = source.openStream();
		try {
			final OutputStream output = new FileOutputStream(dest);
			try {
				final byte buffer[] = new byte[512 * 1024];
				int nbLecture;
				while ((nbLecture = input.read(buffer)) != -1) {
					output.write(buffer, 0, nbLecture);
				}
			} finally {
				output.close();
			}
		} finally {
			input.close();
		}
	}

a++

[luna007]

Tu ne fermes pas les flux dans le bon ordre. Et comme tu utilises un BufferedOutputStream certaines données ne sont pas écrites alors que tu as déjà fermé le fichier

Et avec un fichier de taille plus grande il te manquera la fin...



Quelques conseils :

• Les try/finally doivent commencer juste après la création d'un flux, et le finally ne doit contenir qu'un seul et unique appel de méthode : close(). De plus il est alors inutile de fermer le flux dans les blocs catch (le finally est exécuté dans tous les cas).
• Evites de multiplier les variables cela prêtent à confusion et c'est source d'erreur.
  Englobe les création de flux lorsque c'est possible, par exemple :
  

  Code :

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

  BufferedOutputStream out = new BufferedOutputStream(new FileOutputStream(dest));

• Tu as trois niveaux de catch : A quoi cela peut-il bien servir ??? (mis à part à rendre le tout illisible).
  De plus tu ignores complètement le stacktrace et en cas d'erreur tu perds toutes les infos utile au débuggage
• Toujours au niveau des exceptions : il est plus pratique de remonter une exception que de retourner true/false, car cela ne permet pas de traiter correctement les erreurs et encore moins de les comprendre...
• Tes BufferedStream sont inutile puisque tu effectues toi-même une lecture bufférisé... tu pourrais t'en passer !

Perso j'aurais écrit ta méthode de la manière suivante :

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
	public static void createXmlFromUrl(URL source, File dest) throws IOException {
		final InputStream input = source.openStream();
		try {
			final OutputStream output = new FileOutputStream(dest);
			try {
				final byte buffer[] = new byte[512 * 1024];
				int nbLecture;
				while ((nbLecture = input.read(buffer)) != -1) {
					output.write(buffer, 0, nbLecture);
				}
			} finally {
				output.close();
			}
		} finally {
			input.close();
		}
	}

a++

Merci pour ces remarques,
ça marche