请检查作业，有什么地方需要改进吗？

来源：7-2 编程练习

不会飞的萌物

2019-03-07 20:53:12

<!DOCTYPE html>
<html>

<head>
    <meta charset="utf-8">
    <title>DOM操作</title>
    <style type="text/css">
    html,
    body {
        margin: 0;
        padding: 0;
    }

    div:not(:nth-of-type(2)) {
        width: 500px;
        height: 100%;
        background-color: #00c4ff7a;
        margin: 0 auto;
    }
    /*ul默认有外边距*/

    ul {
        width: 300px;
        height: 100%;
        background-color: #f9c3e6d6;
        list-style: none;
        /*清除默认边距*/
        margin: 0;
        padding: 0;
        margin: 20px auto;
        margin-bottom: 30px;
    }

    li {
        width: 200px;
        height: 30px;
        line-height: 30px;
        margin: 30px auto;
        background-color: #cdffc0;
    }
    </style>
</head>

<body>
    <div id="box">
        <button id="btnAdd">添加元素</button>
        <button id="btnRemove">删除元素</button>
        <ul id="list">我是ul
            <li>我是li1</li>
            <li>我是li2</li>
            <li>我是li3</li>
        </ul>
    </div>
    <script type="text/javascript">

    var box = ById('box'),
        btnAdd = ById('btnAdd'),
        btnRemove = ById('btnRemove'),
        ul = ById('list'),
        li = document.getElementsByTagName('li');
        

    function ById(id){
        return typeof(id) === "string" ? document.getElementById(id) : id;
    }

    (function moveover(){
        for(var i=0;i<li.length;i++){
            li[i].onmouseover = function(){
                this.style.backgroundColor = "skyblue";
            };
            li[i].onmouseout = function(){
                this.style.backgroundColor = "pink";
            };
        }
    })();
    
    (function addItem(){
        btnAdd.onclick = function(){
            var newli = document.createElement('li'),
                txt = document.createTextNode("我是li"+(li.length+1));
            newli.appendChild(txt);
            ul.appendChild(newli);
        };
    })();
    
    (function delItem(){
        btnRemove.onclick = function(){
            if(li.length>2){ ul.removeChild(ul.lastElementChild);}
        };
    })();
    

    </script>
</body>

</html>

写回答

1回答