Schalter auswerten und Status in Datei schreiben

Willst du den aktuellen Zustand oder ein Log?

Setz mal ein Sleep in den Loop, sonst wird das File so schnell geloescht dass du es nicht siehst.

- - -

Versuche auch mal den Modus "w+". Die offizielle Dok behauptet dass das File truncated wird, aber alle anderen sind der Ansicht dass das nicht der Fall ist...

Das würde man nicht mit einer Busy-Waiting-Schleife machen, auch nicht mit sleep(). Es gibt da ja beispielsweise die when_*-Rückrufe, die man setzen kann und wo man Ereignisse in eine Queue schreiben lassen kann, bei der man dann im Hauptprogramm die Werte heraus holt und in eine Datei schreibt.

Am besten in eine temporäre Datei, die dann zu der eigentlichen Datei umbenannt wird, damit das aus Sicht von anderen Programmen halbwegs atomar aussieht.

Den Weg oben halte ich für zu kompliziert und fehleranfällig. gpiozero kennt source und value: https://gpiozero.readthedocs.io/en/stable/source_values.html

BTW: Den Dateizeiger gpio zu nennen halte ich für keine gute Idee.

Ist das ein Taster oder ein Schalter?

Wenn der Status des Deckels erfasst werden soll, ist es wohl ein Taster. Doch egal, wie es, ich würde in diesem Fall mit Interrups arbeiten, dann kannst du dem System jeden anderen Kram überlassen, Bunte Punkte malen, Kreise zeichnen, Musike spielen, Texte verarbeiten, die du eintippst, Filme abspielen .... Und wenn ein Interrupt ausgelöst wird, springt dein Programm an.

Meine Python Skils sind sicher noch schlechter wie deine aber ich würde das so ähnlich lösen und nur schreiben wenn sich der Status wirklich geändert hat.

from gpiozero import Button
taster = Button(27)
taster.pull_up
state = "0"
oldstate = "0"
while True:
   if taster.is_pressed:
       state = "1"
   else:
       state = "0"
   if state != oldstate:
       oldstate = state
       gpio = open('./gpio_27','w')
       gpio.write(oldstate)
       gpio.close()

Oder so, wenns ein Python-Profi macht.
Ich werde einfach nicht warm mit Python.
Mir fehlen die Klammern und das Einrücken geht mir auch auf den Zeiger. Ich bin Tabs gewohnt.

Quote from Der_Imperator

wenns ein Python-Profi macht

Davon bin ich weit entfernt! Da gibt es hier vieeeel kompetentere Leute als mich.

Beim öffnen von Dateien sollte man with verwenden wenn das geht, und bei Textdateien ist es besser immer explizit die Kodierung anzugeben um Überraschungen zu vermeiden.

Und wie gesagt: Temporäre Datei und umbenennen wenn man vermeiden möchte irgendwann auch mal mit einer Leeren Datei dazustehen.

Pin-Nummer und Dateinamen würde man üblicherweise als Konstante definieren, damit man das leicht ändern kann ohne den ganzen Code durchgehen zu müssen.

Der_Imperator Die Klammern kannst Du als Kommentare hinschreiben 😉 und das Problem beim Einrücken verstehe ich nicht so ganze, denn ich benutze einfach die Tab Taste zum Ein- und Ausrücken (mit Shift) und der Editor kümmert sich darum, dass das die entsprechende Zahl von Leerzeichen wird.

Ungetestet:

#!/usr/bin/env python3
from functools import partial
from pathlib import Path
from signal import pause

from gpiozero import Button

BUTTON_PIN = 27
FILE_PATH = Path(f"/var/www/html/gpio/gpio_{BUTTON_PIN}")


def write_state(state):  # {
    temporary_filepath = FILE_PATH.with_suffix(".tmp")
    with temporary_filepath.open("w", encoding="ascii") as file:  # {
        file.write("1" if state else "0")
    # }
    temporary_filepath.rename(FILE_PATH)
# }


def main():  # {
    button = Button(BUTTON_PIN)
    button.when_pressed = partial(write_state, True)
    button.when_released = partial(write_state, False)
    pause()
# }

if __name__ == "__main__":  # {
    main()
# }

Hallo,

etwas kürzer file.write(str(int(state))), aber ob das lesbarer ist

Grüße
Dennis

__blackjack__ Ich hätte mir den state erspart und stattdessen button der Funktion übergeben. Dann sähe das so aus:

#!/usr/bin/env python3
from functools import partial
from pathlib import Path
from signal import pause

from gpiozero import Button

BUTTON_PIN = 27
FILE_PATH = Path(f"/var/www/html/gpio/gpio_{BUTTON_PIN}")


def write_state(button):
    temporary_filepath = FILE_PATH.with_suffix(".tmp")
    with temporary_filepath.open("w", encoding="ascii") as file:
        file.write(button.value)
    
    temporary_filepath.rename(FILE_PATH)



def main():
    button = Button(BUTTON_PIN)
    button.when_pressed = partial(write_state, button)
    button.when_released = partial(write_state, button)
    pause()


if __name__ == "__main__":
    main()

Auch ungetestet.

//Edit

Oder halt direkt per partial(write_state, button.value) . Dann muss man das auch nicht nochmals prüfen.

button.value müsste man noch in eine Zeichenkette wandeln, und wenn beide Rückruffunktionen gleich sind, kann man auch ein partial-Objekt erstellen. Aaaaber man kann auch einfach write_state direkt übergeben, denn wenn die Rückruffunktion genau ein (nicht-optionales) Argument erwartet, dann wird da bereits das Objekt übergeben für das die Funktion registriert wurde.

Ungetestet:

#!/usr/bin/env python3
from pathlib import Path
from signal import pause

from gpiozero import Button

BUTTON_PIN = 27
FILE_PATH = Path(f"/var/www/html/gpio/gpio_{BUTTON_PIN}")


def write_state(button):
    temporary_filepath = FILE_PATH.with_suffix(".tmp")
    with temporary_filepath.open("w", encoding="ascii") as file:
        file.write(str(button.value))
    temporary_filepath.rename(FILE_PATH)


def main():
    button = Button(BUTTON_PIN)
    button.when_pressed = button.when_released = write_state
    pause()


if __name__ == "__main__":
    main()

Ah ok! Das ist noch schöner.

Quote from DJ1NG

funtktioniert, wie ich es brauche

Was, ausgerechnet das nimmst du?

Nichts gegen Dich Der_Imperator aber wie Du selber schrubst

Quote from Der_Imperator

Meine Python Skils sind sicher noch schlechter

dem ich nicht widersprechen werde.

DJ1NG Bitte teste wenigstens eines von den Skripten, die __blackjack__ vorgeschlagen hat. Die sind um einiges pythonischer als eine Wahl.

Wenn wir `pathlib` schon nutzen, könnten wir es noch etwas mehr nutzen:

#!/usr/bin/env python3
from pathlib import Path
from signal import pause

from gpiozero import Button

BUTTON_PIN = 27
FILE_PATH = Path(f"/var/www/html/gpio/gpio_{BUTTON_PIN}")


def write_state(button):
    temporary_filepath = FILE_PATH.with_suffix(".tmp")
    temporary_filepath.write_text(str(button.value), encoding="ascii")
    temporary_filepath.rename(FILE_PATH)


def main():
    button = Button(BUTTON_PIN)
    button.when_pressed = button.when_released = write_state
    pause()


if __name__ == "__main__":
    main()

Grüße
Dennis

Quote from DJ1NG

funtktioniert, wie ich es brauche.

also ich würde das so auf gar keinen Fall implementieren. Da hier pausenlos eine Endlosschleife läuft, ist der Raspi zu nahezu 100% beschäftigt, was aus meiner Erfahrung heraus nicht nur den Raspi erhitzt sondern auch das gesamte System instabil macht. Du kannst das leicht überprüfen indem du die CPU Auslastung mit top im Terminal überprüfst. Bei mir sind das 98% Auslastung. Der richtige Ansatz ist meiner Meinung der von hyle in Beitrag #7 mit dem Auszug aus der Dokumentation.

Der Rest ist auch OK wenn man schönes Python schreiben will. Ob es auch effizienter ist, weiß ich nicht.