Torüberwachung mit einem Raspberry Pi und Magnetsensor

Jonathan_5805 Das ist nicht korrekt eingerückt, das kommt so nicht am Compiler vorbei.

Man verschachtelt keine Funktionsdefinitionen wenn man nicht tatsächlich ein Closure haben möchte. Das macht den Code unübersichtlicher und die Funktionen die in anderen Funktionen stecken, kann man nicht separat testen. Die innerste Funktion — read_sensor() — wird auch gar nicht verwendet‽ Und die tut auch gar nicht was der Funktionsname suggeriert. Und auch execute_after_1_minutes() stimmt nicht. Der Leser interessiert sich vielleicht auch weniger dafür nach wie vielen Minuten die Funktion etwas ausführt (die Funktion selbst wird sofort ausgeführt!), sondern was die Funktion macht.

Um tatsächlich etwas nach einer bestimmten Zeit auszuführen gibt es threading.Timer.

Beim öffnen von Textdateien sollte man immer explizit die Kodierung angeben.

fobj_out ist kein guter Name für ein log_file.

Für das logging steht da im grunde zweimal der gleiche Code. Das sollte in eine Funktion. Wobei man auch überlegen sollte einfach das `logging`-Modul zu verwenden.

Die 20 Minuten stehen zwei mal im Quelltext, einmal als Text und einmal in Sekunden. Das ist fehleranfällig beim schreiben und ändern, weil man immer daran denken muss beide Stellen zu ändern, und sich bei der Umrechnung auch nicht vertun darf. Für so etwas definiert man einmal eine Konstante, die sich dann leicht ändern lässt, und leitet die Werte im Programm dann von dieser Konstante ab.

Zwischenstand (ungetestet):

import smtplib
import threading
import time
from datetime import datetime
from email.mime.text import MIMEText

from RPi import GPIO as gpio

MAGNET_GPIO = 17

EMAIL_DELAY = 20  # In minutes.


def log(message):
    with open("log.txt", "a", encoding="utf-8") as log_file:
        text = f"{datetime.now():%Y-%m-%d %H:%M:%S}: {message}\n"
        log_file.write(text)
        print(text, end="")


def send_email():
    with smtplib.SMTP("192.168.10.13", 25) as server:
        server.sendmail(
            "Torsteuerung@stannek-gmbh.com",
            "j.zwosta@stannek-gmbh.com",
            MIMEText(
                f"Das Rolltor ist seit {EMAIL_DELAY} Minuten geöffnet"
            ).as_string(),
        )


def main():
    try:
        gpio.setmode(gpio.BCM)
        gpio.setup(MAGNET_GPIO, gpio.IN, pull_up_down=gpio.PUD_UP)
        while True:
            gpio.wait_for_edge(MAGNET_GPIO, gpio.BOTH)

            if not gpio.input(MAGNET_GPIO):
                log("Tor offen")

                timer = threading.Timer(EMAIL_DELAY * 60, send_email)
                timer.start()

                while not gpio.input(MAGNET_GPIO):
                    time.sleep(0.05)
            else:
                log("Tor geschlossen")

            time.sleep(0.05)

    finally:
        gpio.cleanup()


if __name__ == "__main__":
    main()

Das Programm hat jetzt das Problem, dass für jedes öffnen nach 20 Minuten eine E-Mail versendet wird. Man muss sich also überlegen unter welchen Bedingungen ein eventuell schon laufender Timer wieder beendet werden muss. 🤔 (Es sind zwei Stellen im Programmablauf an denen man das machen muss.)

Quote from Jonathan_5805

nur leider Funktioniert genau garnix

Das ist ja mal eine sehr hilfreiche Fehlermeldung...

Bitte beschreibe genauer, was Du probiert hast und welche Fehlermeldungen kommen.
Am Besten poste die Ein- und Ausgaben in einem Codeblock.

Jonathan_5805 Was meinst Du mit „die ausgabe in der commandline (ist in deinem code ja nicht mit inbegriffen)“? Das Programm sollte genau das gleiche machen wie der ursprüngliche Code. Das sind letztlich nur Umformungen gewesen, keine andere Programmlogik oder so.

Deine überarbeitete Version macht falsche Sachen. Weder muss man die Logdatei lesend öffnen, noch ist es korrekt die dann nur in einem der beiden Fälle wieder zu schliessen. Diese Änderungen haben den Code verschlechtert. Warum hast Du das so gemacht? Was war der Gedanke dahinter?

Und warum ist "tor" in der Ausgabe jetzt klein geschrieben und die Zeilenenden so komisch als weiteres Zeichenkettenliteral abgetrennt?

Es wurde auch so gut wie gar nicht auf die ganzen Kritikpunkte eingegangen.

Jonathan_5805 Gegen keine Ahnung kann man ja was machen. In der Python-Dokumentation gibt es beispielsweise ein Grundlagentutorial das man mal durchgearbeitet haben sollte.

Moin,

der Code von __blackjack__ ist nach meiner Auffassung richtig.

Nun zu sagen, dass Email nicht funktioniert, finde ich ein wenig vermessen. Das ging ja vorher schon nicht.

Sicher , dass der Email Server Emails ohne vorherige Authentifizierung annnimmt?

Ich sehe nämlich nirgendwo ein Login.

Etwas ungeschickt finde ich das Schreiben ins Logfile. Prinzipiell würde ich das Modul logging benutzen.

Ist zugegebenermassen Geschmacksfrage. Allerdings würde ich aber im gegebenen Fall das Log mit absoluter Pfadangabe schreiben.

Auch um sicherzustellen, dass das Log auch da landet , wo ich es haben möchte. Default gehören Logs nach /var/log.

Grüße

Quote from ottosieben

Sicher , dass der Email Server Emails ohne vorherige Authentifizierung annnimmt?

Ich sehe nämlich nirgendwo ein Login.

Muss auch nicht.
Da es eine private Adresse ist wird es sicher ein internes Mailrelay sein.

Die Frage liesse sich supereinfach klären wenn das eine eigenständige Funktion wäre, die nicht innerhalb einer anderen Funktion definiert ist, denn dann könnte man das Modul einfach importieren und die Funktion mal testweise aufrufen. Also Jonathan_5805 könnte das machen, uns fehlt ja der entsprechende Rechner hinter der privaten IP-Adresse. 🤓

In dem man die Funktion nicht in einer anderen Funktion definiert.