错误：无法读取属性'检查'未定义

Question

我试图将选中复选框的结果存储在本地存储中，但是当我尝试保存时出现上述错误。

不太清楚什么是未定义的，因为我看到了我的代码中的属性。 下面的代码：

var getCheckboxValue = function(){ 
    var checkboxes = document.forms[0].features; 
    for(var i=0, j=checkboxes.length; i<j; i++){ 
     if(checkboxes[i].checked){ 
     console.log(checkboxes[i].value); 
      //featuresValue = $("features").value(); 
     }else{ 
      checkboxes = "No" 
     } 
     } 
    } 

var storeData = function(){ 
    var id = Math.floor(Math.random()*10000001); 
    //getSelectedRadio(); 
    getCheckboxValue(); 
    var item = {}; 
     item.brand = ["Brand",$("brand").value]; 
     item.model = ["Model",$("model").value]; 
     item.comments = ["comments",$("comments").value]; 
     //item.acoustic = ["acoustic", acousticValue]; 
     item.features = ["features",checkboxes]; 

    //Save data into local storage: Use Stringify to convert our object to a string 
    localStorage.setItem(id,JSON.stringify(item)); 
    alert("Contact Saved!"); 
} 


//set link & submit click events 
var save = $("button"); 
save.addEventListener("click", storeData); 

，这里是相关的HTML：

<li>Features: 
        <ul> 
         <li><input type="checkbox" name="features" value = "Cutaway" id="cutaway" /><label for="cutaway">Cutaway</label></li> 
         <li><input type="checkbox" name="features" value = "Finished" id="finished" /><label for="finished">Finished</label></li> 
         <li><input type="checkbox" name="features" value = "Inlay" id="inlay" /><label for="inlay">Inlay</label></li> 
         <li><input type="checkbox" name="features" value = "Wide Neck" id="wneck" /><label for="wneck">Wide Neck</label></li> 
         <li><input type="checkbox" name="features" value = "Left-handed" id="lhanded" /><label for="lhanded">Left-handed</label></li> 
        </ul> 
       </li> 

另外，这里是充分代码的链接在GitHub上：

https://github.com/b80266/VFW-Project-2/blob/master/additem.html

https://github.com/b80266/VFW-Project-2/blob/master/main.js

Answer 1

问题是您所覆盖的checkboxes值（形式复选框的原始集合）位置：

}else{ 
    checkboxes = "No" 
} 

这是内循环的 ...这是通过迭代checkboxes循环。它现在迭代字符串“No”中的字符，而不是您最初检索的元素的集合。该字符串只有2个字符长，而原始循环的缓存值为checkboxes.length（5个元素），存储在j中，每次迭代都不更新，这意味着它将循环超过i = 1，访问第三，第四和第五个索引，它们分别是现在未定义。

另一个“问题”，是在你的storeData功能，你打电话getCheckboxValue但不存储任何地方......再后来你引用一些checkboxes变量后 - item.features = ["features",checkboxes];。您需要存储结果，然后使用该变量。下面是这似乎与一些假设HTML工作演示：

var $ = function (id) { 
    return document.getElementById(id); 
}; 

var getCheckboxValue = function() { 
    var checkboxes = document.forms[0].features; 
    for (var i = 0, j = checkboxes.length; i < j; i++) { 
     if (checkboxes[i].checked) { 
      console.log(checkboxes[i].value + " is checked"); 
     } else { 
      //checkboxes = "No"; 
      console.log(checkboxes[i].value + " is not checked"); 
     } 
    } 
}; 

var storeData = function() { 
    var id = Math.floor(Math.random()*10000001); 
    var checkedBoxes = getCheckboxValue(); 
    var item = {}; 
    item.brand = ["Brand",$("brand").value]; 
    item.model = ["Model",$("model").value]; 
    item.comments = ["comments",$("comments").value]; 
    item.features = ["features",checkedBoxes]; 

    //Save data into local storage: Use Stringify to convert our object to a string 
    localStorage.setItem(id,JSON.stringify(item)); 
    alert("Contact Saved!"); 
}; 


//set link & submit click events 
var save = $("button"); 
save.addEventListener("click", storeData, false); 

DEMO：http://jsfiddle.net/jmUXY/1/

Answer 2

基于伊恩的建议，您可以检查所有无线电设备在任何数量的形式。见为例：

jQuery(document).ready(function() { 
 

 
\t $("input[type='radio']").change(function() { 
 
\t \t if (this.checked) { 
 
\t \t \t //Do stuff 
 
\t \t \t //console.log("foi clicado"); 
 
\t \t \t var marcados = 0; 
 
\t \t \t var naoMarcados = 0; 
 
\t \t \t //conta numero de checados 
 
\t \t \t var inputElems = document.getElementsByTagName("input"), 
 
\t \t \t \t count = 0; 
 
\t \t \t for (var i = 0; i < inputElems.length; i++) { 
 
\t \t \t \t if (inputElems[i].type == "radio" && inputElems[i].checked === true) { 
 
\t \t \t \t \t count++; 
 
\t \t \t \t \t marcados = count; 
 
\t \t \t \t } else { 
 
\t \t \t \t \t naoMarcados = naoMarcados + 1; 
 
\t \t \t \t } 
 
\t \t \t } 
 
\t \t \t $("#marcados").html("Total de Marcados: " + marcados); 
 
\t \t \t $("#naomarcados").html("Total de Não Marcados: " + naoMarcados); 
 

 
\t \t } 
 
\t }); 
 
});

<script 
 
    src="https://code.jquery.com/jquery-3.2.1.js" 
 
    integrity="sha256-DZAnKJ/6XZ9si04Hgrsxu/8s717jcIzLy3oi35EouyE=" 
 
    crossorigin="anonymous"></script> 
 
    <p> 
 
    FORM 1 
 
    </p> 
 
<form id="1"> 
 
<input type="radio">Em grau muito alto 
 
    <input type="radio">Em grau alto 
 
    <input type="radio">Em grau médio 
 
    <input type="radio" CHECKED>Em grau baixo 
 
    <input type="radio">Em grau muito baixo 
 
    <input type="radio">Não observo 
 
    </form> 
 
    <hr> 
 

 
    <p> 
 
    FORM 2 
 
    </p> 
 
    <form id="2"> 
 
<input type="radio">Em grau muito alto 
 
    <input type="radio">Em grau alto 
 
    <input type="radio">Em grau médio 
 
    <input type="radio">Em grau baixo 
 
    <input type="radio">Em grau muito baixo 
 
    <input type="radio">Não observo 
 
    </form> 
 
    <hr> 
 
    <div id="marcados"></div> 
 
    <div id="naomarcados"></div>