Python - Pourquoi est-ce une variable de classe ne sont pas définis dans la méthode?

J'ai une application en python qui est comme suit:

global_counter = 0
connections = {}

class SocketHandler():
    currentid = 0
    def open(self):
        global global_counter
        global connections
        currentid = global_counter
        global_counter += 1
        connections[currentid] = self
        print "WebSocket " + str(currentid) + " opened"

    def on_close(self):
        global connections
        print "WebSocket " + str(currentid) + " closed"
        del connections[currentid]

J'obtiens le message d'erreur:

NameError: global name 'currentid' is not defined

sur les lignes de "ouvert" et "on_close" où j'ai l'impression que je suis de l'ouverture/fermeture de la connexion. J'ai défini dans la classe, pourquoi n'est-il pas dans le champ d'application. Aussi, j'ai lu que l'utilisation de variables globales est mauvais, mais je ne vois pas un moyen de contourner cela. Quelqu'un peut-il préciser ce que je dois faire? Merci.

Utilisation de l'auto.currentid

OriginalL'auteur Lylax | 2012-10-08

9

Vous n'avez pas accès implicite à des attributs à l'intérieur de méthodes, en Python.

Nue un nom comme currentid dans la ligne:
```
del connections[currentid]
```
toujours à la recherche d'un nom dans les locaux de la portée de la fonction, puis dans chaque fonction englobante portée, avant d'essayer le module global de la portée (et puis regarde built-ins en dernier recours). currentid est un attribut de classe, qui ne se trouve dans aucun de ces étendues.

Pour rechercher un attribut en Python vous toujours besoin de spécifier un objet dans lequel chercher. Bien que le protocole de recherche signifie que l'objet ne doivent pas nécessairement avoir l'attribut lui-même; l'attribut de recherche revient à la classe de l'objet que vous avez spécifié (et les classes de base, si l'héritage est en jeu).

Donc, ce serait du travail:
```
del connections[self.currentid]
```
Cependant, je ne pense pas que le reste de votre code fait ce que vous pensez qu'il est. Cette ligne dans le open méthode:
```
currentid = global_counter
```
ne définit pas le currentid attribut de votre SocketHandler objet. L'affectation à un nu nom toujours assigne à une variable locale, sauf si vous explicitement déclarer global (vous semblent être conscients de cela, puisque vous avez utilisé le global mot-clé). Ainsi, dans le open méthode, currentid est une fonction de la variable, sa valeur est perdu à la fin de la open méthode.

En fait, votre SocketHandler objets n'ont pas de currentid attribut à tous (sauf si il y a plus de code que vous n'avez pas montré). Mettre currentid = 0 dans la classe du bloc n'est pas donner à tous le SocketHandler cas currentid attribut. Il donne la SocketHandler classe lui-même un attribut currentid; c'est juste que le def open(self): bloc crée une open attribut (stockage d'une fonction) sur la classe de l'objet, et non pas sur chaque instance.

Lecture self.currentid dans le on_close méthode ne parviennent pas à trouver un currentid attribut dans l'objet self, de sorte que Python va regarder la classe de self qui est SocketHandler. Cet objet ne ont un currentid valeur, de sorte que le résultat de la lecture self.currentid sera 0, si oui ou non vous avez déjà exécuté open sur que SocketHandler.

Si vous destiné à stocker les currentid comme une variable d'instance dans chaque SocketHandler, puis la ligne dans open devra être:
```
self.currentid = global_counter
```
Il attribue à la currentid attribut de l'objet visé par self. Vous aussi, vous avez besoin de changer toutes les autres références à currentid dans vos méthodes de self.currentid.

- Ce vraiment clarifié les choses, merci pour l'écriture.

OriginalL'auteur Ben
4

currentid devrait être self.currentid car il est une variable de classe.

Qui fait sens, merci!
Pourquoi est-ce que beaucoup upvoted? -- le code dans le exampe ci-dessus n'aura plus de rupture avec NameError, mais de rompre avec une grave erreur de logique, si seulement ce remplacement est effectué.
Que voulez-vous dire? Les seules autres variables qui sont modifiées globals.
Quelle est la probabilité pensez-vous, c'est que l'OP doit avoir une variable locale avec le même nom que l'attribut de classe à l'intérieur de la open méthode? Et pour avoir le on_close méthode toujours del connections[0]?
Désolé, j'ai lu jsbueno commentaire en disant qu'il y aura une erreur d'analyse dans le script, pas une erreur d'exécution.

OriginalL'auteur ronak

currentid est l'instance de l'attribut, afin de l'utiliser self.currentid au lieu de currentid:

def on_close(self):
        global connections
        print "WebSocket " + str(self.currentid) + " closed"
        del connections[self.currentid]

OriginalL'auteur defuz

Vous devez vous connecter pour publier un commentaire.