inconsistencies addressed: math.max removed, and alarm sound edge case addressed

Author: zilinskyte

Sprint-3/alarmclock/alarmclock.js

@@ -18,15 +18,15 @@ function updateDisplay() {
   const heading = document.getElementById("timeRemaining");
   const input = document.getElementById("alarmSet");
 
-  totalSeconds = Math.max(0, totalSeconds);
   const formatted = formatTime(totalSeconds);
 
   input.value = formatted;
   heading.innerHTML = `Time Remaining:<br><br>${formatted}`;
 }
 
 function incrementTime(amount) {
-  totalSeconds = Math.max(0, totalSeconds + amount);
+  totalSeconds += amount;
+  if (totalSeconds < 0) totalSeconds = 0;
   updateDisplay();
 }
 
@@ -51,16 +51,22 @@ function setAlarm() {
   totalSeconds = parsed;
   updateDisplay();
 
-  timer = setInterval(() => {
-    totalSeconds--;
-    totalSeconds = Math.max(0, totalSeconds);
-    updateDisplay();
+   if (totalSeconds === 0) {
+    playAlarm();
+    return;
+  } //Alarm plays immediately if time is set to 00:00
 
-    if (totalSeconds === 0) {
+  timer = setInterval(() => {
+    totalSeconds -= 1;
+       if (totalSeconds <= 0) {
+      totalSeconds = 0;
+      updateDisplay();
       clearInterval(timer);
       timer = null;
       playAlarm();
+      return;
     }
+    updateDisplay();
   }, 1000);
 }