Cada vez que se llama a
playMusic()
intentas crear un nuevo elemento de audio. Si esto funcionara, no solo terminarías con múltiples elementos creados a la vez, sino que nunca vas a pausar uno de los creados anteriormente, pues no guardas ninguna referencia a ellos ni haces búsquedas.El elemento de audio, luego de su creación, toma algún tiempo para poder comenzar a reproducir el audio, básicamente el tiempo necesario para descargar los primeros segundos del stream que va a reproducir, por lo que la llamada a
play()
podría fallar y definitivamente no tendría sentido llamar apause()
sobre un objeto que no se está reproduciendo.Los elementos que heredan de
HTMLMediaElement
, como elHTMLAudioElement
creado al llamar aAudio()
, tienen una serie de propiedades con las que puedes evaluar su estado actual, si se está reproduciendo, etc. No me baso en ellas para la solución a tu pregunta, para no complicar las cosas, sino que dejaré el mecanismo de la variableclick
(terrible nombre para una variable por cierto), solo para mantener esto lo más simple posible.Olvidaste poner los paréntesis en la llamada a
new Audio();
Te falta el
;
en la línea donde asignas la propiedadsrc
del nuevo audio.La lógica del
if
está incompleta, pues siempre llamas al métodoplay()
y, cuandoclick
tiene valor 2, inmediatamente llamas apause()
(sobre el mismo audio al que acabas de llamar al métodoplay()
. Esto es lo que provoca el error que ves... pero ese error es secundario, como creo que ha quedado claro con esta explicación.