Code: Alles auswählen
def ControlAPairOfPins(FirstPin,FirstState,SecondPin,SecondState):
GPIO.output(int(FirstPin), FirstState == "1")
GPIO.output(int(SecondPin),SecondState == "1")hmmmm erstmal danke für deine Anregung, aber bitte gibt es das auch in deutsch zu lesen, denn irgendwie ist ein so hochkonzentriertes Englisch nicht so ganz mein Ding....Sirius3 hat geschrieben:als nächstes solltest Du Dir angewöhnen, Dich an PEP-8 zu halten. Vor allem die Einrückung von 4 Leerzeichen.
Dann selltest Du `global` vermeiden, indem Du die Werte per `return` an den Aufrufer zurückgibst.
Haben wir doch. Aber du verwendest noch immer gloabl-Anweisungen.Zappelmann hat geschrieben:Das hättet ihr aber auch direkt schreiben können das es so einfach ist ^^.
Ja, PEP 8 ist ein Style Guide an den du dich unbedingt halten solltest. Zumindest dann, wenn andere deinen Code lesen sollen. Was hier ja der Fall ist. Ansonsten solltest du einfach die Idee vergessen, dass es Dokumentation auf Deutsch gibt. Um Englisch wirst du nicht herumkommen.Zappelmann hat geschrieben:Aber ich werde mal weiter Versuche damit starten und schauen ob ich das richtig umgesetzt bekomme. Aber dieses PEP-8 Das muss ich erstmal verstehen was du mir damit sagen möchtest. Dann dahinter bin ich jetzt bisher noch nicht gekommen auser das es sich scheinbar um ein Style für Python Code geht. Aber vielleicht hast du da ja was ähnlihces was es auch auf deutsch gibt.
Code: Alles auswählen
import time
count = 0
def counter():
while True:
count = count + 1
print count
time.sleep(1)
counter()
Code: Alles auswählen
UnboundLocalError: local variable 'count' referenced before assignmentCode: Alles auswählen
import time
count = 0
def counter(count):
while True:
count = count + 1
print count
time.sleep(1)
counter(count)Code: Alles auswählen
import time
def counter():
count = 0
while True:
count = count + 1
print count
time.sleep(1)
def main():
counter()
if __name__ == '__main__':
main()Code: Alles auswählen
import time
from itertools import count
def counter():
for i in count(1):
print i
time.sleep(1)
Code: Alles auswählen
def counter(start=0):
# usw.
Code: Alles auswählen
from itertools import count
from time import sleep
def counter(start=0):
for c in count(start):
print(c)
sleep(1)
Code: Alles auswählen
status = 0
class EmailThread(threading.Thread):
def __init__(self):
threading.Thread.__init__(self)
def run(self):
global status
while True:
#mach was
if habe_mail == 1: # status wird auf 1 gesetzt
status = 1
def main():
global status
while True:
if status == 0:
pass # mach nichts
elif status = 1:
# Aufruf weiterer Funktionen
Aber man bekäme doch einen Teil davon wieder herein durch die ansonsten anfallenden Heizkosten, die man sich dadurch spart. Ich meine, angesichts des Wetters heute...EyDu hat geschrieben:...sparst du dir auch das Busy-Waiting und treibst deine Stromrechnung nicht in ungeahnte Höhen.
Code: Alles auswählen
# Weg damit, globals sind vorallem mit Threads Ausgeburten der Hölle!
# status = 0
class EmailThread(threading.Thread):
def __init__(self, status):
threading.Thread.__init__(self)
self.status = status
def run(self):
#global status
while True:
#mach was
if habe_mail == 1: # status wird auf 1 gesetzt
self.status.set()
def main():
#global status
# anstelle der globalen Variable nehmen wir eine threadsichere boolsche Variable.
status = threading.Event()
while True:
if not status.is_set():
pass # mach nichts
# vllt. dem Teil ein sleep gönnen ;)
#elif status = 1:
# nix elif hier, da eh nur 1|0 unterschieden wird
else:
...
# Aufruf weiterer Funktionen
...
# iwie fehlt noch der Aufruf des Mailthreads (am besten vor dem while, sonst wird das nie erreicht),
# da kommt das Flag mit rein:
emailthread = EmailThread(status)
# und hier rennt er los ...
emailthread.start()