Discussion :

[josmiley]

bonjour,
j'ai une class dont __init__ modifie une liste

Code :

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

1
2
3
4
5
6
TimerList = []
class TimeEvent :
 
	def __init__(self,TimerList=TimerList) :
		TimeList = TimerList[0:] + [self]
               #TimerList.append(self)

si on utilide la methode .append(), la liste est correcte dans la fonction et en dehors de la fonction, donc pàas de probleme.
si on utilise la methode du slicing(ligne non commentée), la liste est correcte dans la fonction mais erronnée en dehors.
Pourquoi celà, et quelle methode utiliser pour insérer un élément au milieu d'une liste ?
merci.

[Thierry Chappuis]

Lorsque tu écris

Code :

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

TimeList = TimerList[0:] + [self]

tu ne modifies pas ta variable globale de type liste TimerList, car:

1. tu réaffectes ta liste modifiée à une variable locale TimeList (!= TimerList, c'était voulu ?)
2. l'argument TimerList de la méthode __init__ masque la variable globale du même nom. Tu ne peux donc pas y avoir accès depuis le corps de cette méthode.

Thierry

[josmiley]

1. tu réaffectes ta liste modifiée à une variable locale TimeList (!= TimerList, c'était voulu ?)

non, cest une erreur de recopie

2. l'argument TimerList de la méthode __init__ masque la variable globale du même nom. Tu ne peux donc pas y avoir accès depuis le corps de cette méthode.

ok, mais pourquoi ça marche avec la methode append() alors ?

[Thierry Chappuis]

ok, mais pourquoi ça marche avec la methode append() alors ?

Parce qu'avec append(), tu modifies la liste pointée par l'argument TimerListe en place, alors qu'avec la deuxième, tu affectes une nouvelle liste à cette variable.

Les choses te paraîtront certainement moins obscures si tu nommes ton paramètre avec un identificateur différent de TimerList:

Code :

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

1
2
3
4
5
6
7
TimerList = []
class TimeEvent :
 
	def __init__(self, tlist=TimerList) :
                global TimerList
		TimerList = tliste[0:] + [self]
               #TimerList.append(self)

Thierry

[josmiley]

ha okééé ...
mais du coup, l'utilation de 'global' evite le passage de l'argument TimerList

Code :

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

def __init__(self, tlist=TimerList)

devient

Code :

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

1
2
def __init__(self)
     global TimerList

c'est plus clair je trouve, merci Thierry.

[Thierry Chappuis]

ha okééé ...
mais du coup, l'utilation de 'global' evite le passage de l'argument TimerList

Code :

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

def __init__(self, tlist=TimerList)

devient

Code :

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

1
2
def __init__(self)
     global TimerList

c'est plus clair je trouve, merci Thierry.

Par contre, l'usage du mot clé global n'est pas tellement conseillé ici. Perso, je ferais:

Code :

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

1
2
3
4
5
6
 
TimerList = []
class TimeEvent :
 
	def __init__(self, timers=TimerList) :
               timers.append(self)

ou

Code :

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

1
2
3
4
5
 
class TimeEvent :
	timers = []
	def __init__(self) :
               TimeEvent.timers.append(self)

Thierry

[josmiley]

Par contre, l'usage du mot clé global n'est pas tellement conseillé ici.

je lis souvent que global c'est mal. Mais il doit bien avoir une utilité, quel est le danger à utiliser global ?