Refactor quote selection logic into a reusable function and update event listeners for better readability

Author: alexandru-pocovnicu

Sprint-3/quote-generator/quotes.js

@@ -492,43 +492,28 @@ const quotes = [
 
 // call pickFromArray with the quotes array to check you get a random quote
 
-const randomQuote = quotes[Math.floor(Math.random()*quotes.length)];
-
-const quote = document.getElementById("quote");
-quote.innerText = randomQuote.quote;
-
-const author = document.getElementById("author");
-author.innerText = randomQuote.author;
-
-const button = document.getElementById("new-quote");
-button.addEventListener("click", () => {
-  const randomQuote =quotes[Math.floor(Math.random()*quotes.length)];
+function chooseQuote() {
+  const randomQuote = quotes[Math.floor(Math.random() * quotes.length)];
 
   const quote = document.getElementById("quote");
   quote.innerText = randomQuote.quote;
 
   const author = document.getElementById("author");
   author.innerText = randomQuote.author;
-});
+}
 
-function chooseQuote(){
-  const randomQuote = quotes[Math.floor(Math.random() * quotes.length)];
+window.addEventListener("load", chooseQuote);
 
-  const quote = document.getElementById("quote");
-  quote.innerText = randomQuote.quote;
+const button = document.getElementById("new-quote");
+button.addEventListener("click", chooseQuote);
 
-  const author = document.getElementById("author");
-  author.innerText = randomQuote.author;
-}
 const autoGenerate = document.getElementById("auto-play-toggle");
 let interval = null;
-autoGenerate.addEventListener("change",()=>{
-  
-  if(autoGenerate.checked){
-    interval=setInterval(chooseQuote,2000)
-  }else{
-    clearInterval(interval)
-    interval=null
+autoGenerate.addEventListener("change", () => {
+  if (autoGenerate.checked) {
+    interval = setInterval(chooseQuote, 2000);
+  } else {
+    clearInterval(interval);
+    interval = null;
   }
-})
-
+});