如何将分钟添加到番茄钟?

How can I add minutes to a pomodoro clock?

我是一名 JS 新手,正在尝试构建一个功能最少的番茄时钟。虽然我有基本的倒计时工作,但我在为计时器实现 'add additional minute' 功能时遇到了困难。

到目前为止,我在变量声明后立即粘贴了这个:

// Add more minutes to timer
add.addEventListener("click", addMin);

function addMin() {
  mins += 1;
  timer.textContent = mins + ":" + "0" + currentSecs;
}

虽然这会在倒计时之前添加分钟数,但不会更新实际的倒计时分钟数。我哪里出错了?非常感谢对这个问题有新的看法(并拯救几只小猫..)

我在下面添加了HTML和JS代码:

var timer = document.getElementById("timer"),
  button = document.getElementById("button"),
  add = document.getElementById("add"),
  minus = document.getElementById("minus"),
  numOfPomodoros = 0;

// Variables for the Countdown to Work:
var mins = 25,
  secs = mins * 60,
  currentSecs = 0,
  currentMins = 0,
  comingOffBreak = false;


button.addEventListener("click", startCountdown);

function startCountdown() {
  setTimeout(countdown, 500);
}

function countdown() {

  currentMins = Math.floor(secs / 60);
  currentSecs = secs % 60;

  if (currentSecs <= 9) {
    // Add an extra zero to the seconds in order to make it look more like a clock
    currentSecs = "0" + currentSecs;
  }

  secs--;
  
  timer.textContent = currentMins + ":" + currentSecs;
  
  if (secs !== -1) {
    // Continue the countdown
    setTimeout(countdown, 1000);
  } else {
    // When countdown reaches 0...
    

    if (!comingOffBreak) {
      // If user just finished a pomodoro...
      countNumOfPomodoros();

      if (numOfPomodoros % 4 === 0) {
        button.textContent = "Start Long Break";
        resetForLongBreak();
      } else {
        button.textContent = "Start Short Break";
        resetForShortBreak();
      };
    } else {
      // If user just finished a break...
      button.textContent = "Start Pomodoro";
      resetCountdown();
    };
  };
}


function countNumOfPomodoros() {
  // Add 1 to numOfPomodoros after each pomodoro is completed
  numOfPomodoros++;
  return numOfPomodoros;
}

function resetCountdown() {
  // For new pomodoro
  mins = 25;
  secs = mins * 60;
  currentSecs = 0;
  currentMins = 0;
  comingOffBreak = false;
}

function resetForShortBreak() {
  mins = 5;
  secs = mins * 60;
  currentSecs = 0;
  currentMins = 0;
  comingOffBreak = true;
}

function resetForLongBreak() {
  mins = 30;
  secs = mins * 60;
  currentSecs = 0;
  currentMins = 0;
  comingOffBreak = true;
}
<div class="container text-center">
  <h1 id="title">POMODORO CLOCK</h1>
  <div id="timerBox">
    <span id="timer">25:00</span>
  </div>
  <br>
  <button id="minus" class="btn btn-danger">- Minute </button>
  <button id="button" class="btn btn-success">Start Pomodoro</button>
  <button id="add" class="btn btn-primary">+  Minute</button>
</div>

您正在更新方法和文本中的 mins 变量,而不是倒计时方法中考虑的变量秒数。

我也会考虑重构如下:

  • 有一个番茄钟开始的变量,
  • 有另一个增加分钟数的变量,
  • 然后使用以下 pseudo-expression 获取要显示的剩余秒数,而不用倒计时更新变量:25 * 60 + extraMins * 60 - <current-time-in-secs> + <start-time-in-secs>