Whats wrong with my code?

[waqask64]

only the led will work when the button is pressed. the buzzer does nothing

from gpiozero import LED, Button, Buzzer
from signal import pause

led = LED(17)
button = Button(2)
buzzer = Buzzer(21)

button.when_pressed = buzzer.on
button.when_pressed = led.on
button.when_pressed = buzzer.on
button.when_pressed = buzzer.off

pause()

[sprinkmeier]

I'm guessing that the

button.when_pressed

is a simple function pointer, so assigning multiple things to it just makes the last one win.

try something like this:

Code: Select all

def foo():
    buzzer.on()
    led.on()

button.when_pressed = foo

just to check.... does the buzzer/LED work as expected when you just call the on/off functions directly?

[waqask64]

Yes it works when i jus activate them individually

[PeterO]

What is wrong is that you don't seem to understand how callbacks work.
Try this to start with....

Code: Select all

from gpiozero import  Buzzer
from signal import pause


buzzer = Buzzer(21)

button.when_pressed = buzzer.on
button.when_released = buzzer.off

pause()

Note: The above code is untested.

Then take sprinkmeier's suggestion and put all the "pressed" handling code into one function and all the "released" handling code into another function and set these functions as the "when_pressed" and "when_released" handlers.

PeterO