remplacer une chaine ds un fichier rapidement !

**need2learn** · 22/07/2009, 13h48

Bonjour à tous,

je cherche une fonction qui me permet de remplacer une chaine dans fichier texte SANS lire ligne par ligne ... car cette méthode est très lente si le fichier est gros

je voudrais que ça soit aussi rapide que Ctrl-R Ctrl-S

merci d'avance !

**adiGuba** · 22/07/2009, 14h25

Salut,

On pourrait voir le code entier...

a++

**need2learn** · 22/07/2009, 14h33

euuh en fait je devais pas mettre le code source

le code que j'ai mis est juste pour montrer que la méthode est tros lente.

je cherche en fait tout simplement une fonction qui permet de remplacer une chaine de caractère dans un fichier text

**adiGuba** · 22/07/2009, 14h46

Code :

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

quickReplaceStringOnFile();

Moi non plus je ne met pas le code ce serait trop facile...

a++

**need2learn** · 22/07/2009, 14h53

???
je n'ai pas mis le code car je pense que ça n'aidera pas ... mais bon le voici :

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
 
	public static void removeAlertMsg(File f) {
 
		String chaine = "";
		String varName = null;
 
		// lecture du fichier texte
		try {
			InputStream ips = new FileInputStream(f);
			InputStreamReader ipsr = new InputStreamReader(ips);
			BufferedReader br = new BufferedReader(ipsr);
			String ligne;
			while ((ligne = br.readLine()) != null) {
				if (ligne.contains(ALERT_MSG)) {
					int start = ligne.indexOf(ALERT_MSG);
					varName = "$intern_" + ligne.substring(start - 20, start - 4).trim().split("\\$intern\\_")[1];
					System.out.println(varName);
					System.out.println(varName);
				}
 
				if (varName != null && ligne.contains(".alert(" + varName)) {
 
				}
				System.out.println(ligne);
				chaine += ligne + "\n";
			}
			br.close();
		} catch (Exception e) {
			System.out.println(e.toString());
		}
 
		// création ou ajout dans le fichier texte
		try {
			FileWriter fw = new FileWriter(f);
			BufferedWriter bw = new BufferedWriter(fw);
			PrintWriter fichierSortie = new PrintWriter(bw);
			fichierSortie.println(chaine + "\n test de lecture et écriture !!");
			fichierSortie.close();
			System.out.println("Le fichier " + f + " a été créé!");
		} catch (Exception e) {
			System.out.println(e.toString());
		}
	}

**adiGuba** · 22/07/2009, 15h11

Envoyé par need2learn

???
je n'ai pas mis le code car je pense que ça n'aidera pas ...

En même temps si je le demande c'est qu'il y a une raison

Mes remarques :

Comme je m'en doutais, ce n'est pas le readLine() qui est problématique (il est de toute manière bufférisé), mais bel et bien ton code : tu utilises l'opérateur + pour la concaténation de String, ce qui est catastrophique dans une boucle.
Pour générer des String il faut utiliser StringBuilder (voir StringBuffer si on a besoin de synchronisation).
Il manque les try/finally pour fermer les flux (ah si je pouvais recevoir 1€ à chaque fois que je dit cela ).
Ton code n'effectue aucun remplacement...
La partie effectuant l'écriture devrait être conditionné. C'est inutile lorsqu'il n'y a aucun remplacement.
La gestion des exceptions n'est pas terrible
"\n" n'est pas portable. Utilise System.getProperty("line.separator") à la place...
Si tu as de gros fichiers, il pourrait peut-être être intérressant d'ouvrir un autre fichier en écriture en même temps, et d'y écrire directement les données afin d'éviter de générer une grosse variable en mémoire...

a++

**tchize_** · 22/07/2009, 15h16

Envoyé par need2learn

Bonjour à tous,

je cherche une fonction qui me permet de remplacer une chaine dans fichier texte SANS lire ligne par ligne ... car cette méthode est très lente si le fichier est gros

Je pense pas que l'aspect ligne par ligne impacte énormément les performances. Et la conversion binaire -> texte est inévitable dans ton cas. Quand à lire tout le fichier... Je vois pas comment tu imagine qu'on puisse recherche un texte dans un fichier sans le lire complètement.

Dans ton code, ceci

Code :

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

chaine += ligne + "\n"

impacte fortement les performance. Si ton fichier fait, disons, 15M en mémoire cette ligne va faire ceci vers la fin de fichier

création d'un nouveau Stringbuffer avec la string (15M de mémoire allouée)
Agrandissement du StringBuffer pour effectuer le 'add' (de nouveau 15M d'alloué, transférés et les 15 précédent libéré ensuite
Conversion du Stringbuffer en String (de nouveau 15M alloué)
Abandon du StringBuffer(15M libéré)

Et ça, a chaque boucle. Tu devrais plutot écrire chaque ligne au fur et à mesure dans le fichier de destination en faisant ton filtrage.