Skip to main content
1 de 4
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) de manera incorrecta, algo que, además, está completamente desaconsejado. Es mejor tener tus propias funciones.

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.

Para terminar, un consejo: sangra adecuadamente el código para que evites los problemas que has sufrido, ya que el "else" te está fallando porque no has cerrado el bloque de código del "if" al que debería asociarse:

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

function grados(radianes) {
  return radianes * 180 / Math.PI;
}

function radianes(grados) {
  return grados * Math.PI / 180;
}

function formulario_calculadora()
{
  Radianes = document.f1.radianes.value;
  Grados = document.f1.grados.value;
  if (Grados == null) {
    resultado = grados(Radianes);
  } else {
    resultado = radianes(Grados);
  }
  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>

OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64