Remplissage et parcours de liste d'objets

Version imprimable

Bonjour à tous,

Je débute en python et j'ai une question concernant le stockage de différentes instances de classes.
J'ai écrit la fonction suivante qui remplit des attributs d'une instance de la classe Piezometer (peu importe ce que cela représente), sur la base de la lecture de lignes d'un fichier d'un fichier texte (je remplis les attributs d'une instance par ligne lue).

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
def ReadAllPiezometerAttributes(FilePath):
 
    ListPiez =  []
    cpt = 0
 
    with open(FilePath,'r') as fileAtt:
        data = fileAtt.readline()
 
        while data:
            info = data.split()
            ListPiez.append(Piezometer(info[3],info[0],info[8],info[7],info[6],info[1],info[10],info[11],info[9])) 
            print(ListPiez[cpt])
            data = fileAtt.readline()
            cpt+=1
 
        print("{} piezos have been stored.".format(len(ListPiez)))
 
    for i in range(len(ListPiez)):
        print(ListPiez[i])
 
    return ListPiez

Définition de la classe associée :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
class Piezometer:
    def __init__(self, nom, ids, id_mabs, id_layer, id_mintern, bss, x, y, layername):
        self._name = nom                
        self._id = ids                       
        self._id_mabs = id_mabs        
        self._id_layer = id_layer      
        self._id_mintern = id_mintern   
        self._bss = bss                 
        self._x = x              
        self._y = y                  
        self._layername = layername   
        self._obs_timeserie = {}        
        self._sim_timeserie = {}        
 
    @classmethod
    def __str__(self):
        return "Piezo id : {}, Name : {} ({}) - Model location : ABS {} - ({},{})".format(self._id, self._name,self._bss,self._id_mabs,self._id_layer,self._id_mintern)

Mon problème est résumé par l'output obtenu, qui est le suivant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
Piezo id : 1, Name : Gumery (02606X1013-S1) - Model location : ABS 6044 - (0,6043)
Piezo id : 2, Name : Noyen-Sur-Seine (02606X0125-PM3) - Model location : ABS 2737 - (0,2736)
Piezo id : 3, Name : Noyen-Sur-Seine (02606X0112-G5) - Model location : ABS 2541 - (0,2540)
Piezo id : 4, Name : Mouy-Sur-Seine (02605X0062-M4) - Model location : ABS 160 - (0,159)
Piezo id : 5, Name : Rheges (02621X0009-S1) - Model location : ABS 298 - (0,297)
Piezo id : 6, Name : Rheges (02621X0085-PZ) - Model location : ABS 298 - (0,297)
Piezo id : 7, Name : Nogent-Sur-Aube (02628X0005-P1) - Model location : ABS 241 - (0,240)
Piezo id : 8, Name : Chalons-En-Champagne (01891X0047-PZ1) - Model location : ABS 4746 - (0,4745)
Piezo id : 9, Name : Asfeld (01082X0015-P-SUD) - Model location : ABS 2923 - (0,2922)
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)
10 piezos have been stored.
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)
Piezo id : 10, Name : Bucy-Le-Loing (01067X0362-BL7) - Model location : ABS 1806 - (0,1805)

J'ai bien du mal à comprendre pourquoi le second parcours de la liste, en fin de fonction, (et a fortiori, la liste retournée par la fonction) ne contient que la dernière instance dupliquée partout ?!?! (alors que la liste semble correctement constituée (cf. le print en cours de boucle).

J'espère que l'exemple est assez clair.
Merci beaucoup par avance pour votre aide,

oblivion0-dev

Salut,

Lorsque vous postez du code, il faudrait qu'il fonctionne.
Ici:
Code:

1 2 3 4 5 class Piezometer: ... @classmethod def __str__(self): return "Piezo id : {}, Name : {} ({}) - Model location : ABS {} - ({},{})".format(self._id, self._name,self._bss,self._id_mabs,self._id_layer,self._id_mintern)
va planter à cause du classmethod qui induit l'accès a des attributs de classe inexistants.

Et qu'il soit suffisant à reproduire le problème (pas des petits bouts à assembler comme le vendeur de meubles célèbre).

Si on n'arrive pas, comme vous, à trouver l'erreur juste en relisant le code...

- W

24/11/2021, 07h50
VinsS

Salut,

C'est ton décorateur qui te joue ce tour.

Une classmethod reçoit la classe en premier argument et non l'instance.
Tu aurais d'ailleurs dû écrire __print__(cls), ça a l'air d'un détail mais un détail qui alerte sur le type de l'argument.

Personnellement, j'utilise def __repr__(self) pour ça, sans décorateur.
24/11/2021, 08h31
wiztricks

Citation:

Envoyé par VinsS

C'est ton décorateur qui te joue ce tour.

Pas que, sinon ça n'afficherait pas correctement la liste la première fois.
Ça dit juste que le code posté n'est pas cohérent avec l'histoire (si ça plante a tous les coups, ça n'affiche rien!).

Et que si l'histoire est "une liste avec initialisée avec 5 objets différents devient 5 objets identiques" sans montrer où cette liste est modifiée... demande plus de code.

- W

Bonjour

Citation:

Envoyé par oblivion0-dev

J'ai bien du mal à comprendre pourquoi le second parcours de la liste, en fin de fonction, (et a fortiori, la liste retournée par la fonction) ne contient que la dernière instance dupliquée partout ?!?! (alors que la liste semble correctement constituée (cf. le print en cours de boucle).

Désolé mais ton code n'est pas fonctionnel. Comme déjà dit par wiztricks, la méthode __str__() (surcharge de str()) n'est pas une méthode de classe et la déclarer ainsi prooque une exception "AttributeError" puisque ne récupérant pas une instance, la méthode ne trouve pas d'attribut "_name".

Et en tentant de reproduire ton problème en corrigeant l'erreur, ton problème tel que décrit ne se produit pas
Code:

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 #!/usr/bin/env python3 # coding: utf-8 class Piezometer: def __init__(self, nom): self._name = nom def __str__(self): return "Piezo Name : {}".format(self._name) # class Piezometer def ReadAllPiezometerAttributes(): ListPiez=[] for i in range(5): ListPiez.append(Piezometer("i=%d" % i)) print("{} piezos have been stored.".format(len(ListPiez))) # for for i in range(len(ListPiez)): print(ListPiez[i], hex(id(ListPiez[i]))) return ListPiez # ReadAllPiezometerAttributes() piezo=ReadAllPiezometerAttributes() print("ok") for p in piezo: print(p, hex(id(p)))
Résultat
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 1 piezos have been stored. 2 piezos have been stored. 3 piezos have been stored. 4 piezos have been stored. 5 piezos have been stored. Piezo Name : i=0 0x7f398cd11ac8 Piezo Name : i=1 0x7f398cc99278 Piezo Name : i=2 0x7f398cc99390 Piezo Name : i=3 0x7f398cc99400 Piezo Name : i=4 0x7f398cc99470 ok Piezo Name : i=0 0x7f398cd11ac8 Piezo Name : i=1 0x7f398cc99278 Piezo Name : i=2 0x7f398cc99390 Piezo Name : i=3 0x7f398cc99400 Piezo Name : i=4 0x7f398cc99470
Les identifiants étant bien tous différents, j'ai bien 5 objets distincts dans ma liste résultant de la fonction

Citation:

Envoyé par VinsS

Personnellement, j'utilise def __repr__(self) pour ça, sans décorateur.

Cette méthode, bien que ressemblante fortement à __str__ (d'autant plus qu'elle sera utilisée en subsitution à cette dernière si celle-ci n'est pas présente quand elle est invoquée), n'a quand-même pas le même but...

24/11/2021, 10h06
wiztricks

Pour reproduire votre problème, on ajoute @classmethod à __init__.
Ce qui va faire "fonctionner" le classmethod du __str__.

Dans ce cas, on modifiera des attributs de classe et seules les dernières valeurs entrées seront prises en compte.

- W
24/11/2021, 11h26
Sve@r

Citation:

Envoyé par wiztricks

Pour reproduire votre problème, on ajoute @classmethod à __init__.
Ce qui va faire "fonctionner" le classmethod du __str__.

Dans ce cas, on modifiera des attributs de classe et seules les dernières valeurs entrées seront prises en compte.

Ah oui, c'est bien vu :applo:

On peut remarquer en plus (en appliquant ton idée à mon exemple) qu'effectivement on a 5 fois le même nom mais on a quand-même 5 objets distincts (id tous différents). Donc 5 objets distincts avec tous le même attribut de classe...
24/11/2021, 12h26
wiztricks

Citation:

Envoyé par Sve@r

Ah oui, c'est bien vu :applo:

Pas facile d'imaginer la créativité des débutants pour fabriquer des problèmes!

- W