Discussion :

[Amnael]

Bonjour,

tout d'abord, il me faut dire que le code suivant ne se lance pas depuis la machine sur lequel il travaille (d'où la présence de l'ip).


Je vous explique ce que fais le code.

Il va chercher un fichier sur le serveur.
Il le recopie en modifiant une ligne.

Cette partie est OK.

Par la suite, je voudrais effacer l'ancienne version du fichier.

Sauf que delete me renvoie false à ce moment là; et il me semble bien avoir fermé toutes les entrées, ce que ce soit en lecture et écriture.

Par la suite, je veux simplement renommer le nouveaux fichier comme l'ancien (ce qui marche je pense mais échoue pour le moment comme le fichier n'est pas supprimé).


Voici le code: (en vert la partie fonctionnelle; et en rouge ce qui pose problème)

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
52
53
54
	//Return the old UU 
	public String modifGrfiti(String ip,Settings set,FTextArea test){
		String pathGrfiti="\\\\"+ip+"\\\\"+set.getpathGrfiti();
		String newPath="\\\\"+ip+"\\\\"+set.getrepGrfiti()+"\\GrfItiServerV2bis.xml";
		String UU="notFound";
		File grfiti=new File(pathGrfiti);
		if(grfiti.exists()){
			BufferedReader lecteurAvecBuffer = null;
		    String line;
		    
		    try{
		    	lecteurAvecBuffer = new BufferedReader(new FileReader(grfiti.getPath()));
		    	//create the newFile
		    	BufferedWriter writer = new BufferedWriter(new FileWriter(new File(newPath)));
		    	//open writting file mode
		    	FileWriter fw= new FileWriter(newPath,true);
				while ((line = lecteurAvecBuffer.readLine()) != null){
					//Read the file line by line
					if(line.contains("<GraphTC>")){
						String newLine="<GraphTC>"+set.getdestGraph()+"\\geocati_"+this.getNumUU()+"</GraphTC>\n";
						fw.write(newLine);
						UU=line.substring(line.lastIndexOf("_")+1,line.lastIndexOf("_")+6);
					}else{
						fw.write(line+"\n");
					}
				}
				lecteurAvecBuffer.close();
				fw.close();
				writer.close();
				//V2vis is the new File so delete old one
				if(grfiti.delete()){
					System.out.println("Ancien Grfiti supprimé OK\n");
					test.addext("Ancien Grfiti supprimé OK\n");
				}else{
					System.out.println("Echec suppression ancien Grfiti");
					test.addext("Echec suppression ancien Grfiti\n");
				}
				//V2 is destroyed; now change V2bis into V2
				String com= "ren "+newPath+ " GrfItiServerV2.xml";
				Launch testCom=new Launch(com,test);
				//back the new Grfiti
				testCom.exe();
		    }catch (IOException e ) {
				// TODO Auto-generated catch block
		    	System.out.println("Echec de traitement du fichier\n");
				e.printStackTrace();
			}
		}else{
			System.out.println("Le fichier "+pathGrfiti+" n'existe pas !");
			test.addext("Le fichier "+pathGrfiti+" n'as pas été trouvé !");
		}
		return UU;
	}

Il y a bien un fichier ouvert localement en écriture pendant que cette fonction travaille, mais normalement ça ne devrait pas poser de problèmes...

[Népomucène]

Et si tu le supprime "à la main" en passant par l'explorateur de fichiers, ça marche ?
(pour éliminer un problème de droit d'accès)

[Amnael]

Oui bien sûr qu'à la main ça marche^^

J'ai quand même jeté un coup d'oeil aux droits, j'ai mis les utilisateurs en contrôle total, et là j'ai pu supprimer.

Chose étrange, le fichier que j'ai crée derrière lui n'avait pas ses droits, mais j'ai pu le supprimer également...sans modifier les droits.


Mais de toute façon cette histoire de droit ne devrait pas jouer vu que le compte utiliser est admin de tous les serveurs.

En tout cas il semblerait que cela fonctionne à présent. J'avoue ne pas comprendre...

[joel.drigo]

Salut,

Essaye également de passer par l'API java.nio.Path pour supprimer ton fichier, pour avoir un message d'erreur explicite le cas échéant. Tu n'es pas obligé de tout modifier pour faire ce test :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
try {
 
     Files.delete(grfiti.toPath());
					System.out.println("Ancien Grfiti supprimé OK\n");
					test.addext("Ancien Grfiti supprimé OK\n");
 
}
catch(IOException e) {
	System.out.println("Echec suppression ancien Grfiti");
	test.addext("Echec suppression ancien Grfiti\n");
}

Ensuite, il est possible, selon le protocole de partage (smb/cifs ? nfs ?), qu'il y ait de la latence sur les actions sur fichiers distants. S'il s'agit que le fichier n'existe plus lorsque l'application est finie, tu peux essayer un deleteOnExit().

[adiGuba]

Salut,



Pourquoi tu ouvres deux fois le fichier ?

Code :

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

1
2
3
4
		    	//create the newFile
		    	BufferedWriter writer = new BufferedWriter(new FileWriter(new File(newPath)));
		    	//open writting file mode
		    	FileWriter fw= new FileWriter(newPath,true);

Sinon pour la fermeture des fichiers, il vaut mieux passer par des try/finally, ou mieux le try-with-ressource de Java 7.


Au passage si tu es sous Java 7, il est préférable d'utiliser la nouvelle API Path :

Code :

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

Files.delete(grfiti.toPath());

Cela te génèrera une erreur avec un message clair si le fichier ne peut pas être supprimé.


Au passage en Java 7 pour écraser un fichier sur le même disque, il est bien plus simple d'utiliser Files.move() :

Code :

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

Files.move(source, target, StandardCopyOption.REPLACE_EXISTING, StandardCopyOption.ATOMIC_MOVE);

a++

[Amnael]

Joel, je vais regarder ça merci !

adiGuba, je me sers du premier uniquement pour créer le fichier. (C'est surement sale, mais à partir du moment où je ferme bien le flux ça ne me pose pas de problème).
Pour le toPath, effectivement c'est ce que Joel disait, je vais regarder.

[adiGuba]

adiGuba, je me sers du premier uniquement pour créer le fichier. (C'est surement sale, mais à partir du moment où je ferme bien le flux ça ne me pose pas de problème).

C'est sale, c'est inutile, et cela pourrait même te faire perdre le contenu de ton fichier...

ll n'y a aucune raison de faire cela...


a++