Duda acomodando un script

cyber_dark · #1 (**permalink**) 19/08/2011, 16:48

Saludos, estaba leyendo sobre como acomodar de una manera más estetica los scripts y esta me llamo mucho la atencion:

Código:

<html>
<head>
<script language="JavaScript">
function muestraMensaje() {
alert('Gracias por pinchar');
document.getElementById("pinchable").onclick = muestraMensaje;
}
 
</script>
</head>
<body>

<input id="pinchable" type="button" value="Pinchame y verás" />

</body>
</html>

El problema es que no funciona, si lo pongo de esta forma funciona, pero lo que busco es como en el ejemplo de arriba.

Código:

<html>
<head>
<script language="JavaScript">
function muestraMensaje() {
alert('Gracias por pinchar');
}
 
</script>
</head>
<body>

<input id="pinchable" type="button" value="Pinchame y verás" onclick="muestraMensaje()"/>

</body>
</html>

El script original solo es asi:

Código:

function muestraMensaje() {
alert('Gracias por pinchar');
}
document.getElementById("pinchable").onclick = muestraMensaje;
<input id="pinchable" type="button" value="Pinchame y verás" />

No se si lo estoy acomodando mal, o el codigo de por si no funciona, espero puedan ayudarme.

maycolalvarez · #2 (**permalink**) 19/08/2011, 16:52

si te fijas en el primer código, la asignación del evento está dentro de la función, así nunca te funcionará, otro detalle es que no puedes asignar el evento si el DOM no se ha cargado, por lo cual puedes hacerlo en el evento onload del body o buscar una función que detecte el DOM cargado.

cyber_dark · #3 (**permalink**) 19/08/2011, 17:16

Gracias por responder amigo, si te fijas la asignacion que mencionas debe estar fuera de la funcion ya que document.getElementById("pinchable").onclick = muestraMensaje; llama a funcion, y sobre lo otro, el codigo es muy pequeño, no creo que se necesite utilizar un evento, para saber si ya cargo o no.

cyber_dark · #4 (**permalink**) 19/08/2011, 17:36

Me equivoque tenias razon sobre crear una funcion para saber si ya cargo, hice esto y funciona, pero quisiera saber si es correcto crear una funcion dentro de una funcion de esta manera

function cargar() {
function muestraMensaje() {
alert('Gracias por pinchar');
}
}