ayuda sencilla

[trepax]

hola a todos, soy muy novato en javascript y estoy intentando aprender de forma autodidacta. Me encuentro con el siguiente problema. He adaptado una función que muestra y oculta capas, pero como veréis no tengo muchos conocimientos.
había pensado hacerlo mejor con unas condiciones while, if, etc... pero no sé por donde empezar.
os adjunto parte del código. GRACIAS y SALUDOS!

function muestra_oculta(id , id1, id2, id3, id4, id5, id6, id7, id8, id9, id10, id11, id12, id13, id14, id15, id16, id17, id18, id19, id20, id21, id22, id23, id24, id25){
if (document.getElementById){
var el = document.getElementById(id);
el.style.display = (el.style.display = 'block')
var el1 = document.getElementById(id1);
el1.style.display = (el1.style.display = 'none')
var el2 = document.getElementById(id2);
el2.style.display = (el2.style.display = 'none')
var el3 = document.getElementById(id3);
el3.style.display = (el3.style.display = 'none')
var el4 = document.getElementById(id4);
el4.style.display = (el4.style.display = 'none')
???.

[trepax]

alguien puede hecharme un cable?? GRACIAS!!!

[crujikoki]

Hola,

No hace falta que doblepostees... Ten paciencia que seguro que alguien te contesta.

Sobre tu pregunta, no entiendo muy bien que quieres saber. Para empezar, no se que hace el if (document.getElementById) Pero suponiendo que lo se, me parece una locura hacer una función con 25 parámetros!!!

Me puedes explicar que es lo que pretendes que haga tú función? Si lo entiendo quizás te puedo dar una solución más óptima!

Por ejemplo, no entiendo por que todos están a "none" menos el primero...

Saludos!

[stock]

ok, si bien entendi quieres hacer una función que oculte DIVs y muestre uno solo a la vez, de ser asi checa este código.

Código Javascript:

//siempre es bueno encapsular tu codigo
var Helper = {
 
    //funcion que muestra un solo elemento y oculta el resto
    tabs  : function(actual,oculta){
        var actual = this.get(actual);     //elemento a desplegar
        
        for(var i=0;i<oculta.length;i++){
             this.get(oculta[i]).style.display = "none";   //se ocultan todos los demas
        }
        actual.style.display = "block";
    },
 
    //funcion que regresa un nodo del DOM por su ID
    //se puede mejorar mucho mas, pero para dejar las cosas
   //sencillas asi esta bien :)
     get   : function(id){
          return   document.getElementById(id);
     }
 
}
 
 
//cuando el documento este listo....
window.onload = function(){
 
     //ejecutando el codigo
     Helper.tabs("id4",["id1","id2","id3","id5","id6"]);
 
}
 

En tu documento HTML algo asi

Código HTML:

<div id="id1"></div>
<div id="id2"></div>
<div id="id3"></div>
<div id="id4"></div>
<div id="id5"></div>
<div id="id6"></div>
 

la funcion tabs recibe dos parametros, el primero es el identificador del div que quieres mostrar, el segundo parametro es un arreglo con los identificadores de los divs que quieres ocultar, de esta manera lo haras dinamico y no tienes que pasar N cantidad de parametros, solo son necesarios 2.

No he probado el código, solo lo escribí para darte un ejemplo tratando de seguir la lógica de tu primer post ;)

Saludos