# JavaScript Calculator Project: Functionality feedback

**URL:** <https://forum.freecodecamp.org/t/javascript-calculator-project-functionality-feedback/133708>\
**Category:** I Built This\
**Tags:** project, calculator, javascript\
**Created:** [July 3, 2017, 8:30am UTC](https://forum.freecodecamp.org/t/javascript-calculator-project-functionality-feedback/133708 "2017-07-03T08:30:44Z")\
**Posts on this page:** 4\
**Page:** 1

<div class="post-metadata">

**Author:** ![joshamore](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/joshamore/32/50035_2.png) [@joshamore](https://forum.freecodecamp.org/u/joshamore)\
**Post date:** [July 3, 2017, 8:30am UTC](https://forum.freecodecamp.org/t/javascript-calculator-project-functionality-feedback/133708/1 "2017-07-03T08:30:44Z")

</div>

Hi all,

My JavaScript calculator is working… _I think_.

From what I can tell, it seems to meet all the required user stories and doesn’t throwing any weird bugs.

I was hoping to get another set of eyes to look it over and tell me if the functionality seems to be correct (any bug reports would be _greatly_ appreciated!).

**Here’s the live demos:** [https://joshamore.github.io/javascript\_calculator/](https://joshamore.github.io/javascript_calculator/)

**The code is here:** [https://github.com/joshamore/javascript\_calculator](https://github.com/joshamore/javascript_calculator)

No need to comment on the asthetics. I haven’t touched styling past what I needed to test the functionality (next mission).

---

<div class="post-metadata">

**Author:** ![30hjones](https://avatars.discourse-cdn.com/v4/letter/3/a587f6/32.png) [@30hjones](https://forum.freecodecamp.org/u/30hjones)\
**Post date:** [July 3, 2017, 2:41pm UTC](https://forum.freecodecamp.org/t/javascript-calculator-project-functionality-feedback/133708/2 "2017-07-03T14:41:37Z")

</div>

I think I’ve found a bug 🙂

Try this sequence: ‘1’, ‘=’, ‘+’, ‘1’, ‘=’. The problem seems to be `model.toEvaluate` being `null` [here](https://github.com/joshamore/javascript_calculator/blob/f731903e05bb9e7f29da5e44a236d34a3f4d5713/scripts.js#L59).

---

<div class="post-metadata">

**Author:** ![joshamore](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/joshamore/32/50035_2.png) [@joshamore](https://forum.freecodecamp.org/u/joshamore)\
**Post date:** [July 4, 2017, 8:02am UTC](https://forum.freecodecamp.org/t/javascript-calculator-project-functionality-feedback/133708/3 "2017-07-04T08:02:52Z")

</div>

Awesome, thanks!

I fixed the issue by adding an `if` statement that makes sure two numbers are present.

---

<div class="post-metadata">

**Author:** ![ILM](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/ilm/32/363239_2.png) [@ILM](https://forum.freecodecamp.org/u/ILM)\
**Post date:** [June 1, 2021, 2:32pm UTC](https://forum.freecodecamp.org/t/javascript-calculator-project-functionality-feedback/133708/4 "2021-06-01T14:32:24Z")

</div>

Closed on June 1, 2021, 2:32pm UTC