Hm...also die Timerfunktion war wirklich unübsersichtlich, da keine leerzeilen und keine tabstopps drin waren (die hab ich mir selber reingemacht).
Ansonsten war das einzigst unübersichtliche die vielen Shapes.
Das hat aber doch nichts mit Canvas zu tun
Du hast 2 Shapes für die Grünlampe.
1 würde reichen.
Und dazu bräuchte man nichtmal Canvas
Aber ist natürlich deine Sache.
Wie gesagt, Leerzeilen sind nicht unbedingt nötig (ich mach sie aber auch immer), TabStopps allerdings schon.
Wenn man denn mal einen Fehler hat muss man ganz genau den Code durchgehen und jede Zeile nachdenken was sie macht und immer dran denken wo man sich befindent
air
P.S. Dazu sollte man übrigens immer kommentieren