Discussion :

[benzing]

Bonjour à tous,

j'ai un soucis de suppression d'un fichier de properties.
Les fichiers properties sont lus au démarrage de Jonas. Je crée un formulaire listant les fichier properties et permettant de supprimer ces fichiers !

Quand je suis en mode débogage et que je passe dans chaque classe à partir de la méthode Delete (java.io.File puis java.lang.SecurityManager puis java.security.AccessController puis java.security.AccessControlContext) le fichier ce supprime bien !
si j'enleve mon point d'arret le fichier ne se supprime pas !
j'ai mis un try catch :

Code :

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

1
2
3
4
5
6
try {
     deleteFile = fichierEnvironnement.delete();
     System.out.println(deleteFile);
}catch (SecurityException e){
     System.out.println("erreur : " + e.toString());
}

je ne rentre jamais dans l'exception mais mon deleteFile est à false donc le fichier n'est pas supprimer !

Avez vous déjà été confronté à ce genre de problème ?
merci de votre aide.

[natha]

File.delete() ne marche pas toujours car tu n'as pas les droits d'accès sur le fichier (genre il est ouvert dans une autre applic).
File.deleteOnExit() ne marche pas toujours non plus.
Voilà ma solution.

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
/**
 * Thread for deleting a file.
 * Try again as long as the file isn't deleted and max tries allowed hasn't be exceeded.
 * 
 * @author natha
 */
public class DeleteFileThread extends Thread {
 
	/** The Logger */
	private static Logger logger = Logger.getLogger(DeleteFileThread.class);
 
	/** The file to delete */
	private File file;
 
	/** The delay before trying to delete */
	private int delay;
 
	/** Maximum tries allowed */
	private int maxTries;
 
	/** The number of tries */
	private int count;
 
	/** Default value for delay */
	private static int DEFAULT_BEGIN_DELAY = 10000;
 
	/** Default maximum tries allowed */
	private static int DEFAULT_MAXIMUM_TRIES_ALLOWED = 3600; // 3600 * 1000 secondes = 1h
 
	/**
         * Default Constructor
         * 
         * @param file The file to be deleted
         * @param delay The delay before trying to delete
         * @param maxTries Maximum tries allowed (0 means default)
         */
	protected DeleteFileThread(File file, int delay, int maxTries) {
		super();
		this.file = file;
		this.delay = delay;
		this.maxTries = (maxTries == 0 ? DEFAULT_MAXIMUM_TRIES_ALLOWED : maxTries);
		this.count = 0;
	}
 
	/* (non-Javadoc)
	 * @see java.lang.Thread#run()
	 */
	@Override
	public void run() {
		try {
			sleep(delay);
			while (count < maxTries && !file.delete() && file.exists()) {
				count++;
				sleep(1000); // sleep for 1sec (= try again in 1sec)
				if (count == maxTries) {
					logger.info("Max tries exceeded (" + maxTries + ") and the file '" + file.getCanonicalPath() + "' hasn't be deleted");
				}
			}
		} catch (InterruptedException e) {
			logger.fatal("Unexpected error", e);
		} catch (IOException e) {
			logger.fatal("Unexpected error", e);
		}
	}
 
	/**
         * @param file The file to be deleted
         */
	public static void createThread(File file) {
		createThread(file, DEFAULT_BEGIN_DELAY, 0);
	}
 
	/**
         * @param file The file to be deleted
         * @param delay The delay before trying to delete
         */
	public static void createThread(File file, int delay) {
		createThread(file, delay, 0);
	}
 
	/**
         * @param file The file to be deleted
         * @param delay The delay before trying to delete
         * @param maxTries Maximum tries allowed (0 means default)
         */
	public static void createThread(File file, int delay, int maxTries) {
		DeleteFileThread dft = new DeleteFileThread(file, delay, maxTries);
		dft.start();
	}
 
}

Tu fais simplement : DeleteFileThread.createThread(monFichier) et ce thread va tenter de supprimer ce fichier pendant 1h toutes les 1s (paramétrable).

[benzing]

ok merci ! le problème est résolu,
j'ai utiliser ta boucle while dans mon code !
Merci bien

Code :

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

1
2
3
4
5
int maxTries =10000;
int count = 0;
while (count < maxTries && !file.delete() && file.exists()) {
     count++; 
}

[natha]

J'serais toi, je rajouterais un Thread.sleep(100) au moins pour éviter de boucler trop rapidement. Si le fichier est bloqué, tu sortiras de la boucle trop vite et elle aura vite fait de te prendre 100% de temps proc.

[FFF]

'jour

J'ajoute mon opinion sur la question :
j'ai déjà eu aussi ces problèmes et je les ai résolus à 99% en séparant les méthodes manipulant les fichiers (une méthode pour écrire, une autre pour lire, une autre pour créer un fichier,...) en annulant toutes les variables ayant un rapport avec l'objet fichier crée (fileoutputstream etc...).Il me semble que si le fichier ne se supprime pas c'est qu'un des objets crées vérouille le fichier, mais si ton appli le permet le fichier finit toujours par être supprimé même après un certain délai... je n'en suis jamais arrivé à être obligé de créer une boucle de suppression qui me semble pas être l'optimum.

[natha]

Le thread que j'ai copier/coller peut répondre à tous les cas. Il y a des applics où les fichiers sont gérés également par des processus externes à ton programme java.

Exemple :
Tu crées un PDF pour impression, tu lances l'ouverture du PDF par adobe. Tu ne pourras pas supprimer le fichier PDF tant qu'il n'est pas libéré par adobe. Par contre l'utilisateur pourrait décider de quitter l'applic alors qu'adobe est encore ouvert. Résultat, même si tu attendais la fermeture d'adobe, tu ne pourras plus effacer le fichier.

Le thread d'effacement répond à tous les cas et ne te monopolise pas les ressources étant donné qu'il dors à chaque nouvel intervale.