[SOLUCIONADO] Al comprobar valor de radiobutton encuentra valor sin estar seleccionado

[angel_dope]

Buenas a tod@s, tengo un misterio ante mi y no consigo avanzar. Resulta que tengo un formulario donde los usuario tienen que seleccionar una opción entre varios radiobuttons. Al darle a enviar, se les muestra un mensaje para que confirmen si realmente quieren hacerlo o no, pero claro, quiero que seleccionen una opción de forma obligatoria así que había pensado en comprobar primero el valor del radiobutton, si está en blanco, no sigue, y si tiene valor, ya pasa al siguiente paso.
El problema está en que, si no seleccionan ningún valor, y pulsan el botón, el javascript siempre me dice que tiene un valor (el del segundo de los radiobuttons) y no consigo saber por que razón pasa esto.
A ver si alguien me puede echar una manita, os pongo el código:

Código Javascript:

<script>
function enviar2() {
  var txt;
  var decision;
  decision = document.getElementById('decision').value;
 
  alert("entro " + decision); //mostrar valor elegido
  if (document.getElementById('decision').value <> ''){
      if (decision == 1) {
          txt = "Aceptar empresa y ACTUALIZAR datos en GDP";
          if (confirm("¿Estás seguro de querer " + txt + "?")) {
            document.form1.submit();
          } else {
            //txt = "You pressed Cancel!";
          }
      }
      if (decision == 2) {
          txt = "Aceptar empresa y NO actualizar datos en GDP";
          if (confirm("¿Estás seguro de querer " + txt + "?")) {
            document.form1.submit();
          } else {
            //txt = "You pressed Cancel!";
          }
      }
      if (decision == 3) {
          txt = "DESCARTAR empresa";
          if (confirm("¿Estás seguro de querer " + txt + "?")) {
            document.form1.submit();
          } else {
            //txt = "You pressed Cancel!";
          }
      }
      if (decision == 4) {
          txt = "Aceptar empresa e Incorporar a GDP";
          if (confirm("¿Estás seguro de querer " + txt + "?")) {
            document.form2.submit();
          } else {
            //txt = "You pressed Cancel!";
          }
      }
      if (decision == 5) {
          txt = "DESCARTAR empresa";
          if (confirm("¿Estás seguro de querer " + txt + "?")) {
            document.form2.submit();
          } else {
            //txt = "You pressed Cancel!";
          }
      }
  } else {
      alert("Seleccione una opción por favor");
  }
}
</script>
 

Código HTML:

form action="revisar_datos2.asp" method="post" name="form1" id="form1">
            <p>
              <input type="radio" name="decision" id="radio" value="1" required> 
              Aceptar empresa y ACTUALIZAR datos en GDP
            <input type="radio" name="decision" id="decision" value="2" required> Aceptar empresa y NO actualizar datos en GDP
            <input type="radio" name="decision" id="decision" value="3" required> Descartar empresa
            <input type="button" name="button" id="button" value="Botón" onclick="enviar2()">
          
            </p>
</form>
 

[Panino5001]

Sucede que estás usando el mismo valor de id para varios elementos. Y el id debería ser un identificador único. En lugar de usar document.getElementById('decision'), podrías comprobar el value del que esté chequeado de esta manera:

Código Javascript:

var decision=Number(Boolean(document.querySelector('input[name="decision"]:checked'))) && document.querySelector('input[name="decision"]:checked').value;
 

[angel_dope]

Iniciado por Panino5001

Sucede que estás usando el mismo valor de id para varios elementos. Y el id debería ser un identificador único. En lugar de usar document.getElementById('decision'), podrías comprobar el value del que esté chequeado de esta manera:

Código Javascript:

var decision=Number(Boolean(document.querySelector('input[name="decision"]:checked'))) && document.querySelector('input[name="decision"]:checked').value;
 

Ese comando no me ha servido, pero con lo que me has dicho ya me has indicado el camino y lo he resuelto de otra forma. Es un poco más largo, pero funciona, y como tengo prisa, así se queda de momento. Muchas gracias

Código Javascript:

var decision=0;
      decision1 = document.getElementById('decision1').checked;
      decision2 = document.getElementById('decision2').checked;
      decision3 = document.getElementById('decision3').checked;
      if (decision1==true){
          decision=1
      }
      if (decision2==true){
          decision=2
      }
      if (decision3==true){
          decision=3
      }   
 
          if (decision==0){
          alert("Debe seleccionar una opción");
          } else {
           if (decision == 1) {
             txt = "Aceptar empresa y ACTUALIZAR datos en GDP";
           ...