Discussion :

[Seth_75]

Bonjour ,

Je suis sous le FRAMEWORK 1.1, j'ai fais une application console permettant l'import automatique d'un fichier csv.
1) Je déclare une variable qui contient le chemin complet du fichier.
2) Je copy mon le contenu de mon fichier csv dans un fichier txt temporaire.
3) Je parse le fichier txt que j'insere dans la table de ma BDD .
4) Je supprime le fichier txt
5) Je supprimer le fichier csv


L'erreur se produit quand j'ai ajouté la ligne File.delete(monFichierCsv) donc c'est à dire l'étape 5, l'erreur est la suivante :

"Le processus ne peut pas accéder au fichier , car il est en cours d'utilisation par un autre processus."


Je voudrais savoir si quelqu'un a eut le même problème et comment a-t-il fait pour le résoudre?

Merci d'avance

[DroP]

Montre nous ton code que l'on puisse t'aider, tu as sans doute oublier de libérer la ressource à un endroit.

Oublie pas de commencer par lire les cours de formation pour apprendre la programmation C# : http://dotnet.developpez.com/cours/?page=csharp

[Lyche]

tu as pensé à fermer le fichier avec la commande

Code :

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

1
2
 
monStreamReader.Close();

à moins que tu ne passes pas en streamreader. Mais dans tous les cas le message semble indiquer que tu ne refermes pas ton fichier après l'avoir exploité.

Cordialement,
Lyche

[Seth_75]

Montre nous ton code que l'on puisse t'aider, tu as sans doute oublier de libérer la ressource à un endroit

Fonction importDonnées:

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
 
DirectoryInfo dossier = new DirectoryInfo(path);
 
foreach (FileInfo mesfichiers in dossier.GetFiles("*.csv"))
{
  Console.WriteLine(mesfichiers.FullName); 
  path = mesfichiers.FullName.ToString();
}	
 
if (File.Exists(path)) 
    File.OpenRead(path);
 
StreamReader sr	=	File.OpenText(path);
string content = sr.ReadToEnd();
 
import.ValidImport(path);
 
File.Delete(path);

Fonction ValidImport(path)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
private void ValidImport(string path)
{
string cheminFichier= "C:\\Temporaire_file\\monfichier.txt";
 
try
{
 
File.Copy(path,cheminFichier);
this.importation(cheminFichier);
 
}
 
}

Fonction Importation

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
 
private void importation(string cheminFichier)
{
try
{  
StreamReader fichier = File.OpenText(cheminFichier);
...
Ensuite je déclare cette variable pour lire ligne à ligne : 
string lignes = fichier.ReadLine();
}
catch ...
 puis dans le finally : 
 
finally
{
  fichier.Close();
}
 
}

Désolé pour l'indentation mais je n'ai pas l'habitude du site la c'est pour cela voilà .

[Seth_75]

tu as pensé à fermer le fichier avec la commande

Code :

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

1
2
 
monStreamReader.Close();

à moins que tu ne passes pas en streamreader. Mais dans tous les cas le message semble indiquer que tu ne refermes pas ton fichier après l'avoir exploité.

Cordialement,
Lyche

Merci j'ai déja fait ja fais sr.close() juste avant de faire :
File.Delete(path);


ca ne marche pas :s

[DroP]

if (File.Exists(path))
File.OpenRead(path);

StreamReader sr = File.OpenText(path);
string content = sr.ReadToEnd();

import.ValidImport(path);

File.Delete(path);

Tu ne dispose pas ta variable sr !!
je te conseille plutot d'utiliser des using :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
if (File.Exists(path)) 
    File.OpenRead(path);

using (StreamReader sr = File.OpenText(path))
{
string content = sr.ReadToEnd();
 
import.ValidImport(path);
}
 
File.Delete(path);

[Seth_75]

Tu ne dispose pas ta variable sr !!
je te conseille plutot d'utiliser des using :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
if (File.Exists(path)) 
    File.OpenRead(path);

using (StreamReader sr = File.OpenText(path))
{
string content = sr.ReadToEnd();
 
import.ValidImport(path);
}
 
File.Delete(path);

Merci à tout le monde de me répondre pour commencer .

Sinon en ce qui concerne de dispose ma variable sr je ne sais pas comment faire ? Et j'ai tester avec ton using j'ai toujours la même erreur.

[Lyche]

juste avant ton sr.Close(); place un sr.Dispose();
Dispose libère les ressources utilisée par la machine, là tu es sur que ça ne vient pas de ça.
Une question, le fichier texte est généré comment?

[Seth_75]

juste avant ton sr.Close(); place un sr.Dispose();
Dispose libère les ressources utilisée par la machine, là tu es sur que ça ne vient pas de ça.
Une question, le fichier texte est généré comment?

Je peux pas faire de sr.Dispose() cela me générer une erreur de type :

'System.IO.TextReader.Dispose(bool)' est inaccessible en raison de son niveau de protection


Sinon pour le fichier texte il est générer de cette maniere :
string cheminFichier = "C:\\Temporaire_file\\file.txt";
// copy du fichier csv dans le fichier texte

File.Copy(path,cheminFichier);

[Lyche]

Juste une question d'ordre technico-pratique. Pourquoi tu passes du fichier csv au fichier texte? Pourquoi tu insères pas directement tes données dans ta bdd? Les fichiers csv sont prévu pour. Avec un Split de ta ligne tu peux envoyer les données, ou alors tu split en T-SQL. ça pourrait t'éviter des manipulations de fichiers et de te retrouver avec des traitements plus longs.
Mais je dis ça, celà n'engage que moi ^^

Cordialement,
Lyche

[Seth_75]

Juste une question d'ordre technico-pratique. Pourquoi tu passes du fichier csv au fichier texte? Pourquoi tu insères pas directement tes données dans ta bdd? Les fichiers csv sont prévu pour. Avec un Split de ta ligne tu peux envoyer les données, ou alors tu split en T-SQL. ça pourrait t'éviter des manipulations de fichiers et de te retrouver avec des traitements plus longs.
Mais je dis ça, celà n'engage que moi ^^

Cordialement,
Lyche

Parce que le code était déjà tout fait, je débug et je n'ai pas assez de connaissance en .NET ( 1.1 ) donc j'apprends, et comme je passe derriere quelqu'un je code de manière cohérente avec ce qui a déjà été fait étant donnée que chaque page fait 2500 lignes de code et qu'il y en a une trentaine avec des fonctions qui se croisent de partout ...
Voilà pourquoi ....

Mais si tu as des liens qui explique bien ce que tu me dis je suis open en attendant faut que je règle ce problème ...

Merci en tout cas pour ces réponses.

Quelqu'un a-t-il d'autres solutions ???


Cordialement

[Lyche]

En effet, ça me paraît compliqué de pouvoir faire autrement, mais, essaye de lire directement ton fichier csv plutot que de passer par un fichier txt. Les deux fichiers se traitent de la même façon, celà t'enlève juste l'étape de transfert dans le fichier texte. Cela te fera gagner du temps sur ton exécution de processus et cela facilitera la maintenance.
Au pire tu peux inverser ton processus, tu supprimes et recré ton fichier texte en début de traitement, comme ça tu es sur que tu ne t'en sers pas encore.

Code :

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

1
2
 
if (File.Exists(path)) File.delete(); File.logFileCreate();

parfois inverser un problème permet de le résoudre plus facilement ^^.
Enfin ce ne sont que mes idées pas toujours très pertinentes.

Cordialement,
Lyche

[Seth_75]

1) Est ce que quelqu'un sait pourquoi je ne peux pas supprimer ce fichier ?
2) Et pourquoi je ne peux pas faire mon sr.Dispose() ?

- Dispose ne marche pas sous Framework 1.1 ?
- Faut il déclarer un using ?
- Une assembly spéciale ?

[Graffito]

Le fichier .csv ne serait-il pas ouvert sous Excel au moment de la suppression ?

[Seth_75]

Le fichier .csv ne serait-il pas ouvert sous Excel au moment de la suppression ?

LOL ?

Je sais que je ne suis pas super fort en programmation mais je ne suis pas con non plus =) je suis pas un boulet à ce point ...

[StormimOn]

Dans ton import tu fais un appel à File.OpenRead. Cette méthode renvoie un objet de type FileStream que tu ne récupères pas et que donc tu ne libères pas au final. C'est peut être la raison pour laquelle la libération du StreamReader ne suffit pas à résoudre ton problème.

Au passage (en supposant que le code donné est vraiment celui utilisé) :
- la boucle foreach ne sert à rien dans l'état
- le File.OpenRead non plus
- le test d'existence du fichier est également inutile vu comment est récupéré le nom de ce fichier. Il vaut mieux voir si des fichiers ont été trouvés.

Sinon, pour récupérer le contenu de ton fichier tu peux le faire sans passer par un flux intermédiaire

Code :

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

string content = File.ReadAllText(path);

Un équivalent, sans objets de type flux, de ton code pour la méthode d'import, juste pour la forme

Code :

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

1
2
3
4
5
6
7
8
9
10
11
// Récupère les fichiers CSV contenu dans un répertoire donné
string[] files = Directory.GetFiles(path, "*.csv");
// Si on a trouvé des fichiers
if (files.Length != 0)
{
    // Remplace le foreach en prenant le dernier fichier
    // en admettant que le code donné soit vraiment celui utilisé
    string file = files[files.Length - 1];
    // Récupère le contenu du fichier
    string content = File.ReadAllText(file);
}

[Seth_75]

Dans ton import tu fais un appel à File.OpenRead. Cette méthode renvoie un objet de type FileStream que tu ne récupères pas et que donc tu ne libères pas au final. C'est peut être la raison pour laquelle la libération du StreamReader ne suffit pas à résoudre ton problème.

Au passage (en supposant que le code donné est vraiment celui utilisé) :
- la boucle foreach ne sert à rien dans l'état
- le File.OpenRead non plus
- le test d'existence du fichier est également inutile vu comment est récupéré le nom de ce fichier. Il vaut mieux voir si des fichiers ont été trouvés.

Sinon, pour récupérer le contenu de ton fichier tu peux le faire sans passer par un flux intermédiaire

Code :

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

string content = File.ReadAllText(path);

Un équivalent, sans objets de type flux, de ton code pour la méthode d'import, juste pour la forme

Code :

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

1
2
3
4
5
6
7
8
9
10
11
// Récupère les fichiers CSV contenu dans un répertoire donné
string[] files = Directory.GetFiles(path, "*.csv");
// Si on a trouvé des fichiers
if (files.Length != 0)
{
    // Remplace le foreach en prenant le dernier fichier
    // en admettant que le code donné soit vraiment celui utilisé
    string file = files[files.Length - 1];
    // Récupère le contenu du fichier
    string content = File.ReadAllText(file);
}

Merci pour ce cours de C# mais j'ai mentionné que je bossais sur la version 1.1 or la fonction READALLTEXT() n'est pas supporté elle fonctionne sous le framework 2.0 , 3.0 , 3.5 .

Donc ton code ne peut pas fonctionner merci quand même de m'aider.

Cordialement.

[StormimOn]

Erf, j'étais pourtant sûr qu'elle était disponible avec le 1.1

En tout cas, ça ne change rien concernant le fait que ton File.OpenRead renvoie un FileStream qui tu ne récupères pas et donc ne libère pas (Close ou Dispose). C'est sûrement la raison du problème si tu libères bien le StreamReader.

[Seth_75]

Erf, j'étais pourtant sûr qu'elle était disponible avec le 1.1

En tout cas, ça ne change rien concernant le fait que ton File.OpenRead renvoie un FileStream qui tu ne récupères pas et donc ne libère pas (Close ou Dispose). C'est sûrement la raison du problème si tu libères bien le StreamReader.

Ouppppssss!!!! désolé de répondre si tardivement je m'en excuse en tout cas même si ce que tu m'as donné ne fonctionne pas sous 1.1, la fonction File.OpenRead renvoie bien un stream et c'est ce qui me bloquait merci à tous de m'avoir aider , je suis pas doué j'ai cherché tout sur la msdn sauf cette fonction errrrrrffFF ... Quelle perte de temps !!! Merci encore StormimOn et les autres ...

Eu tout cas merci pour le lien sur les cours de formation pour apprendre la programmation C# c'est super : http://dotnet.developpez.com/cours/?page=csharp

Cordialement

[g_tarik0010]

Bonjour,
je voulais donner une sollution,car j'ai pas vu le flag resolu,

donc essaye de le mettre sur