[SOLUCIONADO] Problema con argumentos

[kenproxd]

Hola,

La verdad soy un novato en js así que pido disculpas desde ya si la solución a este problema es muy obvia.

Tengo la siguiente función:

Código Javascript:

<script type="text/javascript">
    function despliega(valor) {
        if(document.getElementById("+valor+").style.visibility=='visible') {
            document.getElementById("+valor+").style.visibility='hidden';
        } else {
            document.getElementById("+valor+").style.visibility='visible';
        }
    }
</script>
 

Intento llamarla al hacer clic en un link, de la siguiente manera:

Código HTML:

<a href="#" onclick="despliega("register-container");" title="Despliega/Oculta el formulario de registro">Registrate</a>
 

Lo cual no da ningún resultado. Al principio, cuando la escribí, no declaraba ningún argumento (lo que en la función llamo 'valor'), simplemente indicaba el id del div con el que quería trabajar la función y funcionaba perfectamente.

Ahora intento acomodarlo de esta manera porque necesito usar la misma función en más de un link y no me parece nada práctico crear una varias funciones con distintos nombres y que hagan lo mismo.

Espero vuestra ayuda.

[ocp001a]

No podrías hacer el llamado de la función así, por la sintáxis que usas:
onclick="despliega("register-container");"

debe ser

onclick="despliega('register_container');"

es decir no puedes tener comillas dentro de comillas

Hasta donde recuerdo, los id de los objetos no deberían contener el guión medio, aunque quizá me equivoque, ya que nunca los uso así.

[kenproxd]

Gracias por responder ocp001a,

He cambiado el nombre del div a register_container y también las dobles comillas por comillas simples tal como me indicaste, sin embargo sigo sin tener resultado alguno.

Yo creo que el problema está al llamar el argumento en la function de la forma en la que lo hago ("+valor+"), he buscado por todas partes como hacerlo pero no encuentro respuesta alguna.

[ocp001a]

bueno, sí, además faltaba eso:

if(document.getElementById(valor).style.visibility =='visible')


cuando recibes el parámetro valor, dentro de la función ya es una cadena, por lo que no hay que meterla dentro de otra.

Código Javascript:

<script type="text/javascript">
    function despliega(valor) {
        if(document.getElementById(valor).style.visibility=='visible') {
            document.getElementById(valor).style.visibility='hidden';
        } else {
            document.getElementById(valor).style.visibility='visible';
        }
    }
</script>
 

[kenproxd]

Muchas gracias! ya está solucionado.

Te comento que también probé con el antiguo nombre del div (con el guión al medio) y no dio ningún problema.

Saludos!