Skip to main content
2 de 2
se añadieron 169 caracteres en el cuerpo
Pablo Gutiérrez
  • 2.4k
  • 2
  • 8
  • 14

El problema está en los paréntesis como te escribieron en los comentarios de tu pregunta duplicada.

¿Pero cuáles paréntesis?

Ten presente lo siguiente:

  • Cuando declaras una función, debe tener paréntesis function ejecuta(){...}
  • Cuando quieres ejecutar una función, debes colocarle paréntesis ejecuta();
  • Cuando asignas una función a una variable (o como manejador de evento en este caso) NO le pongas paréntesis (porque se ejecuta)

En las líneas:

document.getElementsByTagName("p")[1].onclick=...;

window.onload=...;

Tanto .onclick como .onload son eventos a los que les quieres asignar la función que se va a ejecutar cuando el evento ocurra, en estos casos va sin parentesis.

De esta forma, el Javascript corregido quedaría de la siguiente manera:

function ejecuta(){
    //                             sin parentesis  |
    //                                             V 
    document.getElementsByTagName("p")[1].onclick=saludo;
}


function saludo(){  
    
    alert("Hola, que hay de nuevo");
    
}


//  sin parentesis  |
//                  V 
   window.onload=ejecuta;
Pablo Gutiérrez
  • 2.4k
  • 2
  • 8
  • 14