老师请帮忙看看哪里能优化
来源：11-2 编程练习
qq_Mr郭生_0
2020-05-10 16:43:06
<!DOCTYPE html>
<html>
<head>
<title></title>
<style type="text/css">
html,
body {
margin: 0;
padding: 0;
}
div:not(:nth-of-type(2)) {
width: 500px;
height: 100%;
background-color: #a0e4ff;
margin: 0 auto;
}
/*ul默认有外边距*/
ul {
width: 300px;
height: 100%;
background-color: #ecc7ea;
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 li=document.querySelectorAll("li");
for(var i=0;i<li.length;i++){
li[i].onmousemove=function(){
this.style.backgroundColor="blue";
}
li[i].onmouseout=function(){
this.style.backgroundColor="purple";
}
}
var ul=document.getElementById("list")
var btnAdd=document.getElementById("btnAdd");
var btnRemove=document.getElementById("btnRemove");
var txt=document.createTextNode("我是li")
var li=document.createElement("li");
var i=0;
btnAdd.onclick=function(){
txt=document.createTextNode("我是li"+(4+i))
li=document.createElement("li");
li.appendChild(txt);
ul.appendChild(li);
i++;
}
btnRemove.onclick=function(){
ul.removeChild(ul.lastChild)
}
</script>
</body>
</html>
1回答
-
同学你好，可以优化的地方如下：
1、每次点击两下，才可以删除一个元素，这是因为使用的是childNodes获取所有子节点（包括包括文本节点、注释节点即回车、换行、空格、文本等等），这里会先删除空的文本节点。所以每次都点击两下才可以删除一个元素。
建议：可以使用lastElementChild， 返回最后一个子节点（元素节点） ，例：
2、当点击删除元素全部把li元素删除之后，继续点击按钮，会出现报错，如下：
是因为没有子元素了，继续移除就会出错，可以提前添加判断，如下：
3、如下，两个变量都叫li，使用到后面使用上面的li的时候，li已经被替换了，不能使用最开始的。
建议：可以换个变量名，例：
4、当把开始3个li元素删除之后，点击添加按钮添加li元素，鼠标移入前三个的li的时候没有改变背景颜色。且内容也不对，如下：
建议：可以把改变颜色的代码封装起来，对长度进行判断，然后调用，参考：
之前的可以注释掉，例：
5、注意：使用querySelectorAll获取的li是最开始的，添加之后，或者删除之后的获取不到，这样获取长度的时候会有误，建议：可以使用标签名获取，例：
如果的回答帮助了你，欢迎采纳，祝学习愉快~
0
相似问题