2014-09-29 71 views
0

当下方运行,假设男性检查上运行多次,我希望jsonObj看起来像这样jQuery的每个相同的元素

0: Object 
    name: "sex" 
    value: "male_value" 

然而,它结束了看起来像这样。

0: Object 
    name: "sex" 
    value: "male_value" 
1: Object 
    name: "sex" 
    value: "male_value" 
2: Object 
    name: "sex" 
    value: "male_value" 
3: Object 
    name: "sex" 
    value: "female_value" 

HTML

<div id="form"> 
    <div id="radio_buttons"> 
     <input type="radio" name="sex" id="male" value="male_value">Male<br> 
     <input type="radio" name="sex" id="female" value="female_value">Female 
    </div> 
</div> 

JAVASCRIPT

function jsonAdd(name, value) 
{     
    item = {}; 
    item ["name"] = name; 
    item ["value"] = value; 

    jsonObj.push(item); 
} 

jsonObj = []; 

$("#form input").each(function() { 
    //Get Input Type 
    type = $(this).attr("type"); 
    switch(type) 
    { 
     case "radio": 
      if($(this).prop("checked")) 
      { 
       var name = $(this).attr("name"); 
       var value = $(this).val(); 
       jsonAdd(name, value); 
      } 
     case "checkbox": 
      if($(this).prop("checked")) 
       { 
        var name = $(this).attr("name"); 
        var value = $(this).val(); 
        jsonAdd(name, value); 
       } 
     default: 
      var name = $(this).attr("name"); 
      var value = $(this).val(); 
      jsonAdd(name, value); 
    } 
}); 

我似乎可以得到它正常工作,如果我不使用switch语句,但我不明白为什么一个简单的开关会导致这个问题?

+0

看起来你有两个问题。 1)即使您只需要一个值,您也可以迭代所有输入。 2)你忘了在你的'switch'中打破。为了解决第一个问题,一个解决方案是使用[FormData](https://developer.mozilla.org/en-US/docs/Web/API/FormData) – acbabis 2014-09-29 22:53:49

回答

1

的Javascript,像许多类似C语言中,都将落空,从一个case下一个,如果你忘了包括break(如指出here

所以:

switch(type) 
{ 
    case "radio": 
     if($(this).prop("checked")) 
     { 
      var name = $(this).attr("name"); 
      var value = $(this).val(); 
      jsonAdd(name, value); 
     } 
     break; 

    case "checkbox": 
     if($(this).prop("checked")) 
      { 
       var name = $(this).attr("name"); 
       var value = $(this).val(); 
       jsonAdd(name, value); 
      } 
     break; 

    default: 
     var name = $(this).attr("name"); 
     var value = $(this).val(); 
     jsonAdd(name, value); 
     break; 
} 

应该排序。

+0

呵呵,好尴尬。我的简单开关语句不起作用,因为我甚至没有正确切换。-_- 谢谢。 – Matt 2014-09-29 22:53:18

0

你已经忘了break语句添加到您的开关,在这种情况下default选项将始终运行:

case "radio": 
    if($(this).prop("checked")) 
    { 
     var name = $(this).attr("name"); 
     var value = $(this).val(); 
     jsonAdd(name, value); 
    } 
    break; 
case "checkbox": 
....