tkinter Error

AroPix · 20. Januar 2018

Ja ich weiss ich bin doch noch mal da,

Ich habe jetzt den ProgrammCode erfolgreich repariert nur wenn ich jetzt meine Befehle im Programm starten will kommt dieser Fehler:

Code

Exception in Tkinter callback
Traceback (most recent call last):
File "/usr/lib/python3.5/tkinter/__init__.py", line 1562, in __call__
return self.func(*args)
File "/home/pi/Schreibtisch/GPIO Programm.py", line 20, in LedEin
t = 1.0/g.get
TypeError: unsupported operand type(s) for /: 'float' and 'method'

Hier nochmal mein Code damit ich auch wirklich sicherstellen kann dass er richtig ist:

Code

import RPi.GPIO as GPIO
import time
from tkinter import *
GPIO.setmode(GPIO.BCM); LED = [4,18,23,24]
for i in LED:
GPIO.setup(i, GPIO.OUT, initial=0)
w = 5
muster = [
("Lauflicht nach links",1), ("Blinken",2),
("Lauflicht nacht rechts",3)
]
root = Tk(); root.title("LED"); v = IntVar(); v.set(1)
g = IntVar(); g.set(5)
def LedEin():
e = v.get()
t = 1.0/g.get
if e == 1:
for i in range(w):
for j in range(4):
GPIO.output(LED[j], True); time.sleep(t)
GPIO.output(LED[j], False)
elif e == 2:
for i in range(w):
for j in range(4):
GPIO.output(LED[j], True)
time.sleep(t)
for j in range(4):
GPIO.output(LED[j], False)
time.sleep(t)
else:
for i in range(w):
for j in range(4):
GPIO.output(LED[3-j], True); time.sleep(t)
GPIO.output(LED[3-j], False)
Label(root,
text="Bitte Button klicken, um das Lauflicht zu starten").pack()
for txt, m in muster:
Radiobutton(root, text = txt, variable = v,
value = m).pack(anchor=W)
Label(root, text="Geschwindigkeit").pack()
Scale(root, orient=HORIZONTAL, from_ = 1, to = 10,
variable = g).pack()
Button(root, text = "Start", command=LedEin).pack(side=LEFT)
root.mainloop()
GPIO.cleanup()

Alles anzeigen

Und ja ich weiss jetzt wie man einen Code einfügt

Mit vielen Grüßen Florian aka. AroPix:)

noisefloor · 20. Januar 2018

Hallo,

erst mal zu Code:

Woher hast du denn die dolle Idee, Befehle durch Semikolons zu trennen? Syntaktisch versteht Python das zwar, aber das macht man nie. Nie. Nie. Ein Befehl -> eine Zeile.
Du solltest deine Code besser strukturieren. Z.B. alle Funktionen für die LEDs hintereinander, dann die GUI Sachen. Es funktioniert zwar auch so, ist aber extrem schwer für Aussenstehende zu lesen und nachzuvollziehen.
Konstanten / konstante Werte schreibt man per Konvention groß (bei dir: W und MUSTER). Funktionsnamen schreibt man klein_mit_unterstrich (also led_ein, nicht LedEin)
Sternchenimporte möchte man vermeiden, weil man dann nicht weiß, a) welche Konstanten, Funktionen und Klassen im Namensraum landen und b) man im Quellcode auch nicht wirklich nachvollziehen kann, wo was her kommt.
Konzeptionelles Problem: deine GUI blockiert (= ist eingefroren), solange die Funktion LedEin läuft.

Zur Frage:

Du musst die Funktion auch _aufrufen - tust du aber nicht, du referenzierst die Funktion nur. Tipp: schau' mal nach fehlenden Klammern.

Gruß, noisefloor

schlizbäda · 24. Januar 2018

...also wenn ich mir den Code so anschaue, dann fällt mir unser Stammtisch vorgestern abend beim Wirt wieder ein, umgemünzt auf Murphy's Law:

Zitat

Silentium, Silentium!

Da Murphy håd si ned schwoam låssn und håd a boor Bugs einagstäid!

Auf'n Murphy, a dreifachs

De - BUG!

De - BUG!

De - BUG!

Alles anzeigen

AroPix : Sorry, absolut nichts gegen Dich, das fiel mir gerade so ein und ist nicht persönlich gemeint. Ich konnte es mir einfach nicht verkneifen! Dummerweise hat's zufällig Dich derbröselt, Du schriebst einfach den letzten Beitrag über einen fehlerbehafteten Code.

Nix für ungut,

schlizbäda