如何使用数组而不是此重复函数中的变量

您可以使用第二个函数，该函数将接受本地存储密钥和复选框 id，例如

window.onload = function Equal() {
    setCheckboxState('box1', 'b1');
    setCheckboxState('box2', 'b2');
}
function setCheckboxState(id, key) {
    document.getElementById(id).checked = 1 == localStorage.getItem(key);
}

您可以将公共逻辑分离到另一个函数中

window.onload = function Equal() {
  
  function extractFromStorage(a, b) {
    var bookstorname = localStorage.getItem(a)
    if (bookstorname == 1) {
      document.getElementById(b).setAttribute('checked','checked');
    }
    if (bookstorname == 0) {
      document.getElementById(b).removeAttribute('checked','checked');
    }
  }
  
  extractFromStorage('b1', 'box1');
  extractFromStorage('b2', 'box2');
}

function doTheStuff(a, b) {
  var bookstorname = localStorage.getItem(a)
  if (bookstorname == 1) {
    document.getElementById(b).setAttribute('checked','checked');
  }
  if (bookstorname == 0) {
    document.getElementById(b).removeAttribute('checked','checked');
  }  
}

window.onload = function Equal() {
  doTheStuff('b1', 'box1');
  doTheStuff('b2', 'box2');
}

？

这就是我会这样做的方式。

您的代码存在几个问题。

• 您不检查要将属性固定到的元素存在。 您不检查您得到的localStorage项目是否定义。
• 使用函数名称 Equal 污染全局命名空间。
• 该函数不应以大写字母命名，因为它不是对象生成器。
• 无需使用 setAttribute 和 removeAttribute ，在事实上removeAttribute在这种情况下没有意义，因为你不能从元素中删除选中的属性。顺便说一句，为什么在这里使用setAttribute而不是window.onload？
• checked属性为真或假，它不使用字符串"已检查">
• 通过 onload 属性绑定 load 事件并不安全，因为您可能阻止第三方代码，或者更糟的是第三方代码可能会阻止您。
• 没有错误检查。DOM 页面是动态环境，页面有来自许多地方的广告和内容，可能会干扰您的法典。始终牢记这一点进行编码。检查可能的错误，并以对最终用户友好的方式处理它们。在这种情况下，我使用了警报，对普通用户不友好，但对编码员来说
。

我的解决方案。

// add an event listener rather than replace the event listener
window.addEventListener(
    "load",  // for the load event
    function(){
        // the update function that is called for each item;
        var update = function(item){
            // the right hand side equates to true if the localstorage
            // is equal to "1". LocalStorage allways returns a string or
            // undefined if the key is not defined.
            item.element.checked = localStorage[item.storageName] === "1";
        }   
        // safe element getter     
        var getElement = function(eId){
            var e = document.getElementById(eId); // try and get the element
            if(e === null){  // does it exist?
                throw "Missing element:"+eId;  // no then we can not continue
                                               // the program stops here unless
                                               // you catch the error and deal with 
                                               // it gracefully.
            }
            return e; //ok return the element.
        }
        // Item creator. This creates a new item.
        // sName is the local storage name
        // eId id the element ID
        var item = function(sName, eId){
            return {
                storageName: sName,  // set the loaclStorage name
                element:getElement(eId); // get the element and check its safe
            };
        }
        // make it all safe
        try{
            // create an array of items.
            var items = [
                item("b1","box1"),
                item("b2","box2")
            ];
            // for each item update the element status
            items.forEach(update);
        }catch(e){
            alert("Could not update page?");
        }
    }
);