Discussion :

[zequiche]

Bonjour,

Je voudrais surcharger la classe file (pour dans un cas précis de fichier, filtrer certaines lignes, en modifier d'autres)

J'ai un problème de récursion infinie dans la méthode __iter__ que j'ai définie dans ma classe dérivée.
Comme si l'appel à super(classedérivéedefile,self).__iter__() appelait en fait classederiveedefile.__iter__ au lieu de file.__iter__

J'imagine que file a bien une methode __iter__, puisqu'on peut itérer dessus ?

Voici le 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
#!/usr/bin/python
mport os
import sys
 
class MachinFile(file):
  def __init__(self, name, errfile, *args, **kwargs):
    self.errfile=errfile
    super(MachinFile,self).__init__(name, *args,**kwargs)
 
  def __iter__(self):
    for l in super(MachinFile,self).__iter__(): # est ce que ce n est pas le __iter__ de la classe derivee qui est appelée ?
      yield l+self.errfile
 
with MachinFile("ess.py","search") as f:
  for l in f:
    print(l)

ET voici la sortie :

...
for l in s:
File "./ess.py", line 16, in __iter__
for l in s:
File "./ess.py", line 16, in __iter__
for l in s:
RuntimeError: maximum recursion depth exceeded while calling a Python object

Merci de votre aide

[fred1599]

Pouvez-vous expliquez pourquoi l'erreur

Code :

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

for l in s:

alors que vous faîtes

Code :

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

for l in f:

?

À part que le test que vous faîtes ne soit pas en corrélation avec le code présenté, je ne vois pas ! Merci d'apporter une cohérence...

[VinsS]

Salut,

Où est définie la classe file ?

[wiztricks]

Salut,

J'ai un problème de récursion infinie dans la méthode __iter__ que j'ai définie dans ma classe dérivée.

Vous avez 2 protocoles pour réaliser un itérateur avec __iter__: celui qui fait des yield et celui qui retourne self avec une méthode next retournant chaque élément.
De fait, si vous surchargez __iter__, il faut que le protocole reste cohérent avec celui de la classe parente.

- W

[zequiche]

Merci de ces réponses.
Je dois donc surcharger next() ?
Je vais essayer et je vous tiens au courant.

[zequiche]

Ca marche !!

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
 
#!/usr/bin/python
import os
import sys
 
class MachinFile(file):
  def __init__(self, name, errfile, *args, **kwargs):
    self.errfile=errfile
    super(MachinFile,self).__init__(name, *args,**kwargs)
 
  def next(self):
    l=super(MachinFile,self).next()
    return l+"coucou"
 
with MachinFile("fichier_a_lire","search") as f:
  for l in f:
    print(l)

[fred1599]

Oui tu peux aussi surcharger la méthode __iter__ :

Code :

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

1
2
3
4
5
6
7
def __iter__(self):
    while True:
        try:
            line = super(MachinFile, self).next()
            yield line.rstrip() + self.errfile
        except StopIteration:
            break

[wiztricks]

Oui tu peux aussi surcharger la méthode __iter__ :

Code :

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

1
2
3
4
5
6
7
def __iter__(self):
    while True:
        try:
            line = super(MachinFile, self).next()
            yield line.rstrip() + self.errfile
        except StopIteration:
            break

Certes mais si on fait comme çà, on casse le comportement de "next" i.e. une suite d'appels à next ne retournera pas la même chose que "for .... in ...".

- W

[fred1599]

Certes mais si on fait comme çà, on casse le comportement de "next" i.e. une suite d'appels à next ne retournera pas la même chose que "for .... in ...".

Je ne pense pas casser le comportement de next méthode de la classe file, pour en changer le comportement, il me semble que l'on devrait là surcharger, ce qui n'est pas le cas ici.

Si je fais

Code :

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

1
2
3
4
5
6
7
with MachinFile("fichier_a_lire", "search") as f:
    while True:
        try:
            l = next(f).rstrip()
            print l
        except StopIteration:
            break

Normalement je devrais retourner mes lignes de fichier avec la méthode next de base de l'objet file, non ? Ou y a-t-il un bug dans ma compréhension ?

[wiztricks]

Je ne pense pas casser le comportement de next méthode de la classe file, pour en changer le comportement, il me semble que l'on devrait là surcharger, ce qui n'est pas le cas ici.

Ben for...in... retournera des lignes de la forme XXXX + ZZZ alors que next ne retournera que XXXX comme si MachinFile n'était plus au milieu pour y ajouter ZZZ.

- W

[fred1599]

Oui effectivement ça se tient, en y réfléchissant c'est logique que les deux fassent un résultat similaire.

[wiztricks]

Oui effectivement ça se tient, en y réfléchissant c'est logique que les deux fassent un résultat similaire.

D'un autre côté, si on dit que la classe MachinChose est un objet de type file qui ajoute un suffixe à toute ligne lue, il y a encore du boulot (d'autant qu'on n'est pas tenu de lire un fichier ligne par ligne)...
De fait, écrire une fonction genre:

Code :

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

1
2
3
def add_suffix(f, tag):
    for line in f:
        yield line.rstrip() + tag

suffirait largement...
Je comprends que lorsqu'on découvre les class on ait envie de les utiliser à tout et n'importe quoi pour découvrir les limites de ce qu'on peut faire avec et des contraintes qu'elles imposent.

- W

[zequiche]

Oui,
mais c'était pour l'exemple.En pratique je fais un peu plus de traitement que ça.
Je ne voulais pas ajouter dans mon programme principal de traitement sur les lignes du fichier, je préfère pouvoir itérer directement sur les lignes qui m'intéressent, et déléguer le tri des lignes à la méthode __iter__.
Je trouve qu'on comprend mieux le but du programme en le lisant (la sélection des lignes a traiter n etant pas le coeur du programme). Mais je comprend bien qu'on aurait pu faire plus simple.