Как я могу улучшить этот класс JavaScript?

Question

Я новичок в JavaScript, и я знаю, что мой код странный, как я могу это исправить? Кроме того, как я могу вызвать функцию внутри функции? Я хочу преобразовать все значения в число или float и вызвать решение.

<script type="text/javascript"> 

    function Math(){       

     var firstNumber = document.getElementById("firstnumber"); 
     var secondNumber = document.getElementById("secondnumber"); 

     this.add = function(){   
      var fN = Number(firstNumber.value); 
      var sN = Number(secondNumber.value); 
      return (fN + sN); 
     } 

     this.sub = function(){   
      var fN = Number(firstNumber.value); 
      var sN = Number(secondNumber.value); 
      return (fN - sN); 
     } 

     this.div = function(){   
      var fN = Number(firstNumber.value); 
      var sN = Number(secondNumber.value); 
      return (fN - sN); 
     } 


     this.mul = function(){   
      var fN = Number(firstNumber.value); 
      var sN = Number(secondNumber.value); 
      return (fN * sN); 
     } 

    } 

    var myNewMath = new Math();   

</script> 

Answer 1

Код не слишком плохо для новичка, если немного повторы.

Чтобы не усложнять сейчас, я бы, вероятно, сделать что-то вроде этого, чтобы избежать повторных вызовов Number(element.value):

function Calculator() { // NB: not "Math" - that's a standard object     

    var firstNumber = document.getElementById("firstnumber"); 
    var secondNumber = document.getElementById("secondnumber"); 

    function op1() { 
     return Number(firstNumber.value); 
    } 

    function op2() { 
     return Number(secondNumber.value); 
    } 

    this.add = function() {   
     return op1() + op2(); 
    } 

    this.sub = function() { 
     return op1() - op2(); 
    } 

    ... 
}