Age calculator with CSS HTML and Javascript

Community feedback

• 

  HelloTechTinu•670

  @tinuolaPosted over 1 year ago

  @Erick, Good job with the functionality.

  A recommendation to make the JavaScript simpler (or more functional) is to consider breaking up the three logical checks/operations into separate functions that are then called by the event listener. Currently, all that logic inside the event listener can be confusing to read.

  Also, the logic used to validate the required inputs (day, month, year) is the same; therefore it can be refactored into a single function. Consider giving the inputs a shared class and use a forEach loop to add an event listener that calls the function when the fields get a value.

  You might be able to use a similar approach with other areas of the code that are running repetitive logic.

  Marked as helpful

  1

  ---

  

  Erick Melchor•220

  @Melchor16Posted over 1 year ago

  Hey @tinuola , thank you for the feedback! you're right mi logic is repetitive and I should break it up in functions, I'm going to have your recommendations in mind for the next challenge.

  1