6

我想做的事情非常简单：

  • 如果输入是0，这意味着他们没有输入数字，它应该告诉你。
  • 当输入为7时，它应该说你做对了。
  • 其他任何事情，它都应该告诉你你弄错了。

但无论输入是什么，它都只会输出“7 是正确的”行，我无法弄清楚哪里出了问题。

<script type="text/javascript">
function problem2 ()
{
var number = 0;
var text=document.getElementById("output");
number = prompt("Enter a number between 1 and 10 please" , 0);
if (number = 0)
    {
     text.value = "You didn't enter a number!";
    }
if (number = 7)
    {
     text.value = "7 is correct!";
    }
else
    {
     text.value = "Sorry, ", input, "is not correct!";
    }
}
</script>
<input type="button" value="Click here" onclick="problem2()">
<input id="output" type="text">
4

3 回答 3

6

您正在分配=. 使用==或===。

if( 0 == number ){

  text.value = "You didn't enter a number!";
}

另外，请注意您的支架放置。Javascript 喜欢在行尾自动添加分号。来源。

于 2012-12-13T01:09:50.083 回答
3

您正在使用赋值运算符作为条件而不是比较运算符：

if (number = 0) // falsy. Same as if (false)
    {
     text.value = "You didn't enter a number!";
    }
if (number = 7) // truthy. Same as if (true)
    {
     text.value = "7 is correct!";
    }
else
    {
     text.value = "Sorry, ", input, "is not correct!";
    }

或者，您可以使用开关并更轻松地组织条件：

switch (number) {
    case 0: 
        text.value = "You didn't enter a number!";
        break;

    case 7:
        text.value = "7 is correct!";
        break;

    default:
        text.value = "Sorry, ", input, "is not correct!";
        break;
}
于 2012-12-13T01:12:15.627 回答
3

这是包含一些修复和改进的代码（我评论了我所做的更改）：

function problem2 (){
    //I multiplied by * 1 to work with numbers, also used || to default to 0 in case of NaN
    var num = (prompt("Enter a number between 1 and 10 please" , 0) * 1) || 0;
    var msg = "";

    if (!num){ //I prefer this over 'num == 0'
         msg = "You didn't enter a number!";
    //you should use 'else if' in this case
    }else if (num == 7){//'=' is for assignment, use '==' or '===' instead
         msg = "7 is correct!";
    }else{
        //you had an undefined var 'input', you probably meant 'num'
        //you also were connecting var and strings using commas, use '+' instead
         msg = "Sorry, " + num + " is not correct!"; //added a space in ' is'
    }

    //no need to store the element in a var anymore :D
    document.getElementById("output").value = msg;
}

此外，还可以进行另外两项更改：

  • 只有一个var（例如var something = "", somethingElse = 99;）
  • 从头开始分配默认文本，喜欢var msg = "default"并删除else

注意：我所做的一个未记录的更改是重命名了一些 var，我鼓励大家停止使用 var number, text, string，如果你有这个坏习惯，你最终会错误地使用非法的 var 名称。

于 2012-12-13T01:19:35.597 回答