Java : Ecriture BOM ds fichier

**cambouis** · 14/03/2009, 14h56

Bonjour,

Je travaille sur un projet de génération de diagramme dia à partir d'un fichier Excel. Afin d'être correctement interprété par Dia, le fichier généré doit commencer par les octets FFFE (caracètre BOM - Byte Ordre Mark). Je ne parviens pas à écrire ce caractère en début de fichier. Pouvez-vous m'aider ?

Voici mon code :

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
 
//copie de l'en-tête type de dia, avec le BOM
	FileInputStream in=new FileInputStream("Header.dia");
	FileOutputStream out=new FileOutputStream(strChemin);
	FileChannel src=in.getChannel();
	FileChannel dest=out.getChannel();
	src.transferTo(0, src.size(), dest);
	src.close();
	dest.close();
//a ce niveau, le fichier généré contient bien le BOM
 
	bw=new BufferedWriter(new FileWriter(new File(strCheminFichier), true));
	strTemp="\n";
	bw.write(strTemp);
	bw.flush();
	for (int i=0; i<nbNoeud; i++)
	{
	(...)
	//tous les arguments ci-dessous sont de type String
	bw.write(strBackground+currentNoeud.getConstat()+strEndString+strNewLine);
	bw.flush();
	//a la fin de la premiere boucle, le fichier contient toujours le BOM
	//après la deuxième boucle, le fichier ne contient plus le BOM !
	}
	bw.close();

Voyez-vous d'où vient le problème et comment le résoudre ?

**jiddou** · 14/03/2009, 15h10

Le problème est dans la boucle alors essaie de nous donner son contenu
Je veux voir le contenu de
strEndString
strNewLine
strBackground
Le maximum d'informations s'il te plait

**cambouis** · 14/03/2009, 21h21

Voici la boucle complète :

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
 
for (int i=0; i<nbNoeud; i++)
//parcours de tous les noeuds
	{
	numNoeud=i+1;
	currentNoeud=this.eltAt(i);
	x=currentNoeud.getX();
	y=currentNoeud.getY();
	//ajout d'un objet Box
	strTemp=strObjectBoxHead+i+"\">"+strNewLine;
	bw.write(strObjectBoxHead+i+"\">"+strNewLine);
	bw.flush();
	//ajout des coordonnées
	strTemp=strObjectPos+x+".0,"+y+".0\""+strEndLine;
	bw.write(strObjectPos+x+".0,"+y+".0\""+strEndLine);
	bw.flush();
	//ajout des caractéristiques 'corner'
	strTemp=strObjectEltCorner+x+".0,"+y+".0\""+strEndLine;
	bw.write(strObjectEltCorner+x+".0,"+y+".0\""+strEndLine);
	bw.flush();
	//TODO : calculer dynamiquement le EW
	strTemp=strObjectEltWidth+"5"+"\""+strEndLine;
	bw.write(strObjectEltWidth+"5"+"\""+strEndLine);
	bw.flush();
	//TODO : calculer dynamiquement le EH
	strTemp=strObjectEltHeight+"1.9000000000000004"+"\""+strEndLine;
	bw.write(strObjectEltHeight+"1.9000000000000004"+"\""+strEndLine);
	bw.flush();
	//ajout du texte
	strTemp=strBackground+currentNoeud.getConstat()+strEndString+strNewLine;
	bw.write(strBackground+currentNoeud.getConstat()+strEndString+strNewLine);
	bw.flush();
	//fermeture de l'objet
	strTemp=strEndObject;
	bw.write(strEndObject);
	bw.flush();
}

**cambouis** · 14/03/2009, 21h26

Pardon, j'ai oublié la déclaration des variables ... Ces déclarations sont faites avant la boucle. Voici le contenu des variables :

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
 
	String strNewLine="\n";		//saut de ligne
	//chaînes préformatées au format dia
	String strObjectBoxHead="    <dia:object type=\"Flowchart - Box\" version=\"0\" id=\"O";
	String strEndLine="/>"+strNewLine;
	String strRealVal="        <dia:real val=\"";
	String strBoolVal="        <dia:boolean val=\"";
	String strPointVal="        <dia:point val=\"";
	String strEndString="#</dia:string>";
	String strEndAttribute="      </dia:attribute>"+strNewLine;
	String strAttributeName="      <dia:attribute name=\"";
 
	//chaînes préformatées au format dia incluant plusieurs lignes
	String strObjectPos=strAttributeName+"obj_pos\">"+strNewLine;
	strObjectPos+=strPointVal;
 
	String strObjectEltCorner=strEndAttribute;
	strObjectEltCorner+=strAttributeName+"obj_bb\">"+strNewLine;
	strObjectEltCorner+="        <dia:rectangle val=\"0.0,0.0;0.0,0.0\"/>"+strNewLine;
	strObjectEltCorner+=strEndAttribute;
	strObjectEltCorner+=strAttributeName+"elem_corner\">"+strNewLine;
	strObjectEltCorner+=strPointVal;
 
	String strObjectEltWidth=strEndAttribute;
	strObjectEltWidth+=strAttributeName+"elem_width\">"+strNewLine;
	strObjectEltWidth+=strRealVal;
 
	String strObjectEltHeight=strEndAttribute;
	strObjectEltHeight+=strAttributeName+"elem_height\">"+strNewLine;
	strObjectEltHeight+=strRealVal;
 
	String strBackground=strEndAttribute;
	strBackground+=strAttributeName+"show_background\">"+strNewLine;
	strBackground+=strBoolVal+"true\"/>"+strNewLine;
	strBackground+=strEndAttribute;
	strBackground+=strAttributeName+"padding\">"+strNewLine;
	strBackground+=strRealVal+"0.5\"/>"+strNewLine;
	strBackground+=strEndAttribute;
	strBackground+=strAttributeName+"text\">"+strNewLine;
	strBackground+="        <dia:composite type=\"text\">"+strNewLine;
	strBackground+=strAttributeName+"string\">"+strNewLine;
        strBackground+="            <dia:string>#";

L'objet currentNoeud est de type Noeud, une classe que j'ai définit. Les types renvoyés par les méthodes appelés ci-dessus :
getX() et getY() renvoient des entiers (int)
getConstat() renvoient une chaîne de caractères (String)

**jiddou** · 14/03/2009, 22h49

J'ai pas vu réellement ou est le problème mais je suis sûre que tu peux le regler avec juste un peu d'algorithme.Voilà donc comme algo je te propose d'améliorer ton code en écrivant dans le fichier après ta boucle.Donc ta boucle ne sert qu'à préparer la chaîne à écrire avec plusieurs concaténation puis l'écriture se passe à la fin.

Voici à quoi va ressembler ton code:

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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
String strChaineFinale="";

String strNewLine="\n"; //saut de ligne
//chaînes préformatées au format dia
String strObjectBoxHead=" <dia:object type=\"Flowchart - Box\" version=\"0\" id=\"O";
String strEndLine="/>"+strNewLine;
String strRealVal=" <dia:real val=\"";
String strBoolVal=" <dia:boolean val=\"";
String strPointVal=" <dia:point val=\"";
String strEndString="#</dia:string>";
String strEndAttribute=" </dia:attribute>"+strNewLine;
String strAttributeName=" <dia:attribute name=\"";

//chaînes préformatées au format dia incluant plusieurs lignes
String strObjectPos=strAttributeName+"obj_pos\">"+strNewLine;
strObjectPos+=strPointVal;

String strObjectEltCorner=strEndAttribute;
strObjectEltCorner+=strAttributeName+"obj_bb\">"+strNewLine;
strObjectEltCorner+=" <dia:rectangle val=\"0.0,0.0;0.0,0.0\"/>"+strNewLine;
strObjectEltCorner+=strEndAttribute;
strObjectEltCorner+=strAttributeName+"elem_corner\">"+strNewLine;
strObjectEltCorner+=strPointVal;

String strObjectEltWidth=strEndAttribute;
strObjectEltWidth+=strAttributeName+"elem_width\">"+strNewLine;
strObjectEltWidth+=strRealVal;

String strObjectEltHeight=strEndAttribute;
strObjectEltHeight+=strAttributeName+"elem_height\">"+strNewLine;
strObjectEltHeight+=strRealVal;

String strBackground=strEndAttribute;
strBackground+=strAttributeName+"show_background\">"+strNewLine;
strBackground+=strBoolVal+"true\"/>"+strNewLine;
strBackground+=strEndAttribute;
strBackground+=strAttributeName+"padding\">"+strNewLine;
strBackground+=strRealVal+"0.5\"/>"+strNewLine;
strBackground+=strEndAttribute;
strBackground+=strAttributeName+"text\">"+strNewLine;
strBackground+=" <dia:composite type=\"text\">"+strNewLine;
strBackground+=strAttributeName+"string\">"+strNewLine;
strBackground+=" <dia:string>#";


for (int i=0; i<nbNoeud; i++)
//parcours de tous les noeuds
{
numNoeud=i+1;
currentNoeud=this.eltAt(i);
x=currentNoeud.getX();
y=currentNoeud.getY();
//ajout d'un objet Box
strTemp=strObjectBoxHead+i+"\">"+strNewLine;
strChaineFinale+=strTemp;
//ajout des coordonnées
strTemp=strObjectPos+x+".0,"+y+".0\""+strEndLine;
strTemp=strObjectBoxHead+i+"\">"+strNewLine;
strChaineFinale+=strTemp;
//ajout des caractéristiques 'corner'
strTemp=strObjectEltCorner+x+".0,"+y+".0\""+strEndLine;
strChaineFinale+=strTemp;
//TODO : calculer dynamiquement le EW
strTemp=strObjectEltWidth+"5"+"\""+strEndLine;
strChaineFinale+=strTemp;
//TODO : calculer dynamiquement le EH
strTemp=strObjectEltHeight+"1.9000000000000004"+"\""+strEndLine;
strChaineFinale+=strTemp;
//ajout du texte
strTemp=strBackground+currentNoeud.getConstat()+strEndString+strNewLine;
strChaineFinale+=strTemp;
} 

bw.write(strTemp);
bw.flush();
bw.close();

**jiddou** · 14/03/2009, 22h51

En fait il y'a quelque qui est louche dans ton code. A quoi sert strTemp si tu écrases toujours son contenu sans l'utiliser avant.

**cambouis** · 15/03/2009, 08h58

C'est juste un oubli de ma part : initialement, j'utilisais la chaîne strTemp dans mes appels à bw.write. Compte tenu du problème que je rencontrais,j'ai modifié l'appel pour vérifier si ca ne venait pas de l'initialisation de strTemp. J'ai oublié de remettre l'appel initial ensuite

J'ai modifié mon code selon tes préconisations, mais cela ne change rien

Vois-tu une solution pour que je puisse inscrire cet octet en début de fichier ?
Et pour ton optimisation, est-ce que cela ne risque pas d'être trop gourmand en mémoire ?

**gulain** · 15/03/2009, 09h12

Les liens suivants peuvent peut-être t'aider :
http://bugs.sun.com/bugdatabase/view...bug_id=4508058
http://bugs.sun.com/bugdatabase/view...bug_id=6378911.

**jiddou** · 15/03/2009, 14h09

Non t'inquiete pas trop pour la memoire puisque c'est une chaine de caractère c'est l'accès disque qui est inquietant vu sa lenteur donc avec plusieurs accès disque on ne gagne pas.
Envoies moi ton code modifié.

**cambouis** · 15/03/2009, 19h50

Voila le code modifié :

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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
	public boolean SaveToDia (String strCheminFichier)
	{
		try
		{
			//Handle de fichiers
			FileInputStream in=new FileInputStream("Header.dia");
			FileOutputStream out=new FileOutputStream(strCheminFichier);
			FileChannel src=in.getChannel();
			FileChannel dest=out.getChannel();
			BufferedReader br=null;		//fichier en entrée
			BufferedWriter bw=null;		//fichier en sortie
			String strNewLine="\n";		//saut de ligne
			//chaînes préformatées au format dia
			String strObjectBoxHead="    <dia:object type=\"Flowchart - Box\" version=\"0\" id=\"O";
			String strEndLine="/>"+strNewLine;
			String strRealVal="        <dia:real val=\"";
			String strBoolVal="        <dia:boolean val=\"";
			String strPointVal="        <dia:point val=\"";
			String strEndString="#</dia:string>";
			String strEndAttribute="      </dia:attribute>"+strNewLine;
			String strAttributeName="      <dia:attribute name=\"";
 
			//chaînes préformatées au format dia incluant plusieurs lignes
			String strObjectPos=strAttributeName+"obj_pos\">"+strNewLine;
			strObjectPos+=strPointVal;
 
			String strObjectEltCorner=strEndAttribute;
			strObjectEltCorner+=strAttributeName+"obj_bb\">"+strNewLine;
			strObjectEltCorner+="        <dia:rectangle val=\"0.0,0.0;0.0,0.0\"/>"+strNewLine;
			strObjectEltCorner+=strEndAttribute;
			strObjectEltCorner+=strAttributeName+"elem_corner\">"+strNewLine;
			strObjectEltCorner+=strPointVal;
 
			String strObjectEltWidth=strEndAttribute;
			strObjectEltWidth+=strAttributeName+"elem_width\">"+strNewLine;
			strObjectEltWidth+=strRealVal;
 
			String strObjectEltHeight=strEndAttribute;
			strObjectEltHeight+=strAttributeName+"elem_height\">"+strNewLine;
			strObjectEltHeight+=strRealVal;
 
			String strBackground=strEndAttribute;
			strBackground+=strAttributeName+"show_background\">"+strNewLine;
			strBackground+=strBoolVal+"true\"/>"+strNewLine;
			strBackground+=strEndAttribute;
			strBackground+=strAttributeName+"padding\">"+strNewLine;
			strBackground+=strRealVal+"0.5\"/>"+strNewLine;
			strBackground+=strEndAttribute;
			strBackground+=strAttributeName+"text\">"+strNewLine;
			strBackground+="        <dia:composite type=\"text\">"+strNewLine;
			strBackground+=strAttributeName+"string\">"+strNewLine;
	        strBackground+="            <dia:string>#";
 
	        String strEndObject=strEndAttribute;
	        strEndObject+=strAttributeName+"font\">"+strNewLine;
	        strEndObject+="            <dia:font family=\"sans\" style=\"0\" name=\"Helvetica\"/>"+strNewLine;
	        strEndObject+=strEndAttribute;
	        strEndObject+=strAttributeName+"height\">"+strNewLine;
	        strEndObject+=strRealVal+"0.80000000000000004\"/>"+strNewLine;
	        strEndObject+=strEndAttribute;
	        strEndObject+=strAttributeName+"pos\">"+strNewLine;
	        strEndObject+=strPointVal+"0.0,0.0\"/>"+strNewLine;
	        strEndObject+=strEndAttribute;
	        strEndObject+=strAttributeName+"color\">"+strNewLine;
	        strEndObject+="            <dia:color val=\"#000000\"/>"+strNewLine;
	        strEndObject+=strEndAttribute;
	        strEndObject+=strAttributeName+"alignment\">"+strNewLine;
	        strEndObject+="            <dia:enum val=\"1\"/>"+strNewLine;
	        strEndObject+=strEndAttribute;
	        strEndObject+="        </dia:composite>"+strNewLine;
	        strEndObject+=strEndAttribute;
	        strEndObject+="    </dia:object>"+strNewLine;
 
	        String strEndFile="  </dia:layer>"+strNewLine;
	        strEndFile+="</dia:diagram>";
	        String strTemp=null;		//copie temporaire de la chaine a ecrire ds le fichier dia
 
 
			int nbNoeud=this.taille;	//nombre de noeuds contenus dans le graphe
			Noeud currentNoeud=null;	//noeud en cours de traitement
			int numNoeud=0;				//numéro de ce noeud
			int x,y;					//coordonnées de ce noeud
 
			//copie de l'en-tête dia commun à tous les diagrammes
			src.transferTo(0, src.size(), dest);
			src.close();
			dest.close();
 
			//ajout des parties spécifiques : textbox et liens
			//ouverture du fichier en mode 'append'
			bw=new BufferedWriter(new FileWriter(new File(strCheminFichier), true));
			strTemp="\n";
			for (int i=0; i<nbNoeud; i++)
			//parcours de tous les noeuds
			{
				numNoeud=i+1;
				currentNoeud=this.eltAt(i);
				x=currentNoeud.getX();
				y=currentNoeud.getY();
				//ajout d'un objet Box
				strTemp+=strObjectBoxHead+i+"\">"+strNewLine;
				//ajout des coordonnées
				strTemp+=strObjectPos+x+".0,"+y+".0\""+strEndLine;
				//ajout des caractéristiques 'corner'
				strTemp+=strObjectEltCorner+x+".0,"+y+".0\""+strEndLine;
				//calculer dynamiquement le EW
				strTemp+=strObjectEltWidth+"5"+"\""+strEndLine;
				//calculer dynamiquement le EH
				strTemp+=strObjectEltHeight+"1.9000000000000004"+"\""+strEndLine;
				//ajout du texte
				strTemp+=strBackground+currentNoeud.getConstat()+strEndString+strNewLine;
				//fermeture de l'objet
				strTemp+=strEndObject;
			}
			//fermeture de la feuille dia
			strTemp+=strEndFile;
			bw.write(strTemp);
			bw.flush();
			bw.close();
			return true;
		}
		catch (Exception e)
		{
			System.out.println("Erreur inattendue"+e.toString());
			return false;
		}
	}

**jiddou** · 15/03/2009, 20h43

Ajoute un

Code :

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

1
2
3
 
out.close();
in.close();

Après

Code :

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

1
2
3
 
src.close();
dest.close();

**cambouis** · 15/03/2009, 21h23

J'ai modifié le code en conséquence, mais cela n'a pas résolut mon problème

**jowo** · 16/03/2009, 08h29

Bonjour,

Ta méthode contient de nombreuses "mauvaises pratiques" qui alourdissent inutilement ton code.

Utilisation d'une classe String dans une boucle au lieu de la classe
StringBuffer ou si disponible StringBuilder.
Construction d'une chaîne de caractères comme tampon avec BufferedWriter. Chacun sa responsabilité et son travail, BufferedWriter fait très le sien comme tampon.

Ces remarques ne vont pas directement résoudre ton problème.

Un autre point que j'aimerais soulever, pourquoi ne pas nous envoyer un code "purifié" qui reproduit ton problème. Je suis sûr que cela nous aiderait et t'aiderait à le résoudre.

PS: As-tu lu attentivement la javadoc concernant la classe FileWriter et le Charset?

Java : Ecriture BOM ds fichier

Entrée/Sortie Java

Discussions similaires

Partager

Partager