Skip to main content
se añadieron 169 caracteres en el cuerpo
Origen Enlace
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 cuales parentesiscuáles paréntesis?

Ten presente lo siguiente:

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

En las líneas:

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

window.onload=ejecuta()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;

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

¿Pero cuales parentesis?

Ten presente lo siguiente:

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

En las líneas:

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

window.onload=ejecuta();

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(){
    document.getElementsByTagName("p")[1].onclick=saludo;
}


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

window.onload=ejecuta;

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;
Origen Enlace
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 cuales parentesis?

Ten presente lo siguiente:

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

En las líneas:

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

window.onload=ejecuta();

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(){
    document.getElementsByTagName("p")[1].onclick=saludo;
}


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

window.onload=ejecuta;