Python Programm - komme nicht weiter

[H. C. (sunshine)]

Hi,

ich komme einfach nicht weiter mit meinem 1.Python Programm. Ich möchte 
bei Interrupt (Unterbrechung der Lichtschranke) eine Mail bekommen und 
eine Benachrichtigung, den der Pin 600 Sekunden lang unerbrochen ist.
Hier ein Ausschnitt von meinem Code:

TimeOut_Alert_sec = 90
TimeOut_Disconnected_sec = 600

lastdetect = 0
Counter = 0

def setup():
        ...
        GPIO.add_event_detect(BtnPin, GPIO.BOTH, callback=detect, bouncetime=1000)
        global lastdetect       
        global Counter


def detect(channel):
        now = time.time()
        if now - lastdetect < TimeOut_Alert_sec:
                return
        if Counter == TimeOut_Disconnected_sec:
                SendMail("disconnected")
                return;
        SendMail("detected")
        lastdetect = now

def loop():
        while True:
                time.sleep(1)
                if GPIO.input(BtnPin) == GPIO.HIGH:     # Lichtschranke wieder frei          
                        Counter = 0
                if GPIO.input(BtnPin) == GPIO.LOW:     # Lichtschranke unterbrochen
                        Counter += 1

Bei Unterbrechnung der Lichtschranke erhalte ich folgende Meldung:

"...in detect
if now - lastdetect < TimeOut_Alert_sec:
Unbound LocalError: local variable 'lastdetect' referenced before 
assignment"

Ich weiß einfach nicht wo ich die beiden Variablen Counter und 
lastdetect hinstecken soll, damit ich in den beiden Funktionen detect 
und loop darauf zugreifen kann.

Ich wollte sie erst übergeben, doch das geht wohl auch nicht.

Ich wäre über jeden Tipp sehr dankbar!!

[Oliver R. (Gast)]

Das Problem ist, dass du versuchst der globalen Variablen lastdetect 
innerhalb von detect() einen Wert zuzuweisen. Das ist nur erlaubt, wenn 
du sie vorher mit "global lastdetect" deklariert hast.

Also:

def detect(channel):
        global lastdetect
        now = time.time()
        if now - lastdetect < TimeOut_Alert_sec:
                return
        if Counter == TimeOut_Disconnected_sec:
                SendMail("disconnected")
                return;
        SendMail("detected")
        lastdetect = now

Ohne diese Deklaration kannst du nur lesend zugreifen.

[Dennis X. (Gast)]

H. C. schrieb:
> "...in detect
> if now - lastdetect < TimeOut_Alert_sec:
> Unbound LocalError: local variable 'lastdetect' referenced before
> assignment"

Du greifst innerhalb einer Funktion auf eine globale Variable 
schreibend zu. Wenn das der Fall ist musst du das der Funktion auch 
mitteilen, bzw. die Variable bekannt machen:

def detect(channel):
        global lastdetect 
        now = time.time()
        if now - lastdetect < TimeOut_Alert_sec:
                return
        if Counter == TimeOut_Disconnected_sec:
                SendMail("disconnected")
                return;
        SendMail("detected")
        lastdetect = now

Falls du eine Variable innerhalb einer Funtkion nur lesen möchtest ist 
das nicht notwenig.

[Sheeva P. (sheevaplug)]

H. C. schrieb:
> ich komme einfach nicht weiter mit meinem 1.Python Programm. Ich möchte
> bei Interrupt (Unterbrechung der Lichtschranke) eine Mail bekommen und
> eine Benachrichtigung, den der Pin 600 Sekunden lang unerbrochen ist.

Was passiert denn, wenn der Pin erbrechen muß?

> Hier ein Ausschnitt von meinem Code:
>
>

> def setup():
> def loop():
>         while True:
>

Das ist kein Arduino, sondern Python. Was das "setup()"- und 
"loop()"-Zeug da soll, ist mir daher einigermaßen schleierhaft. Das 
"global" in "setup()" kannst Du Dir ebenfalls sparen, ist da überflüssig 
und nur dann nötig, wenn Du auf eine Variable des äußeren Blocks 
schreiben willst wie in "detect()" -- und genau da solltest Du dann 
"global lastdetect" auch hineinschreiben.

[H. C. (sunshine)]

DANKE!! Jetzt hab ich es verstanden! Python ist halt doch anders...
auch und sorry für den Rechtschreibfehler, bei mir wird nichts erbrochen 
und natürlich auch nicht verbrochen ;-)