2

He seguido esta pregunta sobre HTML y Javascript: Convertir radianes a grados

He estado probando y buscando ejemplos de Internet y al final he creado un código por mi cuenta, pero no acaba de funcionar.

Lo adjunto a continuación por si alguien sabe cómo solucionarlo:

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

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

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

function formulario_calculadora(){
Radianes = document.f1.radianes.value
Grados = document.f1.grados.value
if (Grados == null) {
resultado = function(radianes)
else {
resultado = function(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="calculadora_grados_radianes()" type="button" value="Convertir"> 
<br><br>Resultado : <input maxlength="5" size="5" name="res"> 
</form>

2
  • 2
    Sé más específico: ¿qué no funciona? Recordá que Stack Overflow en español no es un servicio de depuración de código; además, aquella pregunta también terminó cerrada ya que tampoco has dado más detalles. Leé Cómo preguntar.
    – padaleiana
    Commented el 5 ago. 2021 a las 11:45
  • 2
    Por otra parte, estás mencionando la pregunta de otro usuario... ¿En realidad quisiste decir que te basaste en esa pregunta?
    – padaleiana
    Commented el 5 ago. 2021 a las 11:47

1 respuesta 1

5

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.
9
  • 2
    Gran respuesta. Dos apuntes: (1) La condición del if debería comparar con la cadena vacía '' y no con null para funcionar correctamente; (2) Desconozco las convenciones de javascript pero en java los nombre de los métodos se recomienda que comiencen por verbos. Ya que empleas una clase, quizas "convertirRadianesEnGrados" sea algo largo pero queda más claro qué es lo que hace el método si ponemos eso en lugar de solo "grados".
    – RubioRic
    Commented el 5 ago. 2021 a las 12:39
  • 2
    Las convenciones o guías de estilo las fijan las guías de estilo de cada proyecto, no es una regla fija o estándar. En javascript, por ejemplo, el uso del punto y coma al final es requerido en la guía de estilo de google, pero están prohibidas en el proyecto npm. Hay muchísimos ejemplos más en los que no quiero entrar, pero el denominador común de casi todas las guías es llevar un buen sangrado del código, entre otros. Commented el 5 ago. 2021 a las 12:53
  • Por ultimo, sería casi mejor hacer una conversión a coma flotante con parseFloat() y gestionar cuándo devuelve un NaN, en vez de delegar el trabajo a una conversión de cadenas. Quizá sea demasiado prematuro explicarle estas cosas al OP, pero se podría modificar el código para mejorar ese aspecto. Commented el 5 ago. 2021 a las 12:55
  • Muchísimas gracias, el código funciona muy bien. Solo tengo una última pregunta, simplemente por curiosidad: ¿existe alguna función de JavaScript que permita la entrada y salida de expresiones matemáticas más complejas? Aplicado a este código, lo que quiero decir es que permitan la entrada en radianes. O por ejemplo, en algún otro código que se pueda expresar un resultado en formato fraccionario... Muchas gracias.
    – Oriol
    Commented el 5 ago. 2021 a las 14:04
  • 1
    @Oriol , existen multitud de bibliotecas de código que podrían ayudarte a conseguir lo que deseas. Creo que podrías empezar echando un vistazo a Math.js. Commented el 6 ago. 2021 a las 5:26

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.