Skip to main content
4 de 4
Olvidé este último error de su código
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

El problema principal que sufres te está apareciendo en la consola de depuración de Javascript cuando cargas la página:

Uncaught SyntaxError: Unexpected token 'else'

Es debido a que no has estructurado adecuadamente los bloques de código correspondientes al "if" y su "else".

Además, aunque no te salga ese error, no estás llamando a la función adecuada en el evento "onclick" del botón:

<input onclick="calculadora_grados_radianes()" type="button" value="Convertir">

Si te das cuenta, tu función se llama formulario_calculadora() y no calculadora_grados_radianes().

Seguimos con la definición de tus funciones, en las que intentas definir nueva funcionalidad de una clase que ya existe (Math), algo que está completamente desaconsejado. Es mejor tener tus propias funciones y clases.

Además, las estás llamando usando el nombre de función function como en este trozo:

resultado = function(radianes)

Cuando deberías haberla llamado usando el nombre con el que la has definido, Math.grados.

Por último, las variables que usas para obtener los grados y radianes llevan una letra mayúscula que luego no usas al pasar como parámetro los valores.

Éste podría ser tu código corregido:

<title>Conversor de grados a radianes</title>
<script type="text/javascript">

class Mates {
  static grados(radianes) {
    return radianes * 180 / Math.PI;
  }
  static radianes(grados) {
    return grados * Math.PI / 180;
  }
}

function formulario_calculadora() {
  /* Convertimos los campos de texto en datos de coma flotante */
  const radianes = parseFloat(document.f1.radianes.value);
  const grados = parseFloat(document.f1.grados.value);
  if (isNaN(radianes) === false) {
    /* Si radianes es correcto */
    resultado = Mates.grados(radianes);
  } else if (isNaN(grados) === false) {
    /* Si grados es correcto */
    resultado = Mates.radianes(grados);
  } else {
    /* En caso de que no hayan proporcionado ningún valor correcto */
    alert("Debe proporcionar algún valor a convertir");
    return;
  }
  document.f1.res.value = resultado;
}

</script>



<form name="f1">
  Grados: <input maxlength="5" size="5" name="grados" /><br/>
  Radianes: <input maxlength="5" size="5" name="radianes" /><br/><br/>
  <input onclick="formulario_calculadora()" type="button" value="Convertir" /><br/><br/>
  Resultado: <input maxlength="5" size="5" name="res" />
</form>

Para terminar, unos consejos:

  • Sangra adecuadamente el código. Así evitarás los problemas que has sufrido con los bloques de código.
  • Aunque sea opcional, finaliza todas las instrucciones con un punto y coma (;). Evitarás sorpresas en situaciones en las que pueda provocar errores.
  • No extiendas la funcionalidad de clases existentes. Crea las tuyas propias. Eso no solo evitará futuras colisiones (aunque sea poco probable), si no que cualquier persona que lea tu código sabrá dónde buscar la implementación del código que se está ejecutando.
  • Deberías usar la función parseFloat() para convertir la cadena almacenada en el campo de texto en un valor de coma flotante y gestionar cuándo devuelve un NaN para saber cuándo no es un valor correcto.
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64