优化以下代码:window.onload=function(){ let changeColor=document.querySelectorAll(".changeColor") let linkcolorDom=document.querySelectorAll(".Color") // console.log(linkcolorDom) //本地存储 if(localStorage.getItem("color")===null){ localStorage.setItem("color",linkcolorDom[length-1].getAttribute("id")) }else{ console.log("getitem:"+localStorage.getItem("color")) judgeColor(localStorage.getItem("color")) } console.log(changeColor) for(let i=0;i<changeColor.length;i++){ changeColor[i].addEventListener("click",function(){ judgeId(changeColor[i].getAttribute("id")) judgeColor(changeColor[i].getAttribute("id")) } ) } function judgeId(id){ for(let i=0; i<linkcolorDom.length;i++){ if(id==="red"){ localStorage.setItem("color","red") }else if(id==="skyblue"){ localStorage.setItem("color","skyblue") }else if (id==="yellow"){ localStorage.setItem("color","yellow") } } } function judgeColor(color){ for(let i=0; i<linkcolorDom.length;i++){ linkcolorDom[i].disabled=true } for(let i=0; i<linkcolorDom.length;i++){ if(linkcolorDom[i].getAttribute("id")===color){ linkcolorDom[i].disabled=false } } }
时间: 2024-02-19 22:58:30 浏览: 59
可以尝试优化以下代码:
```
window.onload = function() {
const changeColor = document.querySelectorAll(".changeColor");
const linkcolorDom = document.querySelectorAll(".Color");
const lastLinkcolorDom = linkcolorDom[linkcolorDom.length - 1];
// 本地存储
if (localStorage.getItem("color") === null) {
localStorage.setItem("color", lastLinkcolorDom.id);
} else {
const color = localStorage.getItem("color");
console.log("getitem:" + color);
judgeColor(color);
}
console.log(changeColor);
changeColor.forEach(function(button) {
button.addEventListener("click", function() {
const id = button.id;
judgeId(id);
judgeColor(id);
});
});
function judgeId(id) {
switch (id) {
case "red":
localStorage.setItem("color", "red");
break;
case "skyblue":
localStorage.setItem("color", "skyblue");
break;
case "yellow":
localStorage.setItem("color", "yellow");
break;
default:
break;
}
}
function judgeColor(color) {
linkcolorDom.forEach(function(link) {
link.disabled = true;
});
linkcolorDom.forEach(function(link) {
if (link.id === color) {
link.disabled = false;
}
});
}
};
```
这里采用了以下优化:
1. 使用 `const` 和 `let` 来声明变量,避免使用 `var`。
2. 缓存了 `linkcolorDom` 中最后一个元素,避免多次访问数组。
3. 使用 `forEach()` 替代了 `for` 循环,代码更简洁。
4. 使用了 `switch` 语句来替代 `if-else`,代码更清晰。
阅读全文