Multi-step-Form

Community feedback

• 

  Khaya•300

  @khaya05Posted over 1 year ago

  Hey Timothy

  Here are a few things that you can do to improve your code.

  Firstly, use variables when selecting your DOM elements, preferably at the top of your file.

  Instead of this:

  function goStepThree() {
  console.log(typeof document.getElementById("totalPrice").innerHTML);
  
  if (document.getElementById("totalPrice").innerHTML == "") {
  console.log(document.getElementById("totalPrice").innerHTML);
  document.getElementById("totalPrice").innerHTML = "0";
  console.log(document.getElementById("totalPrice").innerHTML);
  }
  checkPlan();
  }
  

  Use variables:

  const totalPrice = document.getElementById("totalPrice")
  
  function goStepThree() {
  if (totalPrice.innerHTML == "") {
  console.log(totalPrice.innerHTML);
  totalPrice.innerHTML = "0";
  }
  checkPlan();
  }
  
  

  Also, use const or let when declaring variables

  0

  ---

  

  Timothy Ezenwagu•180

  @tdmoreePosted over 1 year ago

  @khaya05 thanks

  0