FileOutputStream : Syntaxe du nom de fichier incorrect

**stephane.combes** · 19/01/2011, 16h42

Bonjour,

Je souhaite, à partir d'un poste Windows, récupérer des fichiers se trouvant en partage sur un autre poste Windows.

Les noms de fichier sont du style : RIOM MVB T215_6077[2011-1-19][07-43-20].xml

J'utilise la méthode suivante pour faire un copie de fichier :

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
 
private boolean copyFile(File pSrcFile, File pDstFile) throws IOException {
		boolean lResult = false;
 
		// Open channels
		FileInputStream lInputStream = new FileInputStream(
				pSrcFile);
		FileOutputStream lOutputStream = new FileOutputStream(
				pDstFile);
 
		try {
			// Copy from input to output channel with 0.5Mo buffer size
			byte buffer[] = new byte[512 * 1024];
			int nbLecture;
 
			while ((nbLecture = lInputStream.read(buffer)) != -1){
				lOutputStream.write(buffer, 0, nbLecture);
			}
 
			lResult = true;
		} finally {
			// Close channels
			try {
				lInputStream.close();
			} catch (IOException e) {
				// Ne rien faire
			}
			try {
				lOutputStream.close();
			} catch (IOException e) {
				// Ne rien faire
			}
		}
		return lResult;
	}

Lors du new FileOutputStream() j'ai l'erreur suivante : Syntaxe du nom de fichier, de répertoire ou de volume incorrecte.

Si je crée un partage Windows sur ma machine la copie de fichier se fait correctement.

Qu'elle peut être l'origine du problème ?
Toute piste sera la bien venue.

Merci.

**adiGuba** · 19/01/2011, 16h59

Salut,

Avant toute chose, trois remarques :

Le type de retour est totalement inutile puisqu'il vaudra toujours true. En effet en cas d'erreurs une exceptions est remontée (et donc il n'y a pas de retour).
Tu ignores les erreurs des fermetures de fichiers. Même si c'est rare cela peut arriver et tu ne vas pas pouvoir le détecter. Il est préférable de laisser remonter ces erreurs.
De même ton try/finally unique a une faille : si une exception survient lors de l'ouverture du flux de sortie, alors le flux d'entrée ne sera jamais libéré...

Perso j'utiliserais plutôt un code comme 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
19
20
	private void copyFile(File pSrcFile, File pDstFile) throws IOException {
		// Open channels
		FileInputStream lInputStream = new FileInputStream(pSrcFile);
		try {
			FileOutputStream lOutputStream = new FileOutputStream(pDstFile);
			try {
				// Copy from input to output channel with 0.5Mo buffer size
				byte buffer[] = new byte[512 * 1024];
				int nbLecture;
 
				while ((nbLecture = lInputStream.read(buffer)) != -1) {
					lOutputStream.write(buffer, 0, nbLecture);
				}
			} finally {
				lOutputStream.close();
			}
		} finally {
			lInputStream.close();
		}
	}

Quand à ton problème, il semble venir des chemins que tu passes en paramètres à ces méthodes.

Quels sont les valeurs de pSrcFile et pDstFile ???

a++

**stephane.combes** · 20/01/2011, 10h55

Bonjour,

Merci adiGuba pour les précisions

J'ai fait d'autres essais et mon problème survient lorsque le fichier source est utilisé par une autre application.

Connaissez-vous un moyen de savoir si un fichier est en cours d'utilisation afin de ne pas demander la copie de ces fichiers ?

Merci

**tchize_** · 20/01/2011, 13h03

la seule solution est d'essayer et de traiter l'erreur. L'api java ne permet (pas encore) de vérifier si un verrou exclusif est actuellement sur le fichier.