Validacion de Radiobutton

andoencombi · #1 (**permalink**) 24/02/2011, 16:33

Hola foro el problema que tengo es el siguiente tengo que validar que los radiobutton no se queden sin seleccion pero no sirve el codigo quisiera saber si alguien me puede ayudar plis

aqui el codigo

Código PHP:

  function comprobarRadio(radio) 
{ 
  for(i = 0;i < radio.length;i++) 
  { 
    if(radio[i].checked) 
    { 
      return false; 
    } 
  } 
  return true; 
}

y luego lo mando a llamar asi

Código PHP:

  if(!comprobarRadio(form.Valoracion1.value)) 
      alert("Por favor, seleccione una opción.");

ocp001a · #2 (**permalink**) 24/02/2011, 17:34

¿No está al revés? si hay alguno seleccionado (checked) regresas true, de lo contrario si termina de recorrer todos los radios y ninguno esta seleccionado, regresa false.

andoencombi · #3 (**permalink**) 24/02/2011, 17:48

de todos modos no pasa nada no lo verifica

andoencombi · #4 (**permalink**) 24/02/2011, 17:59

Va otra explicacion lo que quiero es que valide varios campos de texto y radio buttons pero los radiobuttons no los valida ya que debe estar seleccionado uno de lo contrario no mostrara el mensaje de datos correctos aqui esta el codigo completo

Código PHP:

  <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> 
<html xmlns="http://www.w3.org/1999/xhtml"> 
<head> 
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> 
<title>Documento sin título</title> 
<script language="javascript" type="text/javascript"> 
function vacio(cadena) 
  {                                    // DECLARACION DE CONSTANTES 
    var blanco = " \n\t" + String.fromCharCode(13); // blancos 
                                       // DECLARACION DE VARIABLES 
    var i;                             // indice en cadena 
    var es_vacio;                      // cadena es vacio o no 
    for(i = 0, es_vacio = true; (i < cadena.length) && es_vacio; i++) // INICIO 
      es_vacio = blanco.indexOf(cadena.charAt(i)) != - 1; 
    return(es_vacio); 
  } 
 
 
function comprobarRadio(radio) 
{ 
  for(i = 0;i < radio.length;i++) 
  { 
    if(radio[i].checked) 
    { 
      return true; 
    } 
  } 
  return false; 
} 
 
// ejemplo validacion formulario 
function ValidaCampos(form) 
  { 
    if(vacio(form.texto.value)) 
      alert("Texto incorrecto."); 
       
    else if(!comprobarRadio(form.Valoracion1.value)) 
    alert("Por favor, seleccione una opción."); 
       
    else if(vacio(form.texto2.value)) 
      alert("Texto2 incorrecto."); 
     
    else 
     
      alert("Los datos son correctos"); 
    return(false); 
  } 
 
</script> 
 
</head> 
 
<body> 
<form name="miformu" action="" onSubmit="return ValidaCampos(this)">Texto:<br><input type="text" name="texto"><br><br /> 
<input type="radio" name="Valoracion1" value="1" /> 
Opción 1 <br /> 
<input type="radio" name="Valoracion1" value="2" /> 
Opción 2 <br /><br /> 
Texto2:<br /> 
<input type="text" name="texto2" /> 
<br><br> 
<input type="submit" value="Validar"><input type="reset" value="Borrar"></form> 
 
</body> 
</html>

esjuanma · #5 (**permalink**) 24/02/2011, 18:21

Solo hay que corregir la línea 37:

Código javascript:

Ver originalelse if(!comprobarRadio(form.Valoracion1.value))

y sacarle el '.value':

Código javascript:

Ver originalelse if(!comprobarRadio(form.Valoracion1))

ya que a la función "comprobarRadio", debemos pasarle los elementos radio, y no su valor

Con solo ese cambio tu script ya está funcionando.
Saludos!

andoencombi · #6 (**permalink**) 24/02/2011, 18:25

genial gracias esjuanma funciona full