Este script está bien pero no funciona

[SeaPirates]

Hola, he creado el siguiente script, que en el editor funciona pero en la web no.

Código Javascript:

<script language="JavaScript" type="text/javascript">
function showinfo(type){
var estado = document.getElementById(''+type+'').style.display;
if(estado == "none"){
document.getElementById(''+type+'').style.display='block';
}
if(estado == "block"){
document.getElementById(''+type+'').style.display='none';
}
}
</script>
 

Código HTML:

<a onclick="showinfo(1);" href="#">» más</a>
<div id="1" style="display:none;">Contenido</div>
 

Que razón hay para que no funcione? Esque no me recoje la variable estado no se porqué.

[caricatos]

Hola:

No están permitidos los identificadores que no sean cadenas, y éstas no pueden empezar por un número.
Otra cosa que no le encuentro sentido son las comillas:

document.getElementById(''+type+'').style.display='block';

es mejor:

document.getElementById(type).style.display='block';

Saludos

[SeaPirates]

Igualmente no funciona:

Código Javascript:

<script language="JavaScript" type="text/javascript">
function showinfo(type){
var estado = document.getElementById(type).style.display;
if(estado == "none"){
document.getElementById(type).style.display='block';
}
if(estado == "block"){
document.getElementById(type).style.display='none';
}
}
</script>
 

<a onclick="showinfo(s_1);" href="#">» más</a>
<div id="s_1" style="display:none;">Contenido</div>

[marlanga]

Iniciado por caricatos

Hola:

No están permitidos los identificadores que no sean cadenas, y éstas no pueden empezar por un número.
Otra cosa que no le encuentro sentido son las comillas:

document.getElementById(''+type+'').style.display='block';

es mejor:

document.getElementById(type).style.display='block';

Saludos

Es obvio. type era un número, y lo fuerza a reconvertirse en cadena añadiéndole cadenas vacías. Es un error, pero "en su mundo" fue un buen intento.

[caricatos]

Hola:

Si no entrecomillas el parámetro, javascript lo considera una variable, que seguramente sea undefined.

onclick="showinfo('s_1')"

Saludos

[SeaPirates]

Iniciado por caricatos

Hola:

Si no entrecomillas el parámetro, javascript lo considera una variable, que seguramente sea undefined.

onclick="showinfo('s_1')"

Saludos

Ahora si, gracias a todos