Question

创建了一个基本表单，三个下拉列表和一个按钮。我试着在下面的列表中乘以三个值。我必须将数据传递给函数。我已经运行代码只是继续得到0作为结果？我不认为我正在将数据从正确传递给函数吗？

<!DOCTYPE HTML>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<title>Untitled Document</title>
</head>

<body>
<script>
function multiply(number1, number2, number3)
{
    var firstnum=0;
    var secondnum=0;
    var thirdnum=0;
    var answer;
    firstnum=parseInt(number1);
    secondnum=parseInt(number2);
    thirdnum==parseInt(number3);
    answer=(firstnum*secondnum*thirdnum);
    alert(answer)
}

</script>

<form name="Example1">
Number 1:
<select id="num1">
<option value="1">1</option>
<option value="2">2</option>
<option value="3">3</option>
<option value="4">4</option>
<option value="5">5</option>
</select>
<br/>
Number 2:
<select id="num2">
<option value="1">1</option>
<option value="2">2</option>
<option value="3">3</option>
<option value="4">4</option>
<option value="5">5</option>
</select>
<br/>
Number 2:
<select id="num3">
<option value="1">1</option>
<option value="2">2</option>
<option value="3">3</option>
<option value="4">4</option>
<option value="5">5</option>
</select>
<br/>
<input type=button value="muliply" onclick="multiply(num1.value, num2.value, num3.value)" />
</form>
</body>
</html>

Answer 1

你的问题是：

thirdnum==parseInt(number3);是比较，而不是作业，因此thirdnum始终为0。

由于你乘以thirdnum，而任何×0都是0，你总是得到0作为输出。

将==更改为=。

如果您使用JS Hint对代码进行了质量检查，则会收到此信息。

您应该修复的不良做法是：

num1.value假设将为每个具有id的元素创建全局JS变量。不要假设，使用document.getElementById
parseInt(number1)没有基数。始终指定您正在使用的数字系统（通常为10），因此不会从数据中推断出意外结果：parseInt(number1, 10);
<form name="Example1">，表单上的name属性是id属性正确出现之前的遗留。使用id标识客户端代码的元素。
onclick属性在separating concerns中表现不佳。如果需要支持旧版本的IE，请使用addEventListener（或者像YUI或jQuery这样的库）。

Answer 2

好的，你因为这个而继续得到0

thirdnum==parseInt(number3);

在您的multiply函数

中

将其更改为

thirdnum=parseInt(number3);

你应该好好去

这是demo

javascript函数参数

2 个答案: