brauche Hilfestellung bei Parameterübergabe bzw. beim speichern eines Wertes

keepfear · May 8th 2020

Ich grüße euch,

ich bin seit ein paar Tagen daran mein Script für eine Alarmanlage umzuschreiben und übersichtlicher zu gestalten und hatte dadurch auch schon einige Aha-Effekte beim lernen von Python.

Jetzt bin ich an einer Stelle wo ich nicht mehr weiter komme und hoffe das der ein oder andere mir zumindest einen Stups in die richtige Richtung gibt.

Ich nutze Telegram um den Alarm Ein- bzw. Auszuschalten.

Beim Einschalten soll zu erst geprüft werden ob alle Türen geschlossen sind. Wenn dies der fall ist schaltet der Alarm Ein ansonsten wird mir gesagt welche Tür noch offen ist bzw. geschlossen werden muss.

Das Funktioniert soweit.

Mein Problem ist das ich bei eingeschaltetem Alarm keine Einbruchsmeldung bekomme.

Die Idee war jetzt eine Statusvariable in der "def send_message" funktion zu setzen. Dann hab ich gemerkt das die Variable bei jedem Funktionsaufruf zurückgesetzt wird.

Da kommt der Schlips ins Rad. Ich habe keine Ahnung wie ich das lösen soll.

Python

#!/usr/bin/env python
import datetime
import telepot
from telepot.loop import MessageLoop
from gpiozero import Button
import time
import functools

now = datetime.datetime.now()
dateTime = now.strftime("%d/%m/%Y %H:%M")

#--------------------------------------------------------
#Telgram 
#--------------------------------------------------------
token = "Zahlen- und Buchstabensalat"
chat_id = irgeneine komische nummer

#--------------------------------------------------------
#Bezeichnung der einzelnen Türkontakte 
#--------------------------------------------------------
holztuer = Button(17, pull_up=False, bounce_time=0.1)
werkstatttuer = Button(18, pull_up=False, bounce_time=0.1)
#bartuer = Button(22, pull_up=False, bounce_time=0.1)
#westtuer = Button(27, pull_up=False, bounce_time=0.1)

#--------------------------------------------------------
#Telegram, Prüfung Türen Auf/zu, Alarm Ein/Aus
#--------------------------------------------------------
def handle(msg):
    chat_id = msg['chat']['id']
    command = msg['text']
 
    print(f"Received: {command}")

    if command == "/an":
        print("prüfe ob alle tueren geschlossen")             
        if not holztuer.is_pressed:                       
            send_message(an = False, text = 2, door = "Holztür")      
            print("holztuer offen")
        elif not werkstatttuer.is_pressed:
            send_message(an = False, text = 2, door = "Werkstatttür")      
            print("Werkstatttür offen")
        else:
            print("alle tueren geschlossen")
            send_message(an = True)
            print("alarm an")
    elif command == "/aus":
        send_message(an = False, text = 1)
        print("alarm aus")
        
#--------------------------------------------------------       
#Nachricht an Telefon
#--------------------------------------------------------
def send_message(an = None, text = 0, onOff = "", door = "", openClose = ""):
    scharf = False
    if an:
        bot.sendMessage (chat_id, f"Alarm AN. \n {dateTime}")
        scharf = True
    elif not an:
        if text == 1:
            bot.sendMessage (chat_id, f"Alarm AUS. \n {dateTime}")
            scharf = False
        elif text == 2:
            bot.sendMessage (chat_id, f"{door} noch offen. Alarm AUS. \n {dateTime}")
    elif scharf:
        bot.sendMessage (chat_id, f"Alarm!!! {door} wurde {openClose} \n {dateTime}")

#--------------------------------------------------------
#Überwachung der Tueren bei eingeschaltetem Alarm
#--------------------------------------------------------
holztuer.when_released = functools.partial(send_message, door ="Holztuer", openClose = "geöffnet")
holztuer.when_pressed = functools.partial(send_message, door ="Holztuer", openClose = "geschlossen")
werkstatttuer.when_released = functools.partial(send_message, door ="Werkstatttuer", openClose = "geöffnet")
werkstatttuer.when_pressed = functools.partial(send_message, door ="Werkstatttuer", openClose = "geschlossen")

bot = telepot.Bot(f"{token}")
print(bot.getMe())

MessageLoop(bot, handle).run_as_thread()
print('Up and Running....')

#Keep the program running.
while 1:
   time.sleep(1)

Display More

beste Grüße und schönes Wochenende

Hofei · May 8th 2020

Wie funktioniert denn ganz generell die Einbruchserkennung.

Die Lösung liegt aber wohl im erstellen einer Klasse zur Verwaltung des Alarm s

keepfear · May 8th 2020

An den Türen sind Magnetkontakte verbaut.

Der Pi soll dann bei Flankenwechsel melden, welche Tür geöffnet wurde.

El_Zetto · May 9th 2020

Du hast if statements nicht verstanden.

Du bekommst keine Nachricht, weil Zeile 65 niemals erreicht werden kann.

Code

if an:
    pass
elif scharf and an:
    pass

Wie soll das zweite statement wahr sein, ohne dass das erste wahr ist und damit kein elif mehr ausgeführt wird?

Reihenfolge tauschen oder aus dem elif ein if machen.

keepfear · May 9th 2020

Morjen El_Zetto,

hab das Script in Beitrag #1 angepasst. Das löst aber nicht mein Problem.

ein Bsp.:

Python

#!/usr/bin/env python
import datetime
import telepot
from telepot.loop import MessageLoop
from gpiozero import Button
import time
import functools

now = datetime.datetime.now()
dateTime = now.strftime("%d/%m/%Y %H:%M")

#--------------------------------------------------------
#Telgram 
#--------------------------------------------------------
token = "Zahlen- und Buchstabensalat"
chat_id = irgeneine komische nummer

#--------------------------------------------------------
#Bezeichnung der einzelnen Türkontakte 
#--------------------------------------------------------
holztuer = Button(17, pull_up=False, bounce_time=0.1)
werkstatttuer = Button(18, pull_up=False, bounce_time=0.1)
#bartuer = Button(22, pull_up=False, bounce_time=0.1)
#westtuer = Button(27, pull_up=False, bounce_time=0.1)

#--------------------------------------------------------
#Telegram, Prüfung Türen Auf/zu, Alarm Ein/Aus
#--------------------------------------------------------
def handle(msg):
    chat_id = msg['chat']['id']
    command = msg['text']
 
    print(f"Received: {command}")

    if command == "/an":
        print("prüfe ob alle tueren geschlossen")             
        if not holztuer.is_pressed:                       
            send_message(an = False, text = 2, door = "Holztür")      
            print("holztuer offen")
        elif not werkstatttuer.is_pressed:
            send_message(an = False, text = 2, door = "Werkstatttür")      
            print("Werkstatttür offen")
        else:
            print("alle tueren geschlossen")
            send_message(an = True)
            print("alarm an")
    elif command == "/aus":
        send_message(an = False, text = 1)
        print("alarm aus")
        
#--------------------------------------------------------       
#Nachricht an Telefon
#--------------------------------------------------------
def send_message(an = None, text = 0, onOff = "", door = "", openClose = ""):
    scharf = False
    print(f"wenn die Funktion send_message aufgerufen wird. scharf = {scharf} ...")
    if an:
        bot.sendMessage (chat_id, f"Alarm AN. \n {dateTime}")
        scharf = True
        print(f"...if an. scharf = {scharf}")
    elif not an:
        if text == 1:
            bot.sendMessage (chat_id, f"Alarm AUS. \n {dateTime}")
            scharf = False
            print(f"...if not an. scharf = {scharf}")
        elif text == 2:
            bot.sendMessage (chat_id, f"{door} noch offen. Alarm AUS. \n {dateTime}")
    elif scharf:
        print("warte auf einbruch")
        bot.sendMessage (chat_id, f"Alarm!!! {door} wurde {openClose} \n {dateTime}")
#--------------------------------------------------------
#Überwachung der Tueren bei eingeschaltetem Alarm
#--------------------------------------------------------
holztuer.when_released = functools.partial(send_message, door ="Holztuer", openClose = "geöffnet")
holztuer.when_pressed = functools.partial(send_message, door ="Holztuer", openClose = "geschlossen")
werkstatttuer.when_released = functools.partial(send_message, door ="Werkstatttuer", openClose = "geöffnet")
werkstatttuer.when_pressed = functools.partial(send_message, door ="Werkstatttuer", openClose = "geschlossen")

bot = telepot.Bot(f"{token}")
print(bot.getMe())

MessageLoop(bot, handle).run_as_thread()
print('Up and Running....')

#Keep the program running.
while 1:
   time.sleep(1)

Display More

Angenommen ich starte das Script und sage dem Pi er solle doch den Alarm anschalten und ich jetzt eine Tür öffne kommt folgendes bei rum.

Python

Up and Running.... #<----Telegram ist start klar
Received: /an #<----ich sende eine Nachricht vom Telephon an den Pi.
prüfe ob alle tueren geschlossen #<----sind alle Türen zu?....
alle tueren geschlossen#<----....JA.
wenn die Funktion send_message aufgerufen wird. scharf = False ...#<------Funktion zur Rückmeldung auf Telegram/Telefon
...if an. scharf = True #<----Statusvariable 
alarm an #<--- Türen sind alle zu. Alarm an. Danke
wenn die Funktion send_message aufgerufen wird. scharf = False ... #<----Tür wurde geöffnet (holztuer.when_released = functools.partial(send_message, .....)
wenn die Funktion send_message aufgerufen wird. scharf = False ... #<----Tür wurde geschlossen holztuer.when_pressed = functools.partial(send_message, .....)

sprich die zeilen...

Python

#--------------------------------------------------------
#Überwachung der Tueren bei eingeschaltetem Alarm
#--------------------------------------------------------
holztuer.when_released = functools.partial(send_message, door ="Holztuer", openClose = "geöffnet")
holztuer.when_pressed = functools.partial(send_message, door ="Holztuer", openClose = "geschlossen")
werkstatttuer.when_released = functools.partial(send_message, door ="Werkstatttuer", openClose = "geöffnet")
werkstatttuer.when_pressed = functools.partial(send_message, door ="Werkstatttuer", openClose = "geschlossen")

...rufen die Funktion immer wieder neu auf, wobei die Variable "Scharf" immer wieder neu aufgerufen wird, also immer False ist -> if scharf: niemals wahr.

Und das ist das Problem. Da egal was ist, die Türen immer geprüft werden und ich keine Meldung möchte wenn der Alarm aus ist. Deswegen die Statusvariable.

Ich hoffe ich hab das etwas besser aufklamüsert.

Jürgen Böhm · May 9th 2020

Nur eine Anmerkung:

Quote from keepfear

#!/usr/bin/env python

Du solltest auf Python3 umschwenken, Python2 wird nicht mehr gepflegt.

Bevor Du in noch mehr Arbeit investierst.

MfG

Jürgen

keepfear · May 9th 2020

Mahlzeit,

Jürgen Böhm

Danke für den Hinweis.

Ist das Script denn in Python2 geschrieben?

Hofei

Ja, ich muss mir die Klassengeschichte noch anschauen.

Schönes Wochenende

MistyFlower59469 · May 9th 2020

python ist python 2. Wenn du python 3 willst, musst du python3 benutzen.

El_Zetto · May 9th 2020

Quote from keepfear

wobei die Variable "Scharf" immer wieder neu aufgerufen wird, also immer False ist

Und warum machst du das? Wenn du die Variable bei jedem Funktionsaufruf als False definierst, kann das ja auch nicht klappen.

Einmal am Anfang außerhalb der Funktionen initialisieren und dann nur noch ändern falls der Alarm an oder ausgeschaltet wird.

keepfear · May 10th 2020

Sprich ich muss mit Rückgabewerten arbeiten?

El_Zetto · May 10th 2020

Arbeite bitte ein Python Tutorial deiner Wahl zum Thema Scope von Variablen durch um das grundlegende Prinzip zu verstehen. Danach solltest du das Problem selber lösen können.

Was du brauchst ist eine globale Variable, wie es auch deine gpiozero buttons sind.

Für das Beispiel kannst du eine globale Variable nutzen. Für komplexere Angelegenheiten solltest du dich aber mit Klassen auseinandersetzen.

keepfear · May 10th 2020

Ich bedanke mich bei allen.

Die globale variable werde ich vorerst nicht nutzen und mich mit dem, was hier schon erwähnt wurde, befassen.

Ich melde mich ggf. wieder.

Schönen Sonntag noch.

keepfear · Jun 3rd 2020

Mahlzeit,

ich poste einfach mal mein fertiges Script. Ist sicher nicht ganz rund, läuft aber.

Python

#!/usr/bin/python3
import datetime
import telepot
import time
import schedule
import functools
from telepot.loop import MessageLoop
from gpiozero import Button

now = datetime.datetime.now()
dateTime = now.strftime("%d/%m/%Y %H:%M")

#Telgram Bot Schlüssel
token = "vom Botfather gegebener Schlüssel"
bot = telepot.Bot(f"{token}")

#Bezeichnung der einzelnen Türkontakte 
holztuer = Button(17, pull_up=False, bounce_time=0.1)
werkstatttuer = Button(18, pull_up=False, bounce_time=0.1)
bartuer = Button(22, pull_up=False, bounce_time=0.1)
westtuer = Button(27, pull_up=False, bounce_time=0.1)

class Alarm():
    #Steuern der Einbruchsmeldung, Nachricht wird gesendet wenn an = True
    an = False

    def __init__(self, tuer = ""):
        self.tuer = tuer

    def report_burglary(self, text = ""):
        if Alarm.an:
            if text == "geöffnet":
                a = 3
                t = self.tuer
                oc = text
                send_message(a, t, oc)
            elif text == "geschlossen":
                a = 3
                t = self.tuer
                oc = text
                send_message(a, t, oc)


def handle(msg):
    #empfangen der Nachrichten zum ein-/ausschalten des Alarms
    chat_id = msg['chat']['id']
    command = msg['text']

    print(f"Received: {command}")

    if command == "/an":
        check_door(on = True)
    elif command == "/aus":
        check_door(on = False)


def check_door(on):
    #Prüfung Türen Auf/zu, Alarm Ein/Aus
    if on:
        if not holztuer.is_pressed:
            t = "Holztuer"
            Alarm.an = False
            send_message(2, t)
        elif not werkstatttuer.is_pressed:
            t = "Werkstatttür"
            Alarm.an = False
            send_message(2, t)
        elif not bartuer.is_pressed:
            t = "Bartür"
            Alarm.an = False
            send_message(2, t)
        elif not westtuer.is_pressed:
            t = "Westtür"
            Alarm.an = False
            send_message(2, t)
        else:
            Alarm.an = True
            t = "AN"
            send_message(1, t)
    elif not on:
        Alarm.an = False
        t = "AUS"
        send_message(1, t)


def send_message(a, t, oc = ""):
    chat_id = Zahlenreihe
    #sende Nachricht und Auswahl des Textes, je nach Zustand der Alarmanlage.
    #1 = Alarm an/aus, 2 = überprüfung Tuer offen/geschlossen,
    #3 = Einbruchsmeldung, 4 = Erinnerung Alarm noch aus
    if a == 1: 
        bot.sendMessage (chat_id, f"Anlage {t} \n{dateTime}")
    elif a == 2:
        bot.sendMessage (chat_id, f"{t} noch offen. \nAlarm ist AUS. \n{dateTime}")
    elif a == 3:
        bot.sendMessage (chat_id, f"Alarm! {t} wurde {oc} \n{dateTime}")
    elif a == 4:
        bot.sendMessage (chat_id, f"{t}")


def remind_alarmstatus(a):
    #Erinnerung das der Alarm zur definierten Zeit (schedule in main) noch aus ist
    #Automatische Aktivierung nach x Sekunden
    if a:
        while not Alarm.an:
            t = (f"Alarm ist noch AUS.\n" 
                 f"Alarm wird in 5 min automatisch eingeschaltet.\n"
                 f"Danach muss der Alarm \"händisch\" Aktiviert werden ")
            send_message(4, t)
            time.sleep(300)
            t = "Prüfe Türen"
            send_message(a, t)
            check_door(on = True)
            return schedule.CancelJob


def main():
    #Telegram Chatinfo und warten auf Nachrichten
    print(bot.getMe())

    MessageLoop(bot, handle).run_as_thread()
    print('Up and Running....')

    #Initialisierung Türen
    ht = Alarm("Holztür")
    wt = Alarm("Werkstatttür")
    bt = Alarm("Bartür")
    wet = Alarm("Westtür")
    
    #Überwachung der Türen, sendet Nachricht bei Alarm ein
    holztuer.when_released = functools.partial(ht.report_burglary, text = "geöffnet")
    holztuer.when_pressed = functools.partial(ht.report_burglary, text = "geschlossen")
    werkstatttuer.when_released = functools.partial(wt.report_burglary, text = "geöffnet")
    werkstatttuer.when_pressed = functools.partial(wt.report_burglary, text = "geschlossen")
    bartuer.when_released = functools.partial(bt.report_burglary, text = "geöffnet")
    bartuer.when_pressed = functools.partial(bt.report_burglary, text = "geschlossen")
    westtuer.when_released = functools.partial(wet.report_burglary, text = "geöffnet")
    westtuer.when_pressed = functools.partial(wet.report_burglary, text = "geschlossen")
    
    schedule.every().day.at("22:00").do(remind_alarmstatus, a = True)

    while True:
        schedule.run_pending()
        time.sleep(1)

if __name__ == "__main__":
    main()

Display More

Edit: Sry. das ich das Thema nochmal hoch hole. Hab das Script nochmal geändert.