Update script.js

[ussingdi]

No description provided.

[ai-coding-app]

1. You can simplify the addition of the current score to the active player's score by using the compound assignment operator:

scores[activePlayer] += currentScore;

2. In the if statement, you can remove the else clause and move the switchPlayer() function call outside of it, since it will always be executed when the if condition is not met:

if (scores[activePlayer] >= 100) {
  // Finish the game
  playing = false;
  diceEl.classList.add('hidden');

  document
    .querySelector(`.player--${activePlayer}`)
    .classList.add('player--winner');
    .classList.remove('player--active');

  switchPlayer();
}

if (playing) {
  // Roll the dice
  ...
}

3. You can chain the add and remove classList methods to reduce the number of method calls:

document
  .querySelector(`.player--${activePlayer}`)
  .classList.add('player--winner')
  .classList.remove('player--active');

4. You can use the textContent property to update the score element, which is more efficient than using innerText:

document.getElementById(`score--${activePlayer}`).textContent = scores[activePlayer];