validar varios radios

ManatiKo · #1 (**permalink**) 08/11/2009, 00:13

saben tengo que hacer una búsqueda avanzada pero no me quiere salir...
aqui esta el codigo completo para que no se enrreden

...

Código:

<html>
<script language="JavaScript">
fuction Valida()
{
	switch(form2.se)
	{
		case 0:
			alert("seleccione alguna opcion");
			return false;
			break;
		case 1:
			if(form2.stip.value == 0)
				{
					alert("debe seleccionar alguna opciond ela lista");	
					document.form2.stip.focus();
					return false;
				}
			break;
		case 2:
			if(form2.smarc.value == 0)
				{
					alert("debe seleccionar alguna opciond ela lista");	
					document.form2.smarc.focus();
					return false;
				}
			break;
		case 3:
			if(form2.smodel.value.length == 0)
				{
					alert("debe seleccionar alguna opciond ela lista");	
					document.form2.smodel.focus();
					return false;
				}
			break;
		cese 4:
			if(form2.de.value.length == 0 || form2.hasta.value.length == 0)
				{
					alert("debe seleccionar alguna opciond ela lista");	
					document.form2.de.focus();
					return false;
				}
			break;
		case 5:
			 if(form2.sdist.value == 0)
				{
					alert("debe seleccionar alguna opciond ela lista");	
					document.form2.sdist.focus();
					return false;
				}
			 break;
	}
	return true;
}
</script>



<body bgcolor="black">
<center>
<img src="images/avanzada.jpg"
<font color="white">
        <form action="Busqueda/avanzada2.php" method="post" id="form2" name="form2" onsubmit="return Valida(this)">
                <input type="radio" name="se" value="1">&nbsp;
                Tipo<select name="stip">
                        <option value="0">_-seleccione tipo-_</option>
                        <option value="notebooks">notebooks</option>
                </select><br><br>
                <input type="radio" name="se" value="2">&nbsp;
                Marca<select name="smarc">
                        <option value="0">_-seleccione marca-_</option>
                        <option value="compaq">Compaq</option>
                        <option value="hp">Hp</option>
                        <option value="packard bell">Packard bell</option>
                        <option value="vaio">Vaio</option>
                </select><br><br>
                <input type="radio" name="se" value="3">&nbsp;
                Modelo <input type="text" name="smodel"><br><br>
                <input type="radio" name="se" value="4">
                Rango precio De:<input type="text" name="de"><br>
		Hasta:<input type="text" name="hasta"><br><br>
                <input type="radio" name="se" value="5">&nbsp;
                Distribucion<select name="sdist">
                        <option value="0">_-seleccione distribucion-_</option>
                        <option value="debian">Debian</option>
                        <option value="fedora">Fedora</option>
               </select><br><br>
                <input type="submit" name="submit">
</font>
</center>
</form>
</body>
</html>

quisas seria mejor usar "if" que dicen ustedes??

caricatos · #2 (**permalink**) 08/11/2009, 01:45

Hola:

Tienes cosas que no entiendo, sipones:

Código:

onsubmit="return Valida(this)"

deberías poner luego:

Código:

fuction Valida(form2)

Tal xomo lo tienes tal vez funcione en explorer, pero no vale en otros navegadores.

También el switch, porque debes consultar el value, y no olvides que se trata de texto:

Código:

switch(form2.se.value) {
case "0":

Revisa esas cosillas y nos cuentas.

Saludos

zerokilled · #3 (**permalink**) 08/11/2009, 01:51

ademas de lo comentado por caricatos, tienes un error de logica pero antes debes comprender un concepto. cuando mas de un elemento comparte el mismo valor en el atributo name, el DOM devuelve una coleccion de elementos. en dicho caso, la coleccion esta compuesta de todos los elementos que contiene el mismo valor. pero, cuando solo existe un elemento con dicho nombre, el DOM devuelve la referencia de este elemento. la diferencia radica en que una coleccion se maneja como un array mientras que un simple elemento es como un objeto.

aplicado a tu caso, los cinco radios tienen el mismo nombre de modo que form2.se es una coleccion. el error de algoritmo que tienes es que le estas pasando la coleccion a switch en lugar del valor que repesenta el radio seleccionado. antes de pasar el valor tienes que determinar cual de los radios esta activado. para determinarlo, recorres la coleccion con un bucle e identifica el radio seleccionado por su propiedad checked. por ejemplo,

Código:

// asumiendo que radios es la coleccion;
for(var index = 0; radios[index]; index++)if(radios[index].checked)break;
switch(radios[index].value){
...
}

lo que resulta de este ejemplo es que la variable de iteracion (index) corresponde a la posicion de la coleccion del elemento seleccionado por el usuario. luego en switch le pasas el valor de la propiedad value del elemento seleccionado.

pero no todo termina ahi. ahora considera el valor que tiene cada radio, pues no encajan correctamente con la estructura de los cases. por ejemplo, el primer radio tiene como valor cero (0) y en el switch, case 0 indica un mensaje que no corresponde a este radio.

@caricatos,
¡¡rayos!! me tomo mucho escribir lo que en pocas palabras tu escribistes. me vas a tener que enseñar redaccion eficaz y simple...

creo que es porque me envuelvo mucho en los detalles.

edit @ 2:59,
otro error que tienes es de bajo nivel, o sea la sintaxis. en especifico, has escrito mal la declaracion function y case.

Código:

fuction Valida()
{
...
		cese 4:
...
}

ManatiKo · #4 (**permalink**) 08/11/2009, 09:39

lo edite para que solo mande un mensaje para cuando se selecciona algo y después poder avanzar segun eso...
miren hice los arreglos que me dijeron... pero sige sin mandar nada :S

Código:

<script language="JavaScript">
function Valida(form1)
{
        for(var index=0;form2.me[index];index++)
        if(form2.se[index].checked)
        break;
        switch(form2.se[index].value)
        {
                case 0:
                        alert ("seleccione alguna opcion")
                        return false;
                        break;
                case 1:
                        alert ("seleccion 1")
                        return false;
                        break;
                case 2:
                        alert ("seleccion 2")
                        return false;
                        break;
                case 3:
                        alert ("seleccion 3")
                        return false;
                        break;
                case 4:
                        alert ("seleccion 4")
                        return false;
                        break;
                case 5:
                        alert ("seleccion 5")
                        return false;
                        break;
        }
        return true;
}
</script>

disculpen por las molestias pero no llevo mucho en javaScript y me pierdo un poco...

zerokilled · #5 (**permalink**) 08/11/2009, 10:31

veras, en tu codigo, ¿donde usas form1?

Código:

function Valida(form1)

ese argumento debio llamarse form2. o si prefieres form1, entonces en el cuerpo de la funcion tienes que cambiar todos los form2 por form1.

ManatiKo · #6 (**permalink**) 08/11/2009, 10:57

si me equivoque hay pero igual no funciona :S

zerokilled · #7 (**permalink**) 08/11/2009, 11:05

Código:

for(var index=0;form2.me[index];index++)

a ver si le encuentras algo raro...

ManatiKo · #8 (**permalink**) 08/11/2009, 11:13

Código:

for(var index=0;form2.se[index];index++)

puse eso... pero no pasa nada T.T

zerokilled · #9 (**permalink**) 08/11/2009, 11:24

cito algunas lineas como recordatorio...

Cita:

Iniciado por caricatos

También el switch, porque debes consultar el value, y no olvides que se trata de texto:

Código:

switch(form2.se.value) {
case "0":

Cita:

Iniciado por zerokilled

pero no todo termina ahi. ahora considera el valor que tiene cada radio, pues no encajan correctamente con la estructura de los cases. por ejemplo, el primer radio tiene como valor cero (0) y en el switch, case 0 indica un mensaje que no corresponde a este radio.

ManatiKo · #10 (**permalink**) 08/11/2009, 13:46

hay me salio !!! muchas gracias por todo

una ultima pregunta :P en el switch se puede usar default??

bueno eso seria... gracias

ManatiKo · #11 (**permalink**) 08/11/2009, 13:47

hay ta el codigo...

Código:

<script language="JavaScript">
function Valida(form2)
{
        for(var index=0;form2.se[index];index++)
        if(form2.se[index].checked)
        break;
        switch(form2.se[index].value)
        {
                case "0":
                        alert ("seleccione alguna opcion")
                        return false;
                        break;
                case "1":
                        alert ("seleccion 1")
                        return false;
                        break;
                case "2":
                        alert ("seleccion 2")
                        return false;
                        break;
                case "3":
                        alert ("seleccion 3")
                        return false;
                        break;
                case "4":
                        alert ("seleccion 4")
                        return false;
                        break;
                case "5":
                        alert ("seleccion 5")
                        return false;
                        break;
		default:
			alert("seleccione algo")
			return false;
			break;
        }
        return true;
}
</script>

zerokilled · #12 (**permalink**) 08/11/2009, 13:51

es correcto, se puede usar default.

ManatiKo · #13 (**permalink**) 08/11/2009, 13:53

de la misma forma en que lo use en el codigo??

zerokilled · #14 (**permalink**) 08/11/2009, 13:55

es correcto, si miras el enlace hay dos ejemplos.

zerokilled · #15 (**permalink**) 08/11/2009, 14:26

supongo que ya se a que viene la pregunta. tu esperas que al no selecionar algun radio, el switch vaya al caso predeterminado, o sea, default. tiene sentido y logica pero aun asi no funciona. la razon, al no haber un radio seleccionado la variable de iteracion (index) del bucle for termina apuntando a un indice que no existe en la coleccion.

aplicado a tu caso, cuando no hay radio seleccionado, index contiene el valor cinco (5), de modo que form2.se[5] devuelve undefined porque no existe dicha posicion. recuerda, los indices de los arrays, al igual que las colecciones de elementos, comienzan desde cero (0) siendo este el primer elemento. al form2.se[5] equivaler undefined, form2.se[5].value genera error porque no existe la propiedad value en undefined, e interrumpe la ejecuccion del script.

para corregirlo, tienes que verificar si dicho indice existe. si no existe significa que no se ha seleccionado un radio, y aqui entonces lanzas el mensaje de default.

Código:

// esto va entre el bucle for y switch;
if(!form2.se[index]){
alert("seleccione algo");
return false;
}

ManatiKo · #16 (**permalink**) 08/11/2009, 15:50

sin eliminar el otro if ya ingresado anteriormente me refiero a este

Código:

if(form2.se[index].checked)
	break;

zerokilled · #17 (**permalink**) 08/11/2009, 15:55

es correcto, sin eliminar ese if. o sea, mas bien el codigo va justo antes de switch.

Gothgauss · #18 (**permalink**) 08/11/2009, 16:06

Te voy a poenr un codigo a ver si te vale:

Código HTML:

<SCRIPT LANGUAGE="JAVASCRIPT">
function valuechecked(radio) {
   for (i=0; i<radio.length; i++) {
      if (radio[i].checked) {
         return radio[i].value;
      }
   }
}
function recoger(){
	var tvit = parseInt(valuechecked(form.vvitoria));

}
</SCRIPT>

Con la primera funcion recorres todos los input radio y con la segunda vas recogiendo el varlor del input concreto, en este caso "vvit"

ManatiKo · #19 (**permalink**) 08/11/2009, 16:10

muchas gracias zerokilled

, me ayudaste mucho....

bueno dejo el codigo terminado con todas las validaciones que le hice acá

Código:

<html>
<script language="JavaScript">
function Valida(form2)
{
//ve si existe un index
	for(var index=0;form2.se[index];index++)
	if(form2.se[index].checked)
	break;
//ve si hay algo seleccionado
	if(!form2.se[index])
	{
		alert("seleccione alguna opcion");
		return false;
	}
//comienza el switch
        switch(form2.se[index].value)
        {
//aqui vemos si hay algo seleccionado en el list
                case "1":
                        if(form2.stip.value == "0")
				{
					alert("elija alguna opcion")
					return false;
					break;
				}
			else
			break
                case "2":
                        if(form2.smarc.value == "0")
				{
					alert("elija alguna opcion")
					return false;
					break;
				}
			else
			break
//aqui vemos si el text esta vacio
                case "3":
                        if(form2.smodel.value.length == 0)
				{
					alert("escriba algo")
					return false;
					break;
				}
			else
			break
//aqui vemos si dos text estan vacios(se supone que son un rango 'de' 'hasta') 
                case "4":
                        if(form2.de.value.length == 0 || form2.hasta.value.length == 0)
				{
					alert("debe llenar los 2 precios")
					return false;
					break;
				}
//si uno es mayor al otro
			if(form2.de.value.length > form2.hasta.value.length)
				{
					alert("el precio 'de' debe ser menos que 'hasta'")
					return false;
					break;
				}
//si es numero
			if(isNaN(form2.de.value) || isNaN(form2.hasta.value))
			{
				alert("Debe ingresar solo numeros")
				return false;
				break
			}

			else
			break;

                case "5":
                        if(form2.sdist.value == "0")
				{
					alert("elija alguna opcion")
					return false;
					break;
				}
			else
			break

		default:
			alert("seleccione algoa")
			return false;
			break;
        	}
		return true;
}
</script>

gracias...