use const instead let in the updateDisplay function \n decrement the remainSecond before calling updateDisplay \n handle zero and negative-input

Author: AhmadHmedann

Sprint-3/alarmclock/alarmclock.js

@@ -2,16 +2,16 @@ let timerId = null;
 let remainingSeconds = 0;
 
 function updateDisplay() {
-  let minutes = String(Math.floor(remainingSeconds / 60)).padStart(2, "0");
-  let seconds = String(remainingSeconds % 60).padStart(2, "0");
+  const minutes = String(Math.floor(remainingSeconds / 60)).padStart(2, "0");
+  const seconds = String(remainingSeconds % 60).padStart(2, "0");
   document.getElementById(
     "timeRemaining"
   ).textContent = `Time Remaining: ${minutes}:${seconds}`;
 }
 
 function startCountdown() {
-  if (remainingSeconds < 0) remainingSeconds = 0;
   timerId = setInterval(() => {
+    remainingSeconds--;
     updateDisplay();
 
     if (remainingSeconds <= 0) {
@@ -20,20 +20,24 @@ function startCountdown() {
       timerId = null;
       playAlarm();
     }
-    remainingSeconds -= 1;
   }, 1000);
 }
 
 function setAlarm() {
   const input = document.getElementById("alarmSet");
   if (input.value === "") return;
-  const inputValue = Number(input.value);
+  remainingSeconds = Number(input.value);
   clearInterval(timerId);
   timerId = null;
-  remainingSeconds = inputValue;
 
-  updateDisplay();
-  startCountdown();
+  if (remainingSeconds <= 0) {
+    remainingSeconds = 0;
+    updateDisplay();
+    playAlarm();
+  } else {
+    updateDisplay();
+    startCountdown();
+  }
   input.value = "";
 }