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

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.

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.

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.

Mejoras en el código y las comprobaciones de los valores de entrada
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64
<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;value);
  const grados = parseFloat(document.f1.grados.value;value);
  if (gradosisNaN(radianes) ===== nullfalse) {
    /* 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>
  • 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.
<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() {
  const radianes = document.f1.radianes.value;
  const grados = document.f1.grados.value;
  if (grados == null) {
    resultado = Mates.grados(radianes);
  } else {
    resultado = Mates.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>
  • 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.
<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>
  • 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.
Reestructurado con los consejos al final
Origen Enlace
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 y clases.

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Éste podría ser tu código del "if" al que debería asociarsecorregido:

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

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

function formulario_calculadora()
  {
  Radianesconst radianes = document.f1.radianes.value;
  Gradosconst grados = document.f1.grados.value;
  if (Gradosgrados == null) {
    resultado = Mates.grados(Radianesradianes);
  } else {
    resultado = Mates.radianes(Gradosgrados);
  }
  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.

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>

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.

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

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.
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64
Loading