Rock Paper Scissors Lizard Spock

Community feedback

• 

  Fazza Razaq Amiarso•2,340

  @fazzaamiarsoPosted over 2 years ago

  Hello Wissem! Nice solution!

  I have some improvements.

  • You can refactor the restart logic to a function as both winning and losing do the same thing.

  function restart() {
  // your restart logic
  }
  document.getElementById("Restart-winning").addEventListener("click", restart);
  document.getElementById("Restart-losing").addEventListener("click", restart);
  

  • You should prefer to use let and const than var. By using var means your variable is always global and not scoped.

  I hope it helps! Cheers!

  0