0

我不确定我的 while 循环或代码的 innerHTML 部分是否有问题，但是当单击提交按钮时，我无法在 div 标签中显示下拉列表。任何人都可以看到它有什么问题。

<html>
<head>
<script type="text/javascript">

function getvalue() {
 number = document.getnumber.input.value;
 document.getElementById("result").value = number; 
} 
</script>

</head>
<body>

<script>
function generatedropdown() {
html = '<select name="select" id="i">';
while (i < number)  {            
html+='<option>Test 1</option>';
html+='<option>Test 2</option>';
html+='<option>Test 3</option>';
html+='<option>Test 4</option>';
html+='<option>Test 5</option>';        
i++;
}
html+='</select>';
document.getElementById("my-output").innerHTML = html;
}
</script>


<form name="getnumber">
Input number: <input type="text" name="input">
<input type="button" value="Next" onClick="getvalue()">
</form>


<form id="showlists">
Number entered: <input type="text" id="result" readonly="readonly">     
<input type="button" value="Show lists" onClick="generatedropdown()">
<div id="my-output">Generated List:</div>
</form>
</body>
</html>
4

1 回答 1

5

几个问题：

  • 您从未为 设置初始值i，因此代码将引发错误，因为您正在尝试读取从未设置或声明的全局值。

  • 你依赖于getvalue被调用来初始化number，我不会指望它。

  • 您依赖于隐式字符串-> 数字转换，我不建议这样做；用于parseInt解析用户提供的数字。

  • （可选）您的循环正是该for构造的设计目的，而不是while（尽管while如果您初始化了它会起作用i）。

  • 你正在成为隐式全局恐怖的牺牲品，因为你从来没有声明你的变量。

我建议阅读关于 JavaScript 的良好入门或教程，以掌握基础知识。

这是一个最小的更新：

function generatedropdown() {
    // Declare your variables
    var html, i, number;

    // Get the number, and convert it from a decimal string
    // to a number explicitly rather than relying on implicit
    // coercion
    number = parseInt(document.getvalue.input.value, 10);

    // Probably bail if it's not a number
    if (isNaN(number)) {
        return;
    }

    // (Optional, but you were doing it) Show the number
    document.getElementById("result").value = number;

    // Build your HTML
    html = '<select name="select" id="i">';

    // This is exactly what `for` loops are for
    for (i = 0; i < number; ++i) {
        html+='<option>Test 1</option>';
        html+='<option>Test 2</option>';
        html+='<option>Test 3</option>';
        html+='<option>Test 4</option>';
        html+='<option>Test 5</option>';        
    }
    html+='</select>';
    document.getElementById("my-output").innerHTML = html;
}
于 2013-02-17T10:45:22.923 回答