Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

JavaScript

Posted by Dante Ortiz
Dante Ortiz
3,844 Points

I am building a small program to calculate my bills using buttons. How can I simplify this code?

This is my code: https://w.trhou.se/b7qzw8ddca

1 Answer

Steven Parker
Steven Parker
243,134 Points
Steven Parker
Steven Parker
243,134 Points

A few ideas:

  • assign "value" attributes to each button
  • set the value of each button to the bill cost in the HTML
  • remove cost variables from JS code
  • use a single delegated handler for all the buttons
  • remove individual button variables from JS code
  • use the event target to change color and get the associated value