1

我正在使用纯 JavaScript 和 HTML 制作颜色选择器。它由三个 HTML 选择（下拉框）和一个 div 组成，其背景颜色将由 JavaScript 更改。我也试图尽可能“正确”地做到这一点。这意味着 HTML 中没有 Javascript 代码。

到目前为止，我的代码如下所示：

    var red = document.getElementById('red');
    red.onchange = update();

    var green = document.getElementById('green');
    green.onchange = update();

    var blue = document.getElementById('blue');
    blue.onchange = update();

    var thebox = document.getElementById('colourbox');

    function d2h(d) {return d.toString(16);}
    function h2d(h) {return parseInt(h,16);} 

    function update(){
        finalcolor = '#' + d2h(red.value) + d2h(green.value) + d2h(blue.value)
        thebox.style.background = finalcolour;
    }

HTML 看起来像这样：

<div id="colourbox"></div>
<form name="myform" action="colour.html">
    <select name="red" id="red">
        <option value="0">0</option>
        .... etc etc ....
    </select>
    <select name="green" id="red">
        <option value="0">0</option>
        .... etc etc ....
    </select>
    <select name="blue" id="red">
        <option value="0">0</option>
        .... etc etc ....
    </select>
</form>

问题是所有 document.getElementById() 调用都返回 null。大概是因为它们在代码运行时不存在。我尝试将代码放在 window.onload = function() {} 中，但是 a) 会让人感到困惑，b) 然后我必须在函数中定义更新函数，这似乎不正确。

任何人都可以对此有所了解吗？是否有任何一般规则可以帮助我理解它是如何工作的？或有关该主题的一些文档？

编辑：修改后的代码：

<script type="text/javascript">
    window.onload = function(){
        var red = document.getElementById('red');
        red.onchange = update();

        var green = document.getElementById('green');
        green.onchange = update();

        var blue = document.getElementById('blue');
        blue.onchange = update();

        var thebox = document.getElementById('colourbox');

        function d2h(d) {return d.toString(16);}
        function h2d(h) {return parseInt(h,16);} 

        function update(){
            var finalcolor = '#' + d2h(document.getElementById('red').value) + d2h(document.getElementById('green').value) + d2h(document.getElementById('blue').value);

            document.getElementById('colourbox').style.background = finalcolor;
        }

    }
</script>
4

2 回答 2

3

将代码放在 onload 事件的处理程序中效果很好，并且被广泛接受。将函数放入函数中也是如此（取决于原因）。

var mypage = {
    red: null,
    green: null,
    blue: null,
    colourbox: null,

    d2h: function(d) {
        var hex = d.toString(16);
        if (hex.length < 2) {
            hex = '0' + hex;
        }
        return hex.toUpperCase();
    },

    h2d: function(d) {
        return parseInt(h, 16);
    },

    update: function() {
        mypage.colourbox.style.backgroundColor = '#' + mypage.d2h(mypage.red.value) + mypage.d2h(mypage.green.value) + mypage.d2h(mypage.blue.value);
    }
};

window.onload = function() {
    mypage.red = document.getElementById('red');
    mypage.red.onchange = mypage.update;

    mypage.green = document.getElementById('green');
    mypage.green.onchange = mypage.update;

    mypage.blue = document.getElementById('blue');
    mypage.blue.onchange = mypage.update;

    mypage.colourbox = document.getElementById('colourbox');
}

这是edwards 院长关于使用 window.onload的博客文章

另一种选择是将您的 javascript 放在页面底部而不是顶部。

于 2010-01-15T16:58:33.937 回答
2

正如其他人所说，您需要将代码包装在一个函数中并从 window.onload 事件中调用它。

此外，当您将函数分配给事件时，仅使用名称（无括号）或函数在此处被调用，然后 ..

所以

red.onchange = update();

应该

red.onchange = update;
于 2010-01-15T17:09:00.993 回答