problemas con funcion validacion de radio button

[steffyn_90]

Buenas tardes Comunidad,

Soy demasiado mala en cuanto a javascript, y encontre este codigo que verifica si todos mis grupos de radio button que tengo en mi formulario

Código Javascript:

function validargrupos(formulario) {
    var grupos = [];
    for (i = 0; elem = formulario.elements[i]; i++) {
        if (elem.type == "radio") {
            if (!grupos[elem.name]) {
                grupos[elem.name] = elem.checked;
            }
        }
    }
    for (grupo in grupos) {
        if (!grupos[grupo]) {
            alert("Tiene que seleccionar: " + grupo);
            return false;
        }
    }
    return true;
}
 

pues en la parte del formulario pongo esto

Código HTML:

<form action="encuesta2.php" name="formulario" id="formulario" method="post" onSubmit="return validargrupos(this)" >
 

No se que estaré haciendo malo, porque quiero que al darle click me valide y sino que salga el msj de error y que me devuelva a la pagina encuesta2.php, pero me la direcciona a la siguiente pagina, porque tengo en mi código php tengo que si !$_POST, me direccione a otra pagina.

Muchas gracias por su ayuda, se los agradecere enormemente.

[rosonator]

Primero de todo, yo haría el botón por JS y no por HTML, es decir, que sea input.button (no input.submit) el que llama a la función en vez del submit del formulario.

Creo que lo que ocurre es que no estás mirando el checked en los hijos del elemento sino en el propio elemento, que no tiene esa propiedad. Prueba con esto:

Código:

todosActivos = [];

for (i = 0; elem = formulario.elements[i]; i++) {

       todosActivos [i]= false;

       if (elem.type == "radio") {

              cantidadRadios = elem.length;

              for (j=0; j < cantidadRadios ; j++) {
                     if (elem[j].checked==true) {
                            todosActivos[i]=true;
                     }
              }
       } 
}                   

if (todosActivos.indexOf(false)== -1) {
      document.forms['formulario'].submit();
} else {
//CASOS DE ERROR
}

[steffyn_90]

Iniciado por rosonator

Primero de todo, yo haría el botón por JS y no por HTML, es decir, que sea input.button (no input.submit) el que llama a la función en vez del submit del formulario.

Creo que lo que ocurre es que no estás mirando el checked en los hijos del elemento sino en el propio elemento, que no tiene esa propiedad. Prueba con esto:

Código:

todosActivos = [];

for (i = 0; elem = formulario.elements[i]; i++) {

       todosActivos [i]= false;

       if (elem.type == "radio") {

              cantidadRadios = elem.length;

              for (j=0; j < cantidadRadios ; j++) {
                     if (elem[j].checked==true) {
                            todosActivos[i]=true;
                     }
              }
       } 
}                   

if (todosActivos.indexOf(false)== -1) {
      document.forms['formulario'].submit();
} else {
//CASOS DE ERROR
}

Hola antes que nada, muchas gracias por tu ayuda y tiempo,

acabo de probar el codigo y pues igualmente si selecciono los grupos de radio siempre me sale el mensaje que no están seleccionado. He probado diversos codigos con el mismo resultado. T_T

De verdad muchas gracias.

[rosonator]

Quizás ayudara ver tu código html, el relativo a tus radiobuttons