老师麻烦检查一下代码
来源：11-2 编程练习
林迦叶
2019-09-26 13:33:46
<!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 ul=document.getElementById("list");
var lis=ul.querySelectorAll("li");
var li;
for(var i=0;i<lis.length;i++){
li=lis[i];
li.onmouseover=function(){
this.style.backgroundColor="#90f";
};
li.onmouseout=function(){
this.style.backgroundColor="#f00";
};
}
var btn1=document.getElementById("btnAdd");
var btn2=document.getElementById("btnRemove");
btn1.onclick=function(){
var text=document.createTextNode("我是li"+(ul.childElementCount+1));
var newLi=document.createElement("li");
newLi.appendChild(text);
ul.appendChild(newLi);
}
btn2.onclick=function(){
ul.removeChild(ul.lastChild);
}
</script>
</body>
</html>写回答
1回答
-
同学你好，1、点击删除的按钮，需要点击两下才可以删除元素。
因为lastChild 属性可返回文档的最后一个子节点（但是包含文本节点），所以第一次删除的是文本节点。建议：可以使用lastElementChild元素节点，直接删除元素节点。例：
2、当点击删除元素全部把li元素删除之后，继续点击按钮，会出现报错，是因为没有子元素了，继续移除就会出错，可以提前添加判断，如下：
3、当把开始3个li元素删除之后，点击添加按钮添加li元素，鼠标移入前三个的li的时候没有改变背景颜色。建议：
（1）可以使用标签名获取li元素，因为使用querySelectorAll获取的是最开始的，添加的就不获取了。
（2）在循环中判断，若是大于3的时候，就不绑定鼠标移入移出事件。
（3）可以把改变颜色的代码封装起来，对长度进行判断，然后调用；可以参考如下：
希望能帮助到你，欢迎采纳。
祝学习愉快！
0
相似问题