Discussion :

[monstroplante]

Bonjour à tous,

Je développe actuellement une petite application pour récupérer des pochètes d'albums MP3. cette application, développée en C#, télécharge de nombreuses images et les stock sous forme de fichiers temporaires. Je n'ai jamais travaillé avec des fichiers temporaires et je ne sais pas quels sont les bonnes pratiques à adopter.

Mes questions sont donc les suivantes :
-Quel répertoire serait le plus approprié pour stocker mes images
-Dois-je supprimer les images à la fermeture de mon application
-Si oui, quel événement de mon application pourrait être approprié pour déclencher le nettoyage (je crains que certains fichiers soient en cours d'utilisation...)

Merci pour votre aide !

[ram-0000]

-Quel répertoire serait le plus approprié pour stocker mes images

Cela dépend de ton OS mais sous linux/unix, c'est /tmp et sous Microsoft, c'est le répertoire pointé par la variable d'environnement TEMP ou TMP. Après cela, tu as la fonction tmpnam() qui est utile pour générer des fichiers temporaires dans un répertoire donné (voir ici : http://man.he.net/man3/tmpnam)

-Dois-je supprimer les images à la fermeture de mon application

Je dirai oui mais cela dépend du contenu de tes fichiers. Si les refaire ne coûte rien, aloors clairement oui sinon, cela peut être des fichiers temporaires dont l'effacement à peu d'importance (genre les fichiers du cache internet local)

-Si oui, quel événement de mon application pourrait être approprié pour déclencher le nettoyage (je crains que certains fichiers soient en cours d'utilisation...)

Cela dépend de ton application, sous linux/unix, tu peux installer une fonction avec atexit()
pour faire le ménage (voir ici : http://www.linux-kheops.com/doc/man/.../atexit.3.html) sinom, pour une appli Windows, tu peux gérer l'événement WM_CLOSE

[monstroplante]

Merci pour tes réponses. J'ai un peu avancé sur le problème et créé cette classe pour me faciliter les choses :

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
 
	static class FileHelper {
		private static string _tempFolder;
		public static string TempFolder{
			get {
				if (_tempFolder == null) {
					_tempFolder = Path.Combine(Path.GetTempPath(),Guid.NewGuid().ToString());
					Directory.CreateDirectory(_tempFolder);
					Application.Current.Exit += new ExitEventHandler(Application_Exit);
				}
				return _tempFolder;
			}
		}
 
		public static string GetTempFileName() {
			return Path.Combine(TempFolder, Guid.NewGuid().ToString());
		}
 
 
		/// <summary>
		/// Supprime le dossier temporaire à la fermeture de l'application
		/// </summary>
		/// <param name="sender"></param>
		/// <param name="e"></param>
		static void Application_Exit(object sender, ExitEventArgs e) {
			Directory.Delete(_tempFolder,true);
		}
	}

Le problème c'est que, comme je le craignais, certains fichiers sont en cours d'utilisation à la fermeture du programme. Cela s'explique car je télécharge les images sur des thread différents qui sont, semble-il, encore actifs lors de l'événement "Exit" de l'application.

J'ai pensé utiliser l'attribut "DELETE_ON_CLOSE" mais je n'ai pas trouvé comment éditer les attributs "Extended" en C#.

Si vous avez un avis, une idée, je suis preneur (ce post serait peut être plus à sa place sur le forum c# ? si un modo passe par là...)

[Greybird]

Oula, arrête tout.

Code :

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

Path.GetTempPath()

Code :

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

Path.GetTempFileName()

[monstroplante]

Oui, j'avais vu la méthode "Path.GetTempFileName()" mais je ne vois pas en quoi elle serait mieu adaptée car :
1) Elle ne gère en rien la suppression des fichiers temporaires. Ce qui est finalement mon seul vrai problème.
2) Elle crée un fichier vide, ce que ne souhaite pas.
3) Elle crée le fichier temporaire à la racine du répertoire temp or je préfère regrouper mes fichiers dans un sous-dossier car j'en utilise un grand nombre.


Finalement ma classe "FileHelper" me plait bien mais mon problème c'est que j'ai plusieurs background workers qui téléchargent des fichiers à l'aide de WebCilents. Je pense que c'est eux qui empèchent la suppression des fichiers à la fermeture de l'application.

(Cela peut paraitre étrange d'utiliser des WebClient avec des BackgroundWorkers mais je n'aprécie pas la méthode "DownloadFileAsync". Cette dernière ne travaile dans un thread séparé qu'après avoir résolu l'éventuel nom DNS contenu dans l'url. Ce qui entraine des lag sur le thread principal.)

PS : Merci pour le déplacement du post.

[Greybird]

Oui, j'avais vu la méthode "Path.GetTempFileName()" mais je ne vois pas en quoi elle serait mieu adaptée car :
1) Elle ne gère en rien la suppression des fichiers temporaires. Ce qui est finalement mon seul vrai problème.
2) Elle crée un fichier vide, ce que ne souhaite pas.
3) Elle crée le fichier temporaire à la racine du répertoire temp or je préfère regrouper mes fichiers dans un sous-dossier car j'en utilise un grand nombre.

Ok, au temps pour moi, j'ai lu le post un peu rapidement qui plus est car tu faisais déjà utilisation de Path.GetTempPath()

Après ça me semble un peu bizarre de classer des fichiers temporaires, surtout s'ils ne doivent pas persister à la fin de l'application. Mais pourquoi pas.

L'idéal serait sans doute, avant de détruire le répertoire, de demander à tous tes threads de te terminer, et d'attendre que ce soit le cas. Cela suppose bien évidemment de conserver une liste de ces threads quelque part, mais j'imagine que c'est déjà le cas.

Mais au fond, pourquoi ne pas détruire ce que tu peux, et laisser le répertoire temporaire dans l'état où il se trouvera ? Le but de ce répertoire est bel et bien de servir de poubelle en cas de besoin, et, dans le cas où l'utilisateur aurait besoin de place, le système lui proposera de lui même de faire le ménage dans ce répertoire (en tout cas sous Windows)

Je conçois qu'il soit satisfaisant de faire le ménage derrière soi, mais est-ce vraiment grave de laisser quelques fichiers dans le répertoire temporaire, qui seront effacés si jamais le besoin s'en fait sentir ?