Discussion :

[oni13]

J'arrive pas à comprendre ce que tu veux, ni ce que tu as actuellement.

Tu veux ca:
2012-06-07;06:25:30;1ScUGn-0002IB-7j;<=;hom@bd.missions-acf.org
Mais tu as ca ?
2012-06-0706:25:301ScUGn-0002IB-7j<=hom@bd.missions-acf.org

Alors avant de faire

Code :

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

writer.writeNext(champ);

Tu ajoutes ";" apres chaque champ[i] (sauf le dernier)

Code :

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

1
2
3
4
for(int i=0; i<champ.length-1 ;i++)
   champ[i]+=";";
 
writer.writeNext(champ);

Tu ne veux pas tout écrire?
Alors fais le ménage:

Si tu veux les 5 premiers champs:

Code :

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

1
2
3
4
5
if (champ.length > 5) 
     for(int i=5; i<champ.length ; i++)
          champ[i]="";
 
writer.writeNext(champ);

[tchize_]

dans la seulle classe csvwriter que je connaisse, writeNext prend un tableau de champs à écrire et met automatiquement les séparateur, les guillements, les échappements. La seule chose c'est qu'il faut instancier le csvwriter avec les paramètres que tu veux utiliser (voir la doc de la classe).

Pour le reste, le code est un sacré méli-melo. tu stocke dans des objets TraceEmail , que tu n'utilise pas au final, que tu met dans une liste, que tu n'utilise pas, et tu balance directement vers ta sortie tes données non traitées alors que tu t'es cassé le cul à les traiter. Alors faut réfléchir, sortir une feuille de papier est écrire ton algorithme. Et normalement tu devrais arriver à quelque chose comme ça

1. Ouverture du fichier
2. Pour chaque ligne
   
   1. parser la ligne
   2. créer un TraceEmail
   3. ajouter ce TraceEmail à ma List
3. fermer le fichier
4. ouvrir en écriture avec csvwriter et les bon séparateurs
5. pour chaque TraceEmail de la liste
   
   1. créer une String[] des éléments dont j'ai besoin
   2. la donner à CsvWriter
6. fermer le fichier

[drac08]

Oui c'était la pagaille

Voilà ou j'en suis :

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
44
45
46
47
48
49
50
51
52
53
54
55
try{
			String f = "/Users/Lazyeight/Documents/workspace/LogRod/exim_mainlog"; // mon fichier
			 String chaine = "";  //initialisation des variables
			  String ligne;
			   InputStream ips=new FileInputStream(f); //déclarations des objets de lecture
			    InputStreamReader ipsr=new InputStreamReader(ips);
			     BufferedReader br=new BufferedReader(ipsr);
			       FileWriter fw = new FileWriter("TEST3.csv", false);
			       BufferedWriter output = new BufferedWriter(fw);
			       
			     List<TraceEmail> list = new ArrayList<TraceEmail>(); //déclaration de notre ArrayList

			     
			     	while ((ligne=br.readLine())!=null){  //tant qu'il y a des lignes
			     		chaine = ligne;
			     		 String[] champ = chaine.split(" "); //on stocke chaque bloc séparé par le caractère espace
			     		  TraceEmail trace = new TraceEmail(); // le constructeur
			     		
			     		 if (champ[2].contains("1S") && champ[3].contains("<=") || champ[3].contains("->") || champ[3].contains("=>")) {
			     			
			     				trace.setDmail(champ[0]);
			     				trace.setHmail(champ[1]);
			     				trace.setIdmail(champ[2]);
			     				if (champ[3].contains("<=")) { 
			     				/*	if (! champ[8].contains("S=")) {
					     				trace.setTaille(champ[8]);
					     				} else trace.setTaille(champ[9]); 
					     		*/ //Pour cette partie ça ne fonctionne pas car forcement sur chacune de mes lignes je n'ai pas forcement de champ 8 et 9 du coup j'ai un jolie OutOfBound :'( une idée de comment je pourrais procéder ?
			     				trace.setDirection("expediteur");
			     				}
			     				else if (champ[3].contains("=>") || champ[3].contains("->") ) { 
				     				trace.setDirection("destinataire");
				     				}
			     				if (champ[4].contains("@")) //ne rien faire et traiter tab[6]
			     				{
			     				trace.setAmail(champ[4]); 
			     				} else if (champ[5].contains("@"))
			     				{ trace.setAmail(champ[5]); }
			     				else continue;
			     					
			     				list.add(trace);
			     			
			     				Collections.sort(list);
			     				
				} else { continue; }

			     	}
output.append("Date"+";"+"Heure"+";"+"Id"+";"+"Direction"+";"+"Adresse Mail"+";"+"Taille"+"\n");
			     	for(TraceEmail mail : list) {
			     		output.write(mail.getDmail() +";"+ mail.getHmail() +";"+ mail.getIdmail() +";"+ mail.getDirection() +";"+ mail.getAmail() +";"+ mail.getTaille() + "\n");
			     		output.flush();
					}
			     	System.out.println("Fichier Crée");
			br.close();
			output.close();

ça commence à ressembler à quelque chose non ?

[deathness]

Petite remarque, tu devrais déclarer String[] champ et TraceEmail trace en dehors du while.

[tchize_]

Petite remarque, tu devrais déclarer String[] champ et TraceEmail trace en dehors du while.

au contraire, il ne servent que dans la boucle, ils n'ont pas de raison de se trouver en dehors.

Par contre, le Collections.sort(list); gagnerait à sortir

[drac08]

Ah oui on gagne en perf en effet..

Par contre j'ai un petit pb je ne sais pas comment je peux faire pour exploiter le S=xxxxx correspondant à la taille de mon mail.
Du fait qu'on ne retrouve pas ce champ sur chacune des lignes de mon fichier log (champ 8 ou 9 dépendamment des lignes)

j'ai essayé ceci dans mon while qui lit mes lignes mais forcement j'ai un jolie OutOfBound vu que certaines lignes n'on pas de champ 8 et 9..

Code :

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

1
2
3
if (! champ[8].contains("S=")) {
				      trace.setTaille(champ[8]);
} else trace.setTaille(champ[9]);

quelqu'un aurait-il une solution ?

d'avance merci

[tchize_]

j'ai essayé ceci dans mon while qui lit mes lignes mais forcement j'ai un jolie OutOfBound vu que certaines lignes n'on pas de champ 8 et 9..

Ben la solution c'est simplement de tester la taille de ton tableau :/