Problema con checkbox

roser24 · #1 (**permalink**) 10/03/2011, 07:02

Hola a todos,primeramente gracias por tomaros el tiempo de leer mi post,vereis no se porque la funcion fvalidar no funciona bien en cuanto a mis checkbox,en cuanto detecta que no esta ninguno marcado si me da el alert de que marque uno pero tras aceptar el alert y marcar el checkbox me sigue dando el mismo alert..esto me trae de cabeza..porfavor si alguien sabe como solucionarlo..gracias de antemano..salu2.

[HIGHLIGHT="Javascript"]
<script language="javascript">
var x=0;
var opcion=false;
var ok='s';
var band='s';

function fenter(caja,elevento){
var eve=elevento||window.event;
if(eve.keyCode==13){
switch(caja){

case 1:document.getElementById('contraseña').focus();
break;
case 2:document.getElementById('aficiones').focus();
break;
case 3:document.getElementById('apellido').focus();
break;
case 4:document.getElementById('x').focus();
break;

}
}

}

function fvalidar(){

if(document.getElementById('nombre').value==""){
ok='n'
document.getElementById('errorusu').style.visibili ty='visible';
}
else{
document.getElementById('errorusu').style.visibili ty='hidden';
}
if(document.getElementById('apellido').value==""){
ok='n';
document.getElementById('errorape').style.visibili ty='visible';
}
else{
document.getElementById('errorape').style.visibili ty='hidden';
}
if (document.getElementById('pass').value==""){
ok='n';
document.getElementById('errorpass').style.visibil ity='visible';
}
else{
document.getElementById('errorpass').style.visibil ity='hidden';
}

var camposCheck=document.getElementsByName("afi");
while(x<camposCheck.length && !opcion){

if(camposCheck[x].checked){
opcion=true;
}
x++;
}
if(!opcion){alert("tienes que seleccionar alguna opcion");band='n';}

if(ok=='s'&& band=='s'){
document.f.submit();
}

}
</script>[HIGHLIGHT]

Código HTML:

Ver original<body>
<table border=0>
<form action="prueba.php" method=POST name=f>
<tr><td>
Nombre:<input type="text" name=nombre id=nombre onkeypress='fenter(1,event)'><label id=errorusu style="visibility:hidden;color:red">Introduce el nombre</label></td></tr>
<tr><td>Conraseña:<input type="password" name=pass id=pass onkeypress='fenter(2,event)'><label id=errorpass style="visibility:hidden;color:red">Introduce contraseña</label></td></tr>
<tr><td>Aficiones:<input type="text" name=aficiones id=aficiones onkeypress='fenter(3,event)'><label id=erroafic style="visibility:hidden;color:red">Introduce aficiones</label></td></tr>
<tr><td>apellido:<input type="text" name=apellido id=apellido onkeypress='fenter(4,event)'><label id=errorape style="visibility:hidden;color:red">Introduce apellido</label></td></tr>
 
<tr><td>Marque al menos una aficion:<br>
<input type=checkbox name="afi" id="musica" value="musica">Musica<br>
<input type=checkbox name="afi" id="leer" value="leer">Leer<br>
<input type=checkbox name="afi" id="correr" value="correr">Correr<br>
</td></tr>
<tr><td><input type=button name=x id=x value="enviar" onclick='fvalidar();'></td></tr>
</form>

IsaBelM · #2 (**permalink**) 10/03/2011, 07:31

eso ocurre por que aunque encuentre un checkbox seleccionado el ciclo continua. una vez que encuentre uno seleccionado has de parar el ciclo con break

roser24 · #3 (**permalink**) 10/03/2011, 09:01

Hola Isabel ,primeramente gracias por responder a mi pedida de socorro con este codigo,veras yo tmb probe lo del break y lo he vuelto a probar pero sigue haciendome lo mismo.
La operacion que realizo es la siguiente:
1º completo todos los campos menos los checkbox
2º me sale el alert de que no he marcado ningun checkbox.
3º acepto el mensaje y marco uno ,dos o tres checkbox .le vuelvo a dar a enviar y me vuelve a salir el alert anterior..
y asi.. sin embargo si mando la pagina con todo marcado me lo valida bien y me hace el submit..pero si hago los pasos anteriores no sale de ese bucle ni con el break;

mi codigo actual es el siguiente:

Código Javascript:

Ver originalvar camposCheck=document.getElementsByName("afi");
        while(x<camposCheck.length && !opcion){
            
            if(camposCheck[x].checked){
            opcion=true;    
            break;
            }
            x++;
        }
            if(!opcion){
                alert("tienes que seleccionar alguna opcion");
                   band='n';
                   }
                  
       
        
        if(ok=='s'&& band=='s'){
        document.f.submit();    
        }

porcierto he estado viendo la consola de errores y me da error en el break ahora..

sysmadryn · #4 (**permalink**) 10/03/2011, 13:59

Hola.

Lo que sucede es que debes resetear las variables porque tanto x como opcion quedan cargadas con el ultimo valor.

Pasa la definicion de x y opcion dentro de la funcion :

Cita:

function fvalidar(){

var x=0;
var opcion=false;

.....

Saludos

roser24 · #5 (**permalink**) 10/03/2011, 14:04

te refieres a que debo de ponerlas al finalizar mi funcion¿?..
porque si las tengo inicializadas pero como globales..,

muchas gracias por responder =)

roser24 · #6 (**permalink**) 10/03/2011, 14:10

un mILllllon de gracias ,tenias razon mi error era de que inicilize esas variables como globales y una vez cogian ese valor en la funcion se quedaban con el.
me has salvado la vida,que mañana tengo un examen y llevo todo el dia con esas funciones..
Que tengas un feliz dia sysmadrin =)

sysmadryn · #7 (**permalink**) 10/03/2011, 14:21

Me alegro !!
Quizas te sea util este consejo : cuando tengas un if o loop o funcion que no te funcione como quieres, ponle dentro un alert y mira los valores que van tomando las variables que controlan el loop/funcion en runtime.

Saludos

IsaBelM · #8 (**permalink**) 10/03/2011, 14:38

cuando te dije que usaras break, me refería a hacerlo así. la explicación no fue lo suficientemente detallada

Cita:

var camposCheck=document.getElementsByName("afi");
for(var x = 0; ck= camposCheck[x]; x++)

if(ck.checked) break;