Python - Warum ist diese Klasse variable nicht definiert in der Methode?

Habe ich ein python-Anwendung, die ist wie folgt:

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]

Ich bin immer der Fehler:

NameError: global name 'currentid' is not defined

auf den Linien von "öffnen" und "on_close", wo ich den Druck, den ich öffnen/schließen der Verbindung. Ich definierte es in der Klasse, warum ist es nicht in den Anwendungsbereich. Auch habe ich gelesen, dass die Verwendung von globalen Variablen ist schlecht, aber ich sehe nicht, einen Weg, um dieses. Kann mir jemand erklären, was ich tun soll? Danke.

Verwenden Sie selbst.currentid

InformationsquelleAutor Lylax | 2012-10-08

9

Du nicht haben, die impliziten Zugriff auf Attribute innerhalb Methoden in Python.

Einen bloßen Namen wie currentid in der Zeile:
```
del connections[currentid]
```
immer sucht Sie einen Namen in der lokalen Funktions-Bereich, dann in jedem umschließenden Funktions-scope, bevor Sie versuchen, die globalen Modul-Umfang (und dann sieht built-ins als letzten Ausweg). currentid ist ein class-Attribut, die nicht gefunden werden in jedem der genannten Bereiche.

Nachschlagen ein Attribut in Python geben, die Sie immer muss ein Objekt angeben, in dem zu sehen. Wenn das lookup-Protokoll bedeutet, das Objekt muss nicht unbedingt das Attribut selbst; Attribut-lookup wird wieder die Klasse des Objekts, das Sie angegeben haben (und die base-Klassen, wenn die Vererbung beteiligt ist).

Damit das funktionieren würde:
```
del connections[self.currentid]
```
Aber ich glaube nicht, dass der rest Ihres Codes zu tun, was Sie denken, es ist entweder. Diese Zeile in der open Methode:
```
currentid = global_counter
```
nicht die currentid Attribut des SocketHandler Objekt. Die Zuordnung zu einem bloßen Namen immer weist auf eine lokale variable, es sei denn, Sie explizit zu deklarieren, es global (Sie scheinen sich dessen bewusst zu sein, da hast du verwendet, die global Stichwort). So in der open Methode currentid ist eine lokale variable, Funktion; der Wert ist, verlor am Ende die open Methode.

In der Tat, Ihre SocketHandler - Objekte haben keine currentid Attribut überhaupt (es sei denn, es gibt noch mehr code, den Sie haben nicht gezeigt). Setzen currentid = 0 im class-block nicht geben, die SocketHandler Instanzen eine currentid Attribut. Es gibt die SocketHandler Klasse sich ein Attribut currentid; dieser ist ebenso wie der def open(self): block erstellt eine open Attribut (Speicherung einer Funktion) auf das class-Objekt, nicht auf jeder einzelnen Instanz.

Lesen self.currentid im on_close Methode nicht zu finden, ein currentid Attribut im Objekt self, so Python wird der Blick auf die Klasse self die SocketHandler. Das Objekt hat haben eine currentid Wert, so das Ergebnis des Lesens self.currentid wird 0, ob oder nicht, Sie haben zuvor ausgeführt open auf, dass SocketHandler.

Wenn Sie bedeutete, zu speichern, die currentid als Instanz-variable in jedem SocketHandler, dann die Zeile in open werden müssten:
```
self.currentid = global_counter
```
Dies weist auf die currentid Attribut des Objekts bezeichnet self. Sie würde auch dann müssen alle Verweise auf currentid in Ihre Methoden, um self.currentid.

Das wirklich Dinge geklärt, vielen Dank für das schreiben.

InformationsquelleAutor Ben
4

currentid sollte self.currentid da es sich um eine Klassenvariable.

Das macht Sinn, danke!
Warum ist dies, dass viel von Ihnen positiv bewertet werden? - den code in die exampe oben brechen nun nicht mehr mit Fehler, aber brechen mit einer schweren logischen Fehler, wenn nur diese Ersetzung wird gemacht.
Was meinst du? Die einzigen anderen Variablen, die geändert wird, sind die globals.
Für wie wahrscheinlich hältst du es, dass die OP soll eine lokale variable mit dem gleichen Namen wie die Klasse, Attribut in der open Methode? Und die on_close Methode immer del connections[0]?
Sorry, ich lese jsbueno Kommentar zu sagen, dass es ein parsing-Fehler im Skript, nicht zu einem Laufzeitfehler.

InformationsquelleAutor ronak

currentid ist das Instanz-Attribut verwenden, so self.currentid statt currentid:

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

InformationsquelleAutor defuz

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.