remplacer une chaine ds un fichier rapidement !

Version imprimable

22/07/2009, 12h48
need2learn

remplacer une chaine ds un fichier rapidement !

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 :roll:

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

merci d'avance !
22/07/2009, 13h25
adiGuba

Salut,

On pourrait voir le code entier...

a++
22/07/2009, 13h33
need2learn

euuh en fait je devais pas mettre le code source :roll:
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 ;)
22/07/2009, 13h46
adiGuba

Code:

quickReplaceStringOnFile();

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

a++

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

Code:

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());
		}
	}

22/07/2009, 14h11
adiGuba
Citation:

Envoyé par need2learn

:roll:???
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.
  :arrow: 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 :D).
- Ton code n'effectue aucun remplacement... :koi:
- 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++
22/07/2009, 14h16
tchize_
Citation:

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 :roll:

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:

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.
05/08/2009, 19h30
need2learn

parfait ! ça marche en même pas une seconde ;)

merci adiGuba & tchize_