Java

BufferedReader qui implémente Iterator

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : BufferedReader qui implémente Iterator

charly007

Salut,

J'ai besoin de lire un fichier texte ligne par ligne.
Je pense utiliser un BufferedReader.

Sans abstraction je ferais :

Code :

BufferedReader in = new BufferedReader(new FileReader("C:/tests/error_log" ));
String s;
while ((s = in.readLine()) != null) {
// Traitement
}

Mais j'aurais voulu ajouter un peu d'abstraction, et donc permettre aux classes utilisatrices de travailler avec un Iterator, du style :

Code :

FileParser in = new FileParser(new FileReader("C:/tests/error_log" ));
String s;
Iterator it = in.iterator();
while (it.hasNext()) {
s = (String) it.next();
// Traitement
}

Je pensais créer une classe héritant de BufferedReader et implémentant Iterator, et ensuite définir les méthodes next() et hasNext().

1. Est-ce une bonne idée ?
2. Pour implémenter la méthode hasNext() je pensais faire comme ça :

Code :

public boolean hasNext() {
boolean hasNext;
// Marquer la position pour la lecture future
this.mark(1);
hasNext = this.read() != -1;
// Revenir à la position initiale
this.reset();
return hasNext;
}

Mais ça me parait un peu gourmand en traitement.

3. Comment récupérer l'objet Iterator ?
4. Pour l'héritage, j'ai trouvé l'idée sur un autre site, je ne sais pas comment la justifier :sweat:

Merci

:jap:

Message cité 1 fois
Message édité par charly007 le 27-11-2005 à 02:30:51

Publicité

masklinn

í dag viðrar vel til loftárása

charly007 a écrit :

1. Est-ce une bonne idée ?

IMO oui, c'est d'ailleurs ce que fait Python de manière standard (de plus cela permet d'utiliser n'importe quel type de donnée itérable comme source de données manipulables, très pratique pour des tests ou pour réutiliser tes fonctionalités)

Citation :

2. Pour implémenter la méthode hasNext() je pensais faire comme ça :

Code :

public boolean hasNext() {
boolean hasNext;
// Marquer la position pour la lecture future
this.mark(1);
hasNext = this.read() != -1;
// Revenir à la position initiale
this.reset();
return hasNext;
}

Mais ça me parait un peu gourmand en traitement.

BufferedReader.ready() ne suffirait-il pas?

Citation :

3. Comment récupérer l'objet Iterator ?

Code :

public class IterableReader extends BufferedReader implements Iterator {
/* snip */
public Iterator iterator() {
return (Iterator) this;
}
/* snip */
}

Un truc du style [:spamafote]

Et n'oublie pas qu'Iterator demande next(), hasNext() et remove() (qui peut ne rien faire, l'opération est optionnelle)

Citation :

4. Pour l'héritage, j'ai trouvé l'idée sur un autre site, je ne sais pas comment la justifier :sweat:

Message cité 1 fois

---------------
I mean, true, a cancer will probably destroy its host organism. But what about the cells whose mutations allow them to think outside the box by throwing away the limits imposed by overbearing genetic regulations? Isn't that a good thing?

the real moins moins

ton truc doit pas implémenter Iterator mais Iteratable.
et ne pas hériter.

Message cité 1 fois

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

the real moins moins

(mask, ton cast ne sert à rien)

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

charly007

masklinn a écrit :

BufferedReader.ready() ne suffirait-il pas?

En effet, BufferedReader.ready() suffit.

masklinn a écrit :

Et n'oublie pas qu'Iterator demande next(), hasNext() et remove() (qui peut ne rien faire, l'opération est optionnelle)

Oui, concernant remove() je lève UnsupportedOperationException().

the real moins moins a écrit :

ton truc doit pas implémenter Iterator mais Iteratable.
et ne pas hériter.

Je dois utiliser le jdk 1.3, donc pas de Iterable.

Voici donc le code corrigé par vos conseils :

Code :

public class IterableReader implements Iterator {
private BufferedReader br;
public IterableReader(Reader in) {
br = new BufferedReader(in);
}
public Iterator iterator() {
return this;
}
public boolean hasNext() {
boolean hasNext = false;
try {
hasNext = br.ready();
} catch (IOException e) {
// TODO Lever une exception
}
return hasNext;
}
public Object next() {
try {
return (Object) br.readLine();
} catch (IOException e) {
// TODO Lever une exception
}
return null;
}
public void remove() {
throw new UnsupportedOperationException();
}
}

Code :

public static void main(String[] args) {
try {
IterableReader in = new IterableReader(new FileReader("C:/tests/error_log" ));
String s;
Iterator it = in.iterator();
while (it.hasNext()) {
s = (String) it.next();
// Traitement
System.out.println(s);
}
} catch (FileNotFoundException e) {
// TODO Auto-generated catch block
e.printStackTrace();
}
}

Merci :jap:

Message cité 1 fois
Message édité par charly007 le 27-11-2005 à 12:25:12

benou

ouais mais nan, ready ça sert pas à ça ...

Citation :

True if the next read() is guaranteed not to block for input, false otherwise

et puis ta méthode iterator() elle sert plus à grand chose ...

je verrais plutôt un truc comme ca :

Code :

import java.io.BufferedReader;
import java.io.IOException;
import java.util.Iterator;
public class LineIterator implements Iterator {
private final BufferedReader reader;
private String line;
public LineIterator(BufferedReader reader) {
this.reader = reader;
this.line = null;
}
public Object next() {
if ((this.line == null) && (! this.hasNext())) {
return null;
} else {
String toReturn = this.line;
this.line = null;
return toReturn;
}
}
public boolean hasNext() throws IOEWrapper {
if (this.line == null) {
try {
line = this.reader.readLine();
} catch (IOException e) {
throw new IOEWrapper(e);
}
return (line != null);
} else {
return true;
}
}
public void remove() {
throw new UnsupportedOperationException("LineIterator does not support remove()" );
}
// ---------------- IOEWrapper --------------------------
public static class IOEWrapper extends RuntimeException {
private final IOException ioe;
public IOEWrapper(IOException ioe) {
this.ioe = ioe;
}
public IOException getIOException() {
return this.ioe;
}
}
}

Message cité 2 fois
Message édité par benou le 27-11-2005 à 12:57:29

---------------
ma vie, mon oeuvre - HomePlayer

the real moins moins

charly007 a écrit :

Je dois utiliser le jdk 1.3, donc pas de Iterable.

t'as pas obligé d'avoir l'interface pour en appliquer les principes. là, tu l'initialises quand ton iterateur ?
et puis, comment tu geres le fait que tu puisses avoir plusieurs processes qui l'appellent en meme temps ?

(perso, j'ai toujours cru c'était une mauvaise idée [:marc] )

Message cité 1 fois

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

the real moins moins

benou a écrit :

en fait ça sert plus à rien ce code, quoi
t'as ni plus ni moins de code en utilisant directement le BufferedReader, et je vois pas l'interet :??:

(et sinon, coté remarques à la con: ton emploi de this est inconsistant et pollue la lecture(là c'est un avis perso), ton exception est mal nommée, et elle cache une exception qui ne devrait probablement pas l'être; si je ne m'abuse tu fais ça juste pour pouvoir te caler sur l'interface Iterator ...)

(et je persiste à dire que cet objet pourrait etre un Iterable - s'il n'y avait pas d'énormes problemes potentiels d'acces concurrent - mais *pas* un Iterator)

Message cité 2 fois

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

charly007

benou a écrit :

ouais mais nan, ready ça sert pas à ça ...

Citation :

True if the next read() is guaranteed not to block for input, false otherwise

J'ai vu, mais j'ai aussi vu dans InputStreamReader, pour la méthode ready() :

Citation :

An InputStreamReader is ready if its input buffer is not empty, or if bytes are available to be read from the underlying byte stream.

De plus j'ai testé et je n'ai pas eu d'exception, alors j'ai fait confiance à masklinn.

benou a écrit :

et puis ta méthode iterator() elle sert plus à grand chose ...

Pour parcourir une liste, on utilise un Iterator. Je me suis dit que ça serait bien de faire pareil [:spamafote]

Message cité 1 fois
Message édité par charly007 le 27-11-2005 à 14:13:14

masklinn

í dag viðrar vel til loftárása

the real moins moins a écrit :

t'as pas obligé d'avoir l'interface pour en appliquer les principes.

Dans ce cas là, l'implémentation d'un itérateur n'a plus aucun intérêt en java (si on implémente pas l'interface je veux dire)

Citation :

t'as ni plus ni moins de code en utilisant directement le BufferedReader, et je vois pas l'interet :??:

Tu utilises des méthodes génériques qui bouffent de l'Iterator (ou de l'iterable, ils ont rajouté l'interface sur tous les types itérables genre ArrayList et autres?), de cette manière t'as des méthodes complètement génériques auquelle tu peux balancer des listes comme des fichiers, la nature réelle de la donnée n'a plus d'importance [:spamafote]

J0re en python quasiment tous les types de conteneurs, fichiers compris, sont itérables

Citation :

perso, j'ai toujours cru c'était une mauvaise idée

De pouvoir itérer sur un fichier [:petrus dei]

Citation :

(et sinon, coté remarques à la con: ton emploi de this est inconsistant et pollue la lecture(là c'est un avis perso), ton exception est mal nommée, et elle cache une exception qui ne devrait probablement pas l'être; si je ne m'abuse tu fais ça juste pour pouvoir te caler sur l'interface Iterator ...)

L'opération remove sur un iterator est optionnelle, donc elle doit bel et bien lancer une exteption quand elle est appelée et qu'elle ne fait rien (ou ne rien faire du tout, mais c'est risqué)

the real moins moins a écrit :

et je persiste à dire que cet objet pourrait etre un Iterable

J2SE5, et ça change pas grand chose, tu lui fais implémenter iterable et iterator (parce que créer un iterator special juste pour qu'il soit largué par le .iterator de l'Iterable c'est plus une perte de temps qu'autre chose)

Message cité 2 fois
Message édité par masklinn le 27-11-2005 à 14:14:20

Publicité

charly007

Je vais peut-être laisser tomber finalement :whistle:

the real moins moins

masklinn a écrit :

Dans ce cas là, l'implémentation d'un itérateur n'a plus aucun intérêt en java (si on implémente pas l'interface je veux dire)

Citation :

t'as ni plus ni moins de code en utilisant directement le BufferedReader, et je vois pas l'interet :??:

C'est bien ce que je dis, Iterable, pas Iterator.
(tu me prend pour un neuneu là ou bien?)
(et puis appliquer tes méthodes generiques qui bouffent de l'iterator - déjà, faudra m'en montrer - sur un putain de fichier, pas un conteneur, donc, c'est la porte ouverte à toutes les fenetres)

Message cité 1 fois
Message édité par the real moins moins le 27-11-2005 à 14:20:53

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

the real moins moins

masklinn a écrit :

J2SE5

mais t'arretes de me prendre pour un con ? [:mlc]

masklinn a écrit :

, et ça change pas grand chose, tu lui fais implémenter iterable et iterator (parce que créer un iterator special juste pour qu'il soit largué par le .iterator de l'Iterable c'est plus une perte de temps qu'autre chose)

si, ça change: au niveau sémantique, et par le fait qu'un crétin va pouvoir caster ton objet à la con en Iterator, et ça, ça serait pas joli-joli.

deuxio, non c'est pas une perte de temps: tu fais une inner classe, t'auras au pire 2 lignes en plus.

Message cité 1 fois
Message édité par the real moins moins le 27-11-2005 à 14:23:31

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

benou

the real moins moins a écrit :

en fait ça sert plus à rien ce code, quoi
t'as ni plus ni moins de code en utilisant directement le BufferedReader, et je vois pas l'interet :??:

à pouvoir itérer sur les lignes d'un fichier [:petrus75]

the real moins moins a écrit :

(et sinon, coté remarques à la con: ton emploi de this est inconsistant et pollue la lecture(là c'est un avis perso)

effectivemenet, je l'ai pas mis partout et c'est une erreur ... j'ai pas activé le warning qui va bien dans mon eclipse chez moi ...
Perso, j'aime bien l'uilisation de this. Je trouve au contraire que ça aide à la lecture. Surement une question d'habitude ...

the real moins moins a écrit :

, ton exception est mal nommée, et elle cache une exception qui ne devrait probablement pas l'être; si je ne m'abuse tu fais ça juste pour pouvoir te caler sur l'interface Iterator ...)

Elle ne la cache pas, elle la présente justement. Elle permet de la récupérer si tu préfères ...
Et elle est runtime de façon à passer dans l'interface Iterator ...

the real moins moins a écrit :

(et je persiste à dire que cet objet pourrait etre un Iterable - s'il n'y avait pas d'énormes problemes potentiels d'acces concurrent - mais *pas* un Iterator)

Faudra que tu m'expliques comment tu gères la lecture d'un flux en accès concurent
Et de quels problèmes tu parles ...

charly007 a écrit :

J'ai vu, mais j'ai aussi vu dans InputStreamReader, pour la méthode ready() :

Ok, mais dans ce cas tu te prends un InputStream en paramètre que tu enveloppe toi même dans un InputStreamReader ... parce que rien ne te dis que le Reader que tu te prends en paramètre du constructeur est un InputStreamReader

charly007 a écrit :

Pour parcourir une liste, on utilise un Iterator. Je me suis dit que ça serait bien de faire pareil [:spamafote]

Ben oui, c'est l'objet qui est un Iterator. Ca sert à quoi de faire une méthode qui retourne this ?
A la rigueur, si tu voulais implémentaer Iterable, mais ce serait plutôt sur un autre objet, pas sur celui là ...

the real moins moins a écrit :

C'est bien ce que je dis, Iterable, pas Iterator.

A quel objet tu veux faire implémenter Iterable ?

the real moins moins a écrit :

si, ça change: au niveau sémantique, et par le fait qu'un crétin va pouvoir caster ton objet à la con en Iterator, et ça, ça serait pas joli-joli.

j'ai pas compris [:petrus75]

Message cité 2 fois

---------------
ma vie, mon oeuvre - HomePlayer

the real moins moins

benou a écrit :

Perso, j'aime bien l'uilisation de this. Je trouve au contraire que ça aide à la lecture. Surement une question d'habitude ...

beurk.

benou a écrit :

Faudra que tu m'expliques comment tu gères la lecture d'un flux en accès concurent

je le gère justement pas, or, cet objet pourrait laisser croire qu'il le fait.

benou a écrit :

A quel objet tu veux faire implémenter Iterable ?

à celui dont tu a posté le code, pardi.

benou a écrit :

j'ai pas compris [:petrus75]

...

Message cité 1 fois

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

masklinn

í dag viðrar vel til loftárása

benou a écrit :

j'ai pas compris [:petrus75]

Il considère que la classe est présentée au publique dans un module que les gens pourront utiliser et que (!!!) ils auront la possibilité de lire le fichier en castant directement le reader en Iterator [:totoz]
(c'est à dire faire ce que je fais déjà)
(parce que là il parle de mon bouzin à moi, pas du tien)

the real moins moins

(si c'est du code "privé", je vois pas, à part pour le plaisir, l'interet d'écrire 25 lignes de code qui ne sert à rien , pour boucler sur un Iterator au lieu d'un bete
while ((line=reader.readLine())!=null) {
}
...)

Message cité 1 fois

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

benou

the real moins moins a écrit :

à celui dont tu a posté le code, pardi.

Ca c'est l'iterateur, pas le truc iterable à partir du quel tu récupères l'iterateur ...

the real moins moins a écrit :

...

[:etoile_filante]

Message cité 1 fois

---------------
ma vie, mon oeuvre - HomePlayer

the real moins moins

benou a écrit :

Ca c'est l'iterateur, pas le truc iterable à partir du quel tu récupères l'iterateur ...

ha oui, juste.

enfin, je continue à trouver ça inutile

Message cité 1 fois

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

benou

the real moins moins a écrit :

enfin, je continue à trouver ça inutile

Bha, dans certains cas peut être ...

Message cité 1 fois

---------------
ma vie, mon oeuvre - HomePlayer

masklinn

í dag viðrar vel til loftárása

the real moins moins a écrit :

J'ai déjà parlé de pouvoir envoyer n'importe quel conteneur itérable à des méthodes, on peut ajouter une syntaxe beaucoup plus simple en Java5 grâce au combo foreach + Iterable

Message cité 1 fois

the real moins moins

benou a écrit :

Bha, dans certains cas peut être ...

masklinn a écrit :

J'ai déjà parlé de pouvoir envoyer n'importe quel conteneur itérable à des méthodes, on peut ajouter une syntaxe beaucoup plus simple en Java5 grâce au combo foreach + Iterable

ya ya, give me concrete use cases, et on en reparle, hein

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

masklinn

í dag viðrar vel til loftárása

Tu traites des logs ligne par ligne, tu permets d'entrer soit le chemin d'un fichier de log soit de coller dans un champ texte le bout de log à analyser.

Tu fais une fonction de traitement qui prend un itérateur sur des lignes de log et baste [:spamafote]

Message édité par masklinn le 27-11-2005 à 16:17:32

the real moins moins

euh, non, je fais un traitement sur un Reader [:marc]

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

benou

BufferedReader [:aloy]

Message cité 1 fois

---------------
ma vie, mon oeuvre - HomePlayer

nraynaud

lol

ton troll est débile, mais j'vais quand même te dire une truc mon p'tit gars, depuis que j'ai eu des newlines dans certains champs d'un fichier en CSV, je relativise vachement la lecture ligne par ligne.

Message cité 1 fois

the real moins moins

benou a écrit :

BufferedReader [:aloy]

ben non

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

nraynaud

lol

moi je mets +1 sur --, si le stream est déjà bufferisé, pas la peine de bufferiser le reader qui est dessus.

Message cité 1 fois

---------------
trainoo.com, c'est fini

the real moins moins

(d'autant que dans l'exemple à la noix de masklinn -(give me real uses cases, morons)-, on utiliserait un BufferedReader(FileReader) ou un StringReader qu'il n'y aurait pas besoin de bufferiser

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

the real moins moins

(ha oué, spour avoir readLine() qu'il veulent faire ça en fait )

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

Jubijub

Parce que je le VD bien

nraynaud a écrit :

un truc pareil ca mérite le pal direct....le fouet à pointes à minima...

---------------
Jubi Photos : Flickr - 500px

benou

nraynaud a écrit :

moi je mets +1 sur --, si le stream est déjà bufferisé, pas la peine de bufferiser le reader qui est dessus.

spourça que mon constructeur prend un BufferedReader et pas un Reader sinon t'es obligé de le (ré)encapsuler par un bufferedreader pour pourvoir faire le readline.

donc c'est plus un -1 que tu mets sur -- ...

---------------
ma vie, mon oeuvre - HomePlayer

Publicité

FORUM HardWare.fr

Programmation

Java

BufferedReader qui implémente Iterator

Sujets relatifs
BufferedReader	BufferedReader en attente pendant lecture socket[Resolu]
Boucle for et list::iterator	STL - multimap - ou le mystère de l'iterator perdu (non résolu)
Iterator	template iterator héritage, OK pour visual, Erreurs avec Gcc
HTTP server - PrintWriter BufferedReader	[JAVA] Limite du BufferedReader ?
<iterator> : conception d'iterator, introduction aux traits	relire un fichier FileReader BufferedReader
Plus de sujets relatifs à : BufferedReader qui implémente Iterator

Page générée en 0.096 secondes