js制作简易计算器我怎样优化这个代码，写了很多重复的了

Question

<table > <caption>简易计算器</caption> <tr> <td>操作数1:</td> <td colspan="4"><input type="text"id="num1"></td> </tr> <tr> <td >操作数2:</td> <td colspan="4"><input type="text"id="num2"/> </td> </tr> <tr> <td></td> <td><input type="button"id="Add"value="+" onclick="Add();"/></td> <td><input type="button"id="Sub"value="-" onclick="Sub();"/></td> <td><input type="button"id="Rid"value="*" onclick="Rid();"/></td> <td><input type="button"id="Div"value="/" onclick="Div();"/></td> </tr> <tr> <td>结果:</td> <td colspan="4"><input type="text"id="result"/></td> </tr> </table>

Answer 1

将html改一些：

<td><input type="button"id="Add"value="+" onclick="myClickOperator(1);"/></td>

<td><input type="button"id="Sub"value="-" onclick="myClickOperator(2);"/></td>

<td><input type="button"id="Rid"value="*" onclick="myClickOperator(3);"/></td>

<td><input type="button"id="Div"value="/" onclick="myClickOperator(4);"/></td>

js：

/**
* 操作两个值
* 传入两个值，得到相应运算符的值
*/
function OperatorTwoNum(num1,num2,operator)
{
num1 = parseFloat(num1);
num2 = parseFloat(num2);
switch(operator)
{
case "+":
return num1 + num2;
case "-":
return num1 - num2;
case "*":
return num1 * num2;
case "/":
return num1 / num2;
default:
return null;
}
}



/**
 * 点击的操作
 * 1:加法
 * 2:减法
 * 3:乘法
 * 4:除法
 */
function myClickOperator(index)
{
var num1 = document.getElementById("num1").value;
var num2 = document.getElementById("num2").value;
var result = "";
if(isNaN(num1) || isNaN(num2))
{
alert("请输入数字");
return;
}
switch(index)
{
case 1:
result = OperatorTwoNum(num1,num2,"+");
break;
case 2:
result = OperatorTwoNum(num1,num2,"-");
break;
case 3:
result = OperatorTwoNum(num1,num2,"*");
break;
case 4:
result = OperatorTwoNum(num1,num2,"/");
break;
default:
alert("请输入正确运算符");
}
if(isNaN(result))
{
document.getElementById("result").value=result;
}
}

追问

你这个更复杂了，要是有加减乘除另外的你的case不断增加，script case又写一遍那么多

追答

sorry，我一般很少用eval方法，除了格式json串之外，恩，清楚了一招，ok，thank