Skip to main content
se añadieron 259 caracteres en el cuerpo
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Al ponerPrimeramente, tienes que dar un id al formulario, e indicarlo en el código asíselector, pues aquí realmente no estás seleccionado nada (ver la documentación):

var formulario = document.getElementById;

formulario.addEventListener("submit"Luego, llamar());al poner el código así:

formulario.addEventListener("submit", llamar());

En vez de asignar la función al listener, lo que haces es que llamas a la función.

Además, conviene dar un Debes quitarle los paréntesis id() al formulario, e indicarlo en el selector.

Otra cosa, si quieres evitar que la página se recargue cuando envíes el formulario (por ejemplo en el caso de Ajax), pásale el evento a llamar y ejecuta dentro de ella preventDefault(). Si quieres que se recargue le quitas esa línea.

var formulario = document.getElementById("miForm");
formulario.addEventListener("submit", llamar);

function llamar(e) {
  e.preventDefault();
  alert("Presionado");
}
<form action="" name="formulario" id="miForm">
  <input type="radio">
  <input type="submit">
</form>

Al poner el código así:

formulario.addEventListener("submit", llamar());

En vez de asignar la función al listener, lo que haces es que llamas a la función.

Además, conviene dar un id al formulario, e indicarlo en el selector.

Otra cosa, si quieres evitar que la página se recargue cuando envíes el formulario (por ejemplo en el caso de Ajax), pásale el evento a llamar y ejecuta dentro de ella preventDefault(). Si quieres que se recargue le quitas esa línea.

var formulario = document.getElementById("miForm");
formulario.addEventListener("submit", llamar);

function llamar(e) {
  e.preventDefault();
  alert("Presionado");
}
<form action="" name="formulario" id="miForm">
  <input type="radio">
  <input type="submit">
</form>

Primeramente, tienes que dar un id al formulario, e indicarlo en el selector, pues aquí realmente no estás seleccionado nada (ver la documentación):

var formulario = document.getElementById;

Luego, al poner el código así:

formulario.addEventListener("submit", llamar());

En vez de asignar la función al listener, lo que haces es que llamas a la función. Debes quitarle los paréntesis ()

Otra cosa, si quieres evitar que la página se recargue cuando envíes el formulario (por ejemplo en el caso de Ajax), pásale el evento a llamar y ejecuta dentro de ella preventDefault(). Si quieres que se recargue le quitas esa línea.

var formulario = document.getElementById("miForm");
formulario.addEventListener("submit", llamar);

function llamar(e) {
  e.preventDefault();
  alert("Presionado");
}
<form action="" name="formulario" id="miForm">
  <input type="radio">
  <input type="submit">
</form>

Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Al poner el código así:

formulario.addEventListener("submit", llamar());

En vez de asignar la función al listener, lo que haces es que llamas a la función.

Además, conviene dar un id al formulario, e indicarlo en el selector.

Otra cosa, si quieres evitar que la página se recargue cuando envíes el formulario (por ejemplo en el caso de Ajax), pásale el evento a llamar y ejecuta dentro de ella preventDefault(). Si quieres que se recargue le quitas esa línea.

var formulario = document.getElementById("miForm");
formulario.addEventListener("submit", llamar);

function llamar(e) {
  e.preventDefault();
  alert("Presionado");
}
<form action="" name="formulario" id="miForm">
  <input type="radio">
  <input type="submit">
</form>