Discussion :

[oblivion0-dev]

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 :

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
 
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 :

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
 
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 :

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
 
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

[wiztricks]

Salut,

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

Code :

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

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

[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.

[wiztricks]

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

[Sve@r]

Bonjour

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 python :

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
#!/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 :

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
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

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...

[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

[Sve@r]

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

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...

[wiztricks]

Ah oui, c'est bien vu

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

- W