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;