                        课程
                    
                        /前端开发
                        
                            /JavaScript
                        
                        /JavaScript进阶篇

找不到代码错误

<!DOCTYPE html>

<html>

<head>

function count(){

var otxt1=document.getElementById("txt1").value;

var otxt2=document.getElementById("txt2").value;

var osel=document.getElementById("select").value;

var result="";

switch(osel){

case"+":

result=parseInt(otxt1) + parseInt(otxt2);

break;

case"-":

result=parseInt(otxt1) - parseInt(otxt2);

break;

case"*":

result=parseInt(otxt1) * parseInt(otxt2);

break;

default:

result=parseInt(otxt1) / parseInt(otxt2);

}

document.getElementById("fruit")=result;

}

</script>

</head>

<body>

</select>

</body>

</html>

慕慕1265030

2016-08-11

源自：JavaScript进阶篇 6-11

关注问题我要回答

741

操作

收起

4 回答

慕粉3512312
2016-08-11

document.getElementById("fruit").value=result; 少个value

0 回复有任何疑惑可以回复我~

收起回答

繁华若梦
2016-08-11

document.getElementById("fruit")=result; 这样的方式是错误的，这行代码的意思是把result的值赋给Id为fruit的这个对象，用变量给对象赋值肯定是错误的。这里要改成document.getElementById("fruit").value=result; 这样才表示的是给Id为fruit的对象的value属性赋值。

另外再给你一点建议：

var otxt1=document.getElementById("txt1").value;

var otxt2=document.getElementById("txt2").value;

这两个地方，并没有错，但是可以进行一些优化会让你的代码更简洁，修改为

var otxt1=parseInt(document.getElementById("txt1").value);

var otxt2=parseInt(document.getElementById("txt2").value);

这样，你拿到这两个值的时候已经转换为整数了，可以直接进行运算。当然也可以写成这样：

var otxt1=document.getElementById("txt1").value;

var otxt2=document.getElementById("txt2").value;

otxt1=parseInt(otxt1);

otxt2=parseInt(otxt2);

这样也会直接将值转换为整数，直接进行运算，不用在每次运算的时候都去转换一次。