Discussion :

[Azzuri]

Salut a tout le monde..
je voudrais savoir s'il est possible de reecrir le code suivant sans la methode "setdefault".. je debute en programmation...

merci d'avance

Code :

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

1
2
3
4
5
6
7
8
9
class classifier:
  def __init__(self):
    self.wx={ }
 
 
 def aug (self,item, kateg):
    self.wx.setdefault(item,{})
    self.wx[item].setdefault(kateg,0)
    self.wx[item][kateg]+=1

[josmiley]

Code :

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

1
2
3
4
5
class classifier(dict):
	''
	def aug(self,item,kateg):
		if not item in self: self[item]={kateg:0}
		else: self[item][kateg]+=1

[Azzuri]

Salut,
Si j ai bien compris on a plus besoin d initialiser le 'dict'?
merci

[josmiley]

tu veux dire: 'pourquoi j'ai viré le __init__ ?'
puisque le seul attribut est un dict, autant dériver d'un dict.

[Azzuri]

ok, et si je veux initialiser un autre variable en plus du 'dict'

[eyquem]

Bien vu josmiley pour l’héritage de dict, je n’y avais même pas pensé.



Par contre, j’ai bien l’impression qu’il faut dans tous les cas d’héritage, même d’un type de base comme dict, exécuter la fonction __init__ de la classe parente dans la définition de la classe fille.


De plus , comparer:

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
23
24
25
26
27
28
29
class classifierAzzuri:
    def __init__(self):
        self.wx={ }
 
    def aug (self,item, kateg):
        self.wx.setdefault(item,{})
        self.wx[item].setdefault(kateg,0)
        self.wx[item][kateg]+=1
 
print 'Azzuri'
dAzz = classifierAzzuri()
dAzz.aug('a',980)
print dAzz.wx
dAzz.aug('a',500)
print dAzz.wx
 
print
class classifierjo(dict):
    ''
    def aug(self,item,kateg):
        if not item in self: self[item]={kateg:0}
        else: self[item][kateg]+=1
 
print 'josmiley' 
djo = classifierjo()
djo.aug('A',1300)
print djo
djo.aug('A',700)
print djo

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
Azzuri
{'a': {980: 1}}
{'a': {500: 1, 980: 1}}
 
josmiley
{'A': {1300: 0}}
 
Traceback (most recent call last):
  File "C:\Python27\dicow.py", line 40, in <module>
    djo.aug('A',700)
  File "C:\Python27\dicow.py", line 34, in aug
    else: self[item][kateg]+=1
KeyError: 700

Voilà pourquoi je propose:

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
class classifier1(dict):
    def __init__(self):
        dict.__init__(self)
    def aug(self,item,kateg):
        if item not in self:
            self[item]={kateg:1}
        elif kateg not in self[item]:
            self[item][kateg] = 1
        else:
            self[item][kateg]+=1
 
dw = classifier1()
dw.aug('a',900)
dw.aug('b',1900)
print dw
dw.aug('a','sup')
print dw
dw.aug('a','sup')
print dw
print 
 
 
class classifier2(dict):
    def __init__(self):
        dict.__init__(self)
    def aug(self,item,kateg):
        try:
            self[item][kateg]+=1
        except KeyError:
            try:
                self[item][kateg] = 1
            except KeyError:
                self[item]={kateg:1}
 
zw = classifier2()
zw.aug('A',443)
zw.aug('B',887)
print zw
zw.aug('A','gol')
print zw
zw.aug('A','gol')
print zw

Code :

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

1
2
3
4
5
6
7
{'a': {900: 1}, 'b': {1900: 1}}
{'a': {900: 1, 'sup': 1}, 'b': {1900: 1}}
{'a': {900: 1, 'sup': 2}, 'b': {1900: 1}}
 
{'A': {443: 1}, 'B': {887: 1}}
{'A': {'gol': 1, 443: 1}, 'B': {887: 1}}
{'A': {'gol': 2, 443: 1}, 'B': {887: 1}}

Cf les considérations attachées au premier exemple dans la page
http://www.python.org/download/relea....3/descrintro/

ok, et si je veux initialiser un autre variable en plus du 'dict'

Prière d’être plus compréhensible

[shadowsam]

Hmm juste pour reprendre la réponse initiale. Il a une erreur dans l'exemple.

Code :

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

1
2
3
4
5
class classifier(dict):
	''
	def aug(self,item,kateg):
		if not item in self: self[item]={kateg:0}
		else: self[item][kateg]+=1

Si l'on fait :

Code :

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

1
2
3
c = classifier()
c.aug("1", "3")
c.aug("1", "4")

On va avoir une exception parce que "4" n'est pas un enregistrement de self[item].

Personnellement, j'écrirais surement qq chose comme:

Code :

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

1
2
3
4
5
class classifier:
    def __init__():
        self.wx = {}
    def aug(self, item, kart):
        self.wx[item][kart] = self.wx.setdefault(item, {}).get(kart, 0)+1