Nose que estoy haciendo mal... una ayudita en este pedazo de código plz

ras_chalo · #1 (**permalink**) 12/12/2010, 09:48

Hola, estoy intentando implementar este pedazo de código de javascript y no me resulta...

En fin, lo que intento hacer es mediante una llamada recursiva poder tener el resultado del factorial de un número ingresado por el usuario, el cual se mota del formulario (este nose si necesariamente se deba poner del formulario)...

Dejo el código:

Código Javascript:

Ver original<body>
<head>
 
<SCRIPT LANGUAGE="JavaScript">
function factorial(numero) {
   if (numero > 0) {
     return numero * (factorial(numero - 1))
   } else {
      return 1;
   }
}
 
</SCRIPT>
 
</head>
 
<body>
 
<div align="center">
<p>Ver el factorial de un n&uacute;mero de manera recursiva</p>
 
<form id="form" name="factorial" method="post" action="" onsubmit="factorial(this);">
    <table>    
    
    <tr align="left"><td>N&uacute;meros </td> <td><input name="factorial" type="text" id="recursivo" size="15" /></td></tr>   
    
    <tr><td> </td><td><br/><input type="submit"  value=" Ver Resultado " />
            </td></tr>
 </table>               
</form>
 
</div>
 
 
</body>

Bueno la verdad que nose si lo estoy haciendo bien. Por favor díganme como lo debo hacer...

Espero sus respuestas

Saludos

maycolalvarez · #2 (**permalink**) 12/12/2010, 09:53

cuado se envia un form (submit), se espera enviar la data del mismo al servidor, en tu caso no quieres eso, pero al usar onsubmit puedes cancelar ese envío al retornar false, el problema es que lo estás implementando mal, una opción seria:

onsubmit="alert(factorial(this.elements['factorial'].value)); return false;"

alert para mostrarlo como un mensaje
this.elements, porque this devuelve el formulario actual, pero ten en cuenta que el formulario puede tener n elementos, debes indicar cual y que propiedad

PD: recuerda que los métodos recursivos pueden minar completamente la memoria, en el caso del factorial puede hacerse con ciclos también

ras_chalo · #3 (**permalink**) 12/12/2010, 10:02

Cita:

Iniciado por maycolalvarez

cuado se envia un form (submit), se espera enviar la data del mismo al servidor, en tu caso no quieres eso, pero al usar onsubmit puedes cancelar ese envío al retornar false, el problema es que lo estás implementando mal, una opción seria:

onsubmit="alert(factorial(this.elements['factorial'].value)); return false;"

alert para mostrarlo como un mensaje
this.elements, porque this devuelve el formulario actual, pero ten en cuenta que el formulario puede tener n elementos, debes indicar cual y que propiedad

PD: recuerda que los métodos recursivos pueden minar completamente la memoria, en el caso del factorial puede hacerse con ciclos también

Implemente el pedazo de código que escribiste... pero no me resulta...
No te entiendo en la parte que dices: "pero ten en cuenta que el formulario puede tener n elementos, debes indicar cual y que propiedad"...

Donde debo indicarlo??

maycolalvarez · #4 (**permalink**) 12/12/2010, 10:15

donde indico this.elements, además de que tienes una función, un formulario y hasta un input con el mismo nombre factorial, en algunos navegadores suele dificultarle al interprete de js tener tantos elementos con el mismo nombre, intenta probar esto y analiza las diferencias de tu código contra este regularmente normalizado y estandarizado

Código Javascript:

Ver original<body>
<head>
 
<script type="text/javascript">
function getFactorial(numero) { //camelCase para funciones y el prefijo "get" para indicar su uso
   if (numero > 0) {
     return numero * (getFactorial(numero - 1))
   } else {
      return 1;
   }
}
 
</script>
 
</head>
 
<body>
 
<div align="center">
<p>Ver el factorial de un n&uacute;mero de manera recursiva</p>
 
<form id="form" name="form_factorial" method="post" action="" onsubmit="alert(getFactorial(parseInt(this.elements['input_factorial'].value))); return false;">
    <table>    
    
    <tr align="left"><td>N&uacute;meros </td> <td><input name="input_factorial" type="text" id="recursivo" size="15" /></td></tr>   
    
    <tr><td> </td><td><br/><input type="submit"  value=" Ver Resultado " />
            </td></tr>
 </table>               
</form>
 
</div>
 
 
</body>

notas:
-codigo HTML todo en minúsculas, incluido los tags <script></script>
-parseInt para convertir string en enteros

ras_chalo · #5 (**permalink**) 12/12/2010, 10:53

Cita:

Iniciado por maycolalvarez

donde indico this.elements, además de que tienes una función, un formulario y hasta un input con el mismo nombre factorial, en algunos navegadores suele dificultarle al interprete de js tener tantos elementos con el mismo nombre, intenta probar esto y analiza las diferencias de tu código contra este regularmente normalizado y estandarizado

notas:
-codigo HTML todo en minúsculas, incluido los tags <script></script>
-parseInt para convertir string en enteros

Wow muchas gracias !!
no esperaba que lo hicieras
Me queda mucho por aprender de este lenguaje
gracias por la ayuda, me sirvió de mucho y me servirá para seguir haciendo mi trabajo..

saludos!

maycolalvarez · #6 (**permalink**) 12/12/2010, 10:57

Cita:

Iniciado por ras_chalo

Wow muchas gracias !!
no esperaba que lo hicieras
Me queda mucho por aprender de este lenguaje
gracias por la ayuda, me sirvió de mucho y me servirá para seguir haciendo mi trabajo..

saludos!

al contrario, tu hiciste bastante y solo necesitabas un empujón, te recomiendo que leas este manual, suerte y que aprendas mucho

_cronos2 · #7 (**permalink**) 12/12/2010, 12:17

Cita:

Iniciado por maycolalvarez

cuado se envia un form (submit), se espera enviar la data del mismo al servidor, en tu caso no quieres eso, pero al usar onsubmit puedes cancelar ese envío al retornar false, el problema es que lo estás implementando mal, una opción seria:

onsubmit="alert(factorial(this.elements['factorial'].value)); return false;"

alert para mostrarlo como un mensaje
this.elements, porque this devuelve el formulario actual, pero ten en cuenta que el formulario puede tener n elementos, debes indicar cual y que propiedad

PD: recuerda que los métodos recursivos pueden minar completamente la memoria, en el caso del factorial puede hacerse con ciclos también

¿No sería más fácil así?

Código HTML:

Ver original<form action="javascript:alert(factorial(this.elements['factorial'].value))">

Saludos (:

maycolalvarez · #8 (**permalink**) 12/12/2010, 14:35

Cita:

Iniciado por _cronos2

¿No sería más fácil así?

Código HTML:

Ver original<form action="javascript:alert(factorial(this.elements['factorial'].value))">

Saludos (:

hay muchas formas, todo depende de como lo desees implementar