Problema al recoger en una variable los checkbox con un mismo name que estan checked

sqa212 · #1 (**permalink**) 25/07/2010, 13:15

Estoy intentando recoger en una variable todos los checkbox con un mismo name que estan checked y concatenarlos con | de la siguiente forma:

<html>
<head>
<script language="javascript" type="text/javascript">
function comprobarCheckBoxCheked(){
var ckbx_arr=document.getElementsByName('Idcondenv');

var TodosPerfilesSeleccionados='' // Variable que recoge todos los CheckBox seleccionados
var ckbx_arr_ln=ckbx_arr.length;
for(var k=0;k<ckbx_arr_ln;k++){
if(ckbx_arr[k].checked)
if (TodosPerfilesSeleccionados==''){
TodosPerfilesSeleccionados=ckbx_arr[k].value
alert('TodosPerfilesSeleccionados:'+ TodosPerfilesSeleccionados);
}else{
TodosPerfilesSeleccionados= TodosPerfilesSeleccionados+ '|' + ckbx_arr[k].value;
alert('TodosPerfilesSeleccionados:'+ TodosPerfilesSeleccionados);

}
}
}
</script>
</head>
<body>
1. <INPUT value=1 CHECKED type=checkbox name=Idcondenv>
<br />
2. <INPUT value=2 type=checkbox name=Idcondenv unchecked>
<br />
3. <INPUT value=3 type=checkbox name=Idcondenv unchecked>
<a href="javascript:void(0)" rel=nofollow onClick="return comprobarCheckBoxCheked();">comprobar checkbox CHECKED</a>
</body>
</html>

Sin embargo al seleccionar más de un checkbox la variable TodosPerfilesSeleccionados solo muestra un valor aunque seleccione más de un checkBox, es decir, no me esta concatenado los checkBox en la variable TodosPerfilesSeleccionados.

¿Alguna idea de que estoy haciendo mal?

Gracias

sqa212 · #2 (**permalink**) 25/07/2010, 13:49

Ya lo solucione, olvide poner el { despues de if(ckbx_arr[k].checked)

_cronos2 · #3 (**permalink**) 25/07/2010, 14:46

De todas maneras a mí ese código no me funciona. Si marco más de un checkbox, me salen tantos alerts como checkboxs hay marcados. Supongo que no es eso lo que tú quieres, así que prueba con este código:

Código Javascript:

Ver originalfunction comprobarCheckBoxCheked(){
 var chk = document.getElementsByName('Idcondenv');
 var len = chk.length;
 var sel = new Array();
 for (i = 0; i<chk.length; i++){
  if(chk[i].checked){
   sel[sel.length] = chk[i].value;
  }
 }
 alert('TodosPerfilesSeleccionados: ' + sel.join(' | ')); 
}

Y por cierto, es mejor que uses un botón en vez de un enlace.
Saludos (:

sqa212 · #4 (**permalink**) 25/07/2010, 16:08

Hola _cronos2, gracias por tu respuesta, efectivamente salian tantos alert como checkbox habia selecionado, porque los puse para ver lo que iba trayendo la variable (Debugging), poniendo el alert al final la variable trae todos los valores .

¿Por que es mejor usar un boton que un enlace?

<script language="javascript" type="text/javascript">
function comprobarCheckBoxCheked(){
var ckbx_arr=document.getElementsByName('Idcondenv');

var TodosPerfilesSeleccionados='' // Variable que recoge todos los CheckBox seleccionados
var ckbx_arr_ln=ckbx_arr.length;
for(var k=0;k<ckbx_arr_ln;k++){
if(ckbx_arr[k].checked){
if (TodosPerfilesSeleccionados==''){
TodosPerfilesSeleccionados=ckbx_arr[k].value
}else{
TodosPerfilesSeleccionados= TodosPerfilesSeleccionados+ '|' + ckbx_arr[k].value;
}
}
}
alert('TodosPerfilesSeleccionados:'+ TodosPerfilesSeleccionados);

}
</script>

_cronos2 · #5 (**permalink**) 25/07/2010, 16:40

Si te aclaras mejor con ese código vale, el mío sólo era para orientarte y porque al usar arrays es más corto, pero es mejor que tú lo entiendas del todo.
Lo del enlace, no es que funcione mejor o peor, pero digamos que es más correcto, o a mí me lo parece, pero por funcionar funcionan igual de bien.

Saludos (:

sqa212 · #6 (**permalink**) 26/07/2010, 15:34

Gracias _cronos2