Discussion :

[chrime71]

bonjour

Je cherche à remplacer une sous chaine dans une chaine typiquement utiliser la méthode replace de String. Et là le bas blesse !!

1/ Je n"oublie pas que les chaines sont immuables don je fais bien
String s = ma_string.replace("TOTO", "TATA");

Je lis donc un fichier ligne par ligne et je remplace dans les lignes les TOTO par TATA. Qd je sors en System.out je vois bien la chaine modifiée mais lorsque j'écris cette chaine dans un OutputStream, rien n'est modifié ?

Bug de cette méthode ? Bug dans l'OutputStream ? Bug de moi ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
BufferReader input = new BufferReader(New InputStreamReader(new FileInputStream(mon_fichier)));
DataOutputStream dos = new DataOutputStream(new FileOutputStream(mon_output)));
 
String line = null;
While ((line=input.readLine()) != null) {
      String tmp = line.replace("TOTO", "TATA");
      dos.writeBytes(tmp);
      dos.flush();
}
 
input.close();
dos.close();

[onylink]

Il ne serait pas plus simple de remplacer toutes tes occurrences de ton fichier avec un replaceAll() par exemple. Et ensuite, écrire ton fichier.

[chrime71]

J'ai essayé aussi le replaceAll et ca donne le même résultat !!!

ps : pour info je suis en 5.0

[thelvin]

bBug de cette méthode ? Bug dans l'OutputStream ? Bug de moi ?

En gras l'explication la plus probable.

Possibilités de choses qui peuvent causer ça :
- Il n'y a pas de TATA dans le fichier de départ, c'est donc assez normal qu'il n'y ait pas de modification.
- Au lieu de TATA, c'est un truc avec caractères spéciaux que tu cherches à remplacer, et tu ne gères pas correctement l'encodage du fichier lu. Ce qui fait la même chose que le point précédent.
- Le fichier lu et le fichier écrit sont un seul et même fichier. Dans ce cas-là les résultats sont imprévisibles (mais bon c'est étrange quand même).
- En fait ce n'est pas TOTO et TATA que tu cherches à remplacer, et un hasard monumental fait que ce que tu cherches à remplacer n'est pas correctement géré par writeBytes() (qui ne gère que les caractères iso-8859-1.)

[chrime71]

Alors

1/ Il y a bien TATA, qd je fais un System.out de mon programme je vois bien la ligne lue et la ligne transformée !!

2/ j'écris les données dans une fichier temporaire donc pas de lecture écriture dans un même fichier ce qui me parait déjà improbable à faire ...

3/ A la base mon fichier est en UTF-16

Je veux bien buguer mais tout de même

[onylink]

Pour éviter tout conflit, j'aurai extrait le texte de mon document dans un String par exemple pour faire un replaceAll().
Puis dans une second temps, j'aurai ré-écris le fichier.


Essaie déjà ça, car autant ça vient d'un conflit qui ne saute pas aux yeux.

[tchize_]

D'abord, utilise un Writer et non pas un Dataoutputstream. Dataoutputstream c'est pour du formatage de données, par pour des fichiers textuels.
Ensuite, si ton fichier est en utf-16, tu dois l'indiquer quand tu crée le Reader. La, t'es parti pour avoir non pas "TATA" dans ta String mais "T A T A " parce que, au mieux, t'es occupé de lire en utf-8. Pareil quand tu crée le writer, précise l'encodage.

Enfin, tu dis que "ca marche en System.out", mais je ne vois pas là dedans le code qui utilise System.out en parallèle pour te permettre de controler

[onylink]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
BufferedReader input = new BufferedReader(new InputStreamReader(new FileInputStream(mon_fichier)));
StringBuffer txt = new StringBuffer();
String line = null;
while ((line=input.readLine()) != null) {
	String tmp = line.replace("TOTO", "TATA");
	txt.append(tmp);
}
 
input.close();
 
DataOutputStream dos = new DataOutputStream(new FileOutputStream(mon_output));
dos.writeBytes(txt.toString());
dos.close();

Ce code marche. Il faudra juste penser à remettre les sauts de ligne. Mais tout marche.

Je ne connais pas ton problème, mais il est peut-être nécessaire de tout mettre en majuscule dans ta chaîne, pour ne pas loupé un "toto" par exemple.

[chrime71]

D'abord, utilise un Writer et non pas un Dataoutputstream. Dataoutputstream c'est pour du formatage de données, par pour des fichiers textuels.
Ensuite, si ton fichier est en utf-16, tu dois l'indiquer quand tu crée le Reader. La, t'es parti pour avoir non pas "TATA" dans ta String mais "T A T A " parce que, au mieux, t'es occupé de lire en utf-8. Pareil quand tu crée le writer, précise l'encodage.

Enfin, tu dis que "ca marche en System.out", mais je ne vois pas là dedans le code qui utilise System.out en parallèle pour te permettre de controler

Et un grand merci à Tchize .. le Writer est efficace (avec l'utf-16 de précisé pour l'input) ..

[thelvin]

2/ j'écris les données dans une fichier temporaire donc pas de lecture écriture dans un même fichier ce qui me parait déjà improbable à faire ...

Cela dépend des environnements. Mais le résultat est en effet aléatoire.

3/ A la base mon fichier est en UTF-16

Bon ben voilà. Il y a aucune chance que InputStreamReader devine qu'il doit lire en UTF-16. Du coup il voit des caractères séparés de caractères nuls.

Il faut indiquer que c'est de l'utf-16 :

Code :

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

new InputStreamReader(new FileInputStream(mon_fichier), "utf-16")

ou si c'est du little-endian sans BOM :

Code :

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

new InputStreamReader(new FileInputStream(mon_fichier), "utf-16LE")

[chrime71]

J'ai essayé en précisant utf-16 et ca ne changeait pas gd chose !!

Pour mon System.out qui n'est pas dans mon code, je ne l'ai pas mis mais sinon cela fonctionne.

Concernant TOTO et TATA ce ne sont que des exemples bateau pour mon problème mes strings à remplacer sont moins simples que cela

Je vais tenter le Writer pour voir si ca change qque chose !!

[thelvin]

J'ai essayé en précisant utf-16 et ca ne changeait pas gd chose !!

Si. Les problèmes que tu rencontrais ensuite étaient complètement différents.