fix: correct timer logic and prevent multiple intervals

Author: Tarawally

Sprint-3/alarmclock/alarmclock.js

@@ -18,6 +18,13 @@ function formatTime(totalSeconds) {
   return `Time Remaining: ${formattedMinutes}:${formattedSeconds}`;
 }
 
+/**
+ * Displays the remaining time.
+ */
+function displayTime() {
+  titleElement.innerText = formatTime(timeRemainingInSeconds);
+}
+
 /**
  * Updates the display and checks if the alarm should sound.
  */
@@ -26,23 +33,11 @@ function updateTime() {
   displayTime();
 
   if (timeRemainingInSeconds === 0) {
-    titleElement.innerText = formatTime(0);
     playAlarm();
     document.body.classList.add("flash");
-    return;
+    clearInterval(alarmTimerIdentifier);
+    alarmTimerIdentifier = null;
   }
-
-  alarmTimerIdentifier = setInterval(() => {
-    updateTime();
-  }, ONE_SECOND_IN_MILLISECONDS);
-}
-
-/**
- * Displays the remaining time.
- */
-function displayTime() {
-  const titleElement = document.getElementById("timeRemaining");
-  titleElement.innerText = formatTime(timeRemainingInSeconds);
 }
 
 /**
@@ -73,6 +68,13 @@ function setAlarm() {
 
   displayTime();
 
+  // Handle zero-second edge case
+  if (timeRemainingInSeconds === 0) {
+    playAlarm();
+    document.body.classList.add("flash");
+    return;
+  }
+
   alarmTimerIdentifier = setInterval(() => {
     updateTime();
   }, ONE_SECOND_IN_MILLISECONDS);