Pegar ID dos CheckBox Marcados

[Raphael Machado]

Estou com uma grande dúvida e tenho urgência em ajuda! :rolleyes:

Tenho uma página com vários CheckBoxes e quero pegar o ID de todos os que estiverem marcados com JS.

Tenho esse código em uma página:

<script>
        function get_all_id()
        {
            var str = "";
            var boxes = document.getElementsByName("check_list");
            for(var i = 0; i < boxes.length; i++){
             if (boxes[i].checked == true) {
              var tmp = boxes[i].id.split("_");
              str+=tmp[1]+",";
             }
            } 
            document.getElementById('user_ids').value=str;
        }
</script>

<form name="myform" action="checkboxes.asp" method="post">
<div id='something'><input type="checkbox" id="cbx_88" name="check_list"></div>
<div id='something'><input type="checkbox" id="cbx_89" name="check_list"></div>
<div id='something'><input type="checkbox" id="cbx_90" name="check_list"></div>
<div id='something'><input type="checkbox" id="cbx_91" name="check_list"></div>
</form>

<form action="" method="post">
<input type="hidden" id="user_ids" value="" name="user_ids" />
    
    <br>
    <select name="I_want">
        <option value="0">I want to recycle them</option>
        <option value="1"> I want to renew them</option>
    </select>    
    
    <input type='button' name='submitUpdate' onclick="get_all_id();" value='Update'>

    </form>

Porém esse código não está funcionando!

Alguém pode me ajudar?

Editado Julho 27, 2010 por Raphael Machado

[Jonathan Queiroz]

Testei aqui no Firefox 3.6.7 e funcionou perfeitamente. O único problema (se realmente for um problema) é que sobra uma vírgula na variável str (ou seja, ao invés de 10,15 o valor do campo é 10,15,).

[Raphael Machado]

Vlw ai Jonathan!

Deve ser isso!

[Jonathan Queiroz]

De nada, qualquer coisa é só postar. Se for realmente isso peço que poste avisando para que o tópico seja marcado como Resolvido.

Para corrigir o problema da vírgula você pode trocar essa linha:

str+=tmp[1]+",";

Por:

str+=(i ? "," : "")+tmp[1];

[Eric Freitas]

function get_all_id()
        {
            var str = "";
            var boxes = document.getElementsByName("check_list");
            for(var i = 0; i < boxes.length; i++){
             if (boxes[i].checked == true) {
              var tmp = boxes[i].id.split("_");
              str+=tmp[1]+",";
             }
            }
            document.getElementById('user_ids').value=str;
        }

correção

if (boxes[i].checked == 'checked') alert(' fazer alguma coisa');

[Jonathan Queiroz]

Na verdade pelo que sei o correto em Javascript é comparar com true (booleano), e não com "checked" (string). Tanto é que, mesmo se você escrever explicitamente no HTML checked="checked" o element.checked no Javascript valerá true.