While Schleife Button

schnibli · Jan 7th 2016

Hallo zusammen,
Kann mir jemand sagen was ich Falsch mache?

Code

while (GPIO.input(37) == GPIO.HIGH) and (GPIO.input(38) == GPIO.HIGH) and (GPIO.input(36) == GPIO.HIGH):
			time.sleep(0.1)	

		if GPIO.input(37) == GPIO.HIGH:
			mittag = 1.00
			lcd.lcd_display_string("Mittagspause = 1h", 4)
		elif GPIO.input(38) == GPIO.HIGH:
			mittag = 0.50
			lcd.lcd_display_string("Mittagspause = 0.5h", 4)
                        lcd.lcd_display_string("Arbeitsende", 3)   
		elif GPIO.input(36) == GPIO.HIGH:
			mittag = 0.00
			lcd.lcd_display_string("Mittagspause = 0.0h", 4)
                        lcd.lcd_display_string("Arbeitsende", 3)

Display More

Dies sollte dafür dienen dass, das Programm wartet bis ein Taster gedrückt wird.

meigrafd · Jan 7th 2016

Also als erstes fallen mir falsche Einrückungen auf - das ist bei Python extrem wichtig.

Dann fehlen mir Informationen was GPIO 37, 38 oder 36 sein sollen? Auf welchen Taster soll gewartet werden? :s

Matze208 · Jan 7th 2016

Irgendwie komisch ist auch, dass du im ersten while überprüfst, ob alle drei GPIO-Ports auf "HIGH" sind und anschließend versuchst du jeden Status einzeln abzufragen, ob jeder Taster (?) den Wert "HIGH" hat.
Gruß
Matze

lochraster · Jan 7th 2016

Ich bin ja gerade erst am Lernen von Python, aber für mich sehen deine Einrückungen seltsam aus. Sollte das time.sleep nicht mit der if-Anweisung bündig sein?
Python "lebt" von den richtigen Tabs.

Dibbelschisser · Jan 7th 2016

Also wenn du gar nicht mehr aus dieser Schleife raus willst, würde ich einfach eine Dauerschleife empfehlen. Du schreibst einfach while(1). Danach kannst du mit den anderen if-Abfragen den Status deiner Taster überprüfen. Kenne mich leider in Python nicht aus aber das funktioniert zumindest in allen Programmiersprachen die ich kenne.
Das was du da gebaut hast ist eher, solange alle Taster "High" sind, überprüfe jeden einzelnen Taster ob dieser "High" ist und führe dann die entsprechende Funktion aus. Also gehst du davon aus, dass alle Taster auf "High" stehen und überprüfst danach nochmal das gleiche. Wie gesagt wenn du das in ner Dauerschleife haben willst nehme while(1) und den Rest kannste (bis auf die Einrückungen aber kenne mich da nicht aus) so lassen. Wenn nicht musste vielleicht dein Problem etwas genauer beschreiben.

Grüße

schnibli · Jan 7th 2016

Nein es soll gewartet werden bis einer der Eingänge schaltet
36,37 und 38 sind Eingänge.

Das Programm soll warten bis einer der Eingänge schaltet!
Und anschliessen je nach dem welcher geschaltet hat den wert setzen.

bootsmann · Jan 7th 2016

Wie Dibbelschisser schon schrieb...
... überleg/überdenk dir die Bedingung der ``while`` Schleife

schnibli · Jan 7th 2016

Wie komme ich denn aus der schleiffe raus :s

Code

while (1):	
			if GPIO.input(37) == GPIO.HIGH:
				mittag = 1.00
				lcd.lcd_display_string("Mittagspause = 1h", 4)
				lcd.lcd_display_string("Arbeitsende", 3)   
			elif GPIO.input(38) == GPIO.HIGH:
				mittag = 0.50
				lcd.lcd_display_string("Mittagspause = 0.5h", 4)
                        	lcd.lcd_display_string("Arbeitsende", 3)   
				next = 1
				print("0.5H")
			elif GPIO.input(36) == GPIO.HIGH:
				mittag = 0.00
				lcd.lcd_display_string("Mittagspause = 0.0h", 4)
                        	lcd.lcd_display_string("Arbeitsende", 3)

Display More

Erledigt danke

meigrafd · Jan 7th 2016

Die while wird so lange ausgeführt wie dessen Bedingung zutrifft. In Deinem Fall wird die while also ausgeführt solange alle 3 GPIO's ein HIGH anzeigen, was somit nicht dem entspricht was du beschrieben hast.

//EDIT: erneut: In Python sind Einrückungen extrem wichtig! Du hast weiterhin 2 fehlerhafte Einrückungen!

Eine while beendest du durch die Bedingung, oder durch break

schnibli · Jan 7th 2016

Hat sich erledigt!
In meinem Programm sind die Einrückungen richtig, ich weis nicht wiso das nicht stimmt im Forum
(lcd.lcd)