adding new random number after a link on every press

Question

the goal of my code is to generate a new link each time "generate" is pressed, so that i can find new songs for geometry dash without having to spam numbers.

i have tried the string below but every time the button were to be clicked, it would append a new set of numbers on the end. i wish for the program to not require a refresh

document.querySelectorAll("a").forEach(link =>
     link.setAttribute("href", link.getAttribute("href") + Math.floor(Math.random() * 111392.8) + 1)

this is my current attempt. i have gotten the goal of a new number on every button press with a refresh but the prefix of "https://www.newgrounds.com/audio/listen/" won't append to the start.

window.onclick = () => {
  document.querySelectorAll("a").forEach(link =>
    link.setAttribute("href", Math.floor(Math.random() * 111392.8) + 1)
  );
};


  Newgrounds Song Generator




  Click the button to visit a random song!

user_295783 · Answer

Don't have a button in a link!

The code can be much simpler with a form

const url = "https://www.newgrounds.com/audio/listen/";
document.getElementById("generate").addEventListener("submit", function() {
  const link = url + Math.floor(Math.random() * 111392.8) + 1;
  console.log(link)
  this.action = link; // SO does not allow a target="_blank" in a snippet
})


  Newgrounds Song Generator

Click the button to visit a random song!

Or a link styled as a button

const url = "https://www.newgrounds.com/audio/listen/";
document.getElementById("generate").addEventListener("click", function() {
  const link = url + Math.floor(Math.random() * 111392.8) + 1;
  console.log(link)
  this.href = link; // SO does not allow a target="_blank" in a snippet
})

.button {
  font: bold 11px Arial;
  text-decoration: none;
  background-color: #EEEEEE;
  color: #333333;
  padding: 2px 6px 2px 6px;
  border-top: 1px solid #CCCCCC;
  border-right: 1px solid #333333;
  border-bottom: 1px solid #333333;
  border-left: 1px solid #CCCCCC;
}


  Newgrounds Song Generator

Click the button to visit a random song!
Generate

user_14145056 · Answer

I'd recommend storing the rest of the domain in a separate variable, like so


 

 







Newgrounds Song Generator
 



Click the button to visit a random song!


Generate

Hopefully that helps!

user_12597155 · Answer

You have some problems with it:

You shouldn't use script tag inside the head tag.
You shouldn't use button tag inside an anchor tag.
You shouldn't use window.onclick because it will activate the function no matter where you click.





  



  Newgrounds Song Generator




  Click the button to visit a random song!

adding new random number after a link on every press

3 Answers