Discussion :

[alavoler]

Bonjour à tous,

Existe une manière plus élégante de remplacer une expression dans un 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
static public void ReplaceInFile(string filePath, string searchText, string replaceText)
{
	using (StreamReader reader = new StreamReader(filePath))
	{
	    string content = reader.ReadToEnd();
	}
 
    	content = Regex.Replace(content, searchText, replaceText );
 
    	using (StreamWriter writer = new StreamWriter(filePath))
	{
   	     writer.Write(content);
	}
}

En l'occurence c'est pour un fichier HTML de quelques KO, mais sur un fichier de plusieurs MO... je pense que c'est pas le mieux ?

PS: je ne cherche pas l'optimisation absolu, mais plutôt un bon rapport entre lisibilité du code / performance pas trop désastreuse, et blindage du code

merci

[emmr.rida]

Pour les petits fichiers d'accord, mais, pour les fichiers volumineux vaux mieux charger le contenu du fichier bloc par bloc dans un buffer, effectuer le remplacement des données et après réécrir le buffer modifié dans le fichier.

[alavoler]

emmr.rida merci de ta réponse,

pour le moment j'arrive a créer le fichier en passant par un fichier "temporaire".

C'est possible d'optimiser sans passer par un tel fichier ?

Je ne trouve pas grand chose sur le net, pourtant pour un probleme commun... j'aimerais avoir un model "best practice" histoire d'avoir un truc robuste et propre

je suppose que ca peut interesser pas mal de gens ici...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
        public void ReplaceInFile(string pathFile, string oldStr, string newStr)
        {
            string line;
 
            using (TextReader tr = File.OpenText(pathFile))
            using (TextWriter tw = File.CreateText("ResulltFile"))
            {
                while ((line = tr.ReadLine()) != null)
                {
                    line = line.Replace(oldStr, newStr);
                    tw.WriteLine(line);
                }
            }
        }