esta funcion no funciona correctamente, ¿me podeis ayudar?

[zaida]

Hola, tengo un problema y necesito que me ayudeis a resolverlo, espero saber explixarme puesto que soy nueva en esto de javascript.
tengo un boton que llama a la funcion: elija_lector(); que es esta:

function elija_lector()
{
if(document.getElementById('lector1').checked==fal se&&document.getElementById('lector2').checked==fa lse&&document.getElementById('lector3').checked==f alse)
{
alert("Ha de seleccionar al menos una zona");
}
else
{
if(document.getElementById('select_zonas2').length ==0)
{
document.getElementById('zonas').style.visibility= 'hidden';
document.getElementById('cabecera').style.visibili ty='visible';
document.getElementById('fondo_horario').style.vis ibility='visible';
document.getElementById('tabla_horario').style.dis play="block";
document.getElementById('horario2').style.visibili ty='visible';
document.getElementById('subhorario').style.visibi lity='hidden';
document.getElementById('nombre_subhorario').style .visibility='hidden';
document.getElementById('almacen_subhorario').styl e.visibility='hidden';
document.getElementById('calendario_fecha_inicio_f in').style.visibility='hidden';
document.getElementById('bloque').style.visibility ='hidden';
document.getElementById('nombre_bloque').style.vis ibility='hidden';
document.getElementById('almacen_bloque').style.vi sibility='hidden';
document.getElementById('periodo').style.visibilit y='hidden';
document.getElementById('almacen_periodo').style.v isibility='hidden';
document.getElementById('capa_almacen').style.visi bility='hidden';
oculta_calendarios();
document.getElementById('celda_horario').borderCol or="#000000";
document.getElementById('celda_horario').bgColor=" #9999ff";
document.getElementById('celda_subhorario').border Color="#d9e89b";
document.getElementById('celda_subhorario').bgColo r="#d9e89b";
document.getElementById('celda_periodo').borderCol or="#00ccff";
document.getElementById('celda_periodo').bgColor=" #00ccff";
document.getElementById('celda_bloque').borderColo r="#ff99ff";
document.getElementById('celda_bloque').bgColor="# ff99ff";
document.getElementById('almacen_horario').style.v isibility="visible";
document.getElementById('nombre_horario').style.vi sibility="hidden";
document.getElementById('select_elija_horario').di sabled=false;
document.getElementById('select_elija_horario').se lectedIndex=0;
limpiar();
numero4="";
}

después continúa el "else"

El problema es que no me funciona correctamente, esta líne no me funciona:


document.getElementById('tabla_horario').style.dis play="block";

Alguien puede darme una solución por favor??
Muchísimas gracias!!!

[zaida]

ya lo resolvi, no se como pero funciona!!!jeje

gracias de todos modos

saludos

[sir_bowen]

Imagino que las separaciones que hay en el código serán por culpa del copy-paste. Porque si no, te daría error de compilación...

Si no te funciona esa línea en concreto, asegúrate de que el identificador del elemento es correcto, porque la llamada está bien hecha...

Con respecto a la visibilidad de cada elemento, te recomendaría que bien utilices la propiedad display(block,none) o utilices la propiedad visibility(visible,hidden), pero no las mezcles dentro de un mismo elemento.

Espero que te haya servido de algo...

Salu2

[zaida]

gracias sir bowen!!!
claro que me sirve, todo lo que sea aprender es bueno, muchisimas gracias por tu recomendacíon, lo tendré en cuenta a la hora de hacerlo.

aH!!Muchas gracias por la rapidez en contestarme!!!

[chilenozzz]

Iniciado por zaida

ya lo resolvi, no se como pero funciona!!!jeje

gracias de todos modos

saludos

Sería bueno que publicaras cómo lo solucionaste...

Gracias

[derkenuke]

Una sugerencia (pijerías) para que sea más legible el código:

Código PHP:

  var aEsconder=[ 'zonas', 'subhorario', 'nombre_subhorario', 'almacen_subhorario', 'calendario_fecha_inicio_fin', 'bloque', 'nombre_bloque',' almacen_bloque', 'periodo', 'almacen_periodo', 'capa_almacen', 'nombre_horario' ];
for(var i in aEsconder)
    document.getElementById(aEsconder[i]).style.visibility='hidden';
var aMostrar=['cabecera', 'fondo_horario', 'horario2' ];
for(var i in aMostrar)
    document.getElementById(aEsconder[i]).style.visibility='visible'; 
   

A mí se me hace más cómodo de entender.


Saludos, y no olvides poner cómo lo solucionaste.