Ecriture d'une list() dans un fichier Pb d'optimisation

**sebastien2222** · 13/06/2006, 15h16

bonjour,

je souhaiterais avoir un avis sur une solution technique.

je dispose d'une application qui fait des calculs et qui initialise une List() avec 3000 "elements" de 100 caracteres (en moyenne). (ici la list me sert de tampon pour eviter de stocker tes les resultats des calculs en memoire)

jusqu' la pas de Pb

apres je dois ecrire dans un fichier les resultats (3000 x 100 caracteres en moyenne) , pour cela j'utilise une boucle for qui ajoute 100caracteres + 1saut de ligne .... jusqu'a disposé d'un string de 300000 caracteres qui seront ecrie (pas de Pb non plus)

Mon pb vient du temps mis par ma boucle for pour realiser l'initialisation de mon tampon de 300000 caracteres.

Donc ma question : est-il possible de gagner du temps dans mon for ?(exemple si j'ai 1 000 000 de resultats divisé par 3000 et que le for prend 15 sec je dois attendre 5000 secondes avant de pouvoir disposé de mon fichier de sortie.)

je rajoute un peu de code pour mieux comprendre.

Merci pour vos conseils

Sebastien

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
 
//traitement qui initialise list (3000 "lignes" de 100 caracteres) 
 
if (list.size()==3000)
{
 
	for (int boucle=0;boucle<list.size();boucle++)
	{
 
		if (tampon=="")
		{
			tampon = list.get(boucle).toString();	
		}
		else							
		{
			tampon = tampon+"\n"+list.get(boucle).toString();
		}
 
 
	}
	System.out.println("Ecriture ds le fichier.txt");
	Ecriture(tampon);
	tampon="";
	list.clear();
}

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
 
		public static void Ecriture(String tampon)
		{
			try 
			{		 		
				FileWriter flux = new FileWriter("EnTest.txt",true);
				BufferedWriter tampon = new BufferedWriter(flux);
 
				tampon.write(tampon);			
				tampon.newLine();			  
				tampon.close();				
				flux.close();
			}
			catch (IOException e ) 
			{
				System.out.println("ERREUR : Il y a eu une erreur lors de l'ecriture des resultats dans le fichier ");
				e.printStackTrace();
			}
 
		}

**g_rare** · 13/06/2006, 15h36

C'est faux ton truc (enfin il me semble sérieusement) !
Tu vas te retrouver avec un fichier du type :

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
1e element de ma liste

1e element de ma liste
2e element de ma liste

1e element de ma liste
2e element de ma liste
3e element de ma liste

1e element de ma liste
2e element de ma liste
3e element de ma liste
4e element de ma liste

...

car à chaque passage tu ajoutes à ton fichier tout ce que tu lui as déjà écrit, plus la nouvelle.

**le y@m's** · 13/06/2006, 15h40

Pour la concaténation tu devrais passer par un StringBuilder :
cf FAQ : Comment concatener des chaînes de caractères ?

Code :

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

1
2
3
4
5
6
7
 
StringBuilder tampon = new StringBuilder();
for (int boucle=0;boucle<list.size();boucle++) {
     tampon.append(list.get(boucle).toString());
}
System.out.println("Ecriture ds le fichier.txt");
Ecriture(tampon);

**sebastien2222** · 13/06/2006, 16h07

Envoyé par g_rare

C'est faux ton truc (enfin il me semble sérieusement) !
Tu vas te retrouver avec un fichier du type :

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
1e element de ma liste

1e element de ma liste
2e element de ma liste

1e element de ma liste
2e element de ma liste
3e element de ma liste

1e element de ma liste
2e element de ma liste
3e element de ma liste
4e element de ma liste

...

car à chaque passage tu ajoutes à ton fichier tout ce que tu lui as déjà écrit, plus la nouvelle.

regarde mieux tu verras que cela fonctionne !

Pour la concaténation tu devrais passer par un StringBuilder :
cf FAQ : Comment concatener des chaînes de caractères ?

Code :
StringBuilder tampon = new StringBuilder();
for (int boucle=0;boucle<list.size();boucle++) {
tampon.append(list.get(boucle).toString());
}
System.out.println("Ecriture ds le fichier.txt");
Ecriture(tampon);

ok merci pour l'information je vais essayer avec StringBuffer a cause que je dois rester en jdk 1.4.

par contre j'ai fais differents essai avec un string "normal" et la taille de la list influe beaucoup sur la rapidité ..... je vais essayer avec StringBuffer pour voir si cela est plus rapide.

encore merci pour l'info.

Sébastien

**KiLVaiDeN** · 13/06/2006, 16h11

StringBuilder c'est sous la JDK 1.5 Tiger, sinon tu utilises StringBuffer; StringBuilder est à utiliser si tu n'as pas à gérer le threading.

A+

**Satch** · 13/06/2006, 16h16

Fais aussi attention à ça :

Code :

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

if (tampon=="")

utilise plutôt equals pour comparer 2 String.
Il vaudrait mieux faire un truc du genre :

Code :

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

1
2
 
if ("".equals(tampon))

**g_rare** · 13/06/2006, 16h18

Envoyé par sebastien2222

regarde mieux tu verras que cela fonctionne !

+1

Par contre, il me semble que les Buffers existe pour "flusher" régulièrement les données (pas un seul gros bloc) :

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
public static void Ecriture(List list) {
  FileWriter flux = null;
  BufferedWriter tampon = null;
  try {		 		
    flux = new FileWriter("EnTest.txt",true);
    tampon = new BufferedWriter(flux);
    for (int boucle=0;boucle<list.size();boucle++) {		
      tampon.write(list.get(boucle).toString());			
      tampon.newLine();			  
      tampon.flush();
  }
  catch (IOException e ) {
    System.out.println("ERREUR : Il y a eu une erreur lors de l'ecriture des resultats dans le fichier ");
    e.printStackTrace();
  }
  finally {
    if (tampon != null) {
      tampon.close();				
    }
    if (flux != null) {
      flux.close();
    }
  }			
}