[SOLUCIONADO] Java basico, factorial de numeros

[leoalvis]

Buenas noches a todos espero que estén bien:

Tengo un inconveniente con mi lógica al tratar de mostrar los factoriales del 1 al 5. Puedo listarlos, pero el acumulador no me muestra los resultados de manera correcta y no se "reinicia" para que me muestra el factorial de acuerdo al numero indicado. que me esta haciendo falta?. Anexo el código para que me puedan orientar. Muchas gracias.

import javax.swing.JOptionPane;
import javax.swing.JTextArea;
public class Ejercicio58 {
public static void main(String args[]){

int fact=1;

JTextArea areaSalida = new JTextArea();
areaSalida.setText("Numero\tFactorial\n");

for(int i=1;i<=5;i++){
for(int j=5;j>=1;j--)

fact=fact*j;
areaSalida.append(i+"\t"+fact+"\n");

}

JOptionPane.showMessageDialog(null, areaSalida,"Resultados",JOptionPane.INFORMATION_ME SSAGE);

}

}

[Omarchh]

Lamento decirte que tienes un for de mas, lo que estas haciendo es entrar 5 veces a calcular el factorial de 5 y cada que entras a realizar este calculo ya tienes datos calculados anteriormente. En pocas palabras lo que estas haciendo es calcular el factorial de 5 (1*2*3*4*5=120) con el for de la letra j y despues regresas al for de la i con fact=120, por lo tanto comienzas a multiplicar el 120 en vez del 1...

Mi recomendacion, utiliza solo un for...

for(int i=1;i<=5;i++){ //Este for o el de abajo salen sobrando

for(int j=5;j>=1;j--) //Este for deberia tener corchetes como estos {} por puro control

fact=fact*j; //puedes utilizar la i en vez de la j y el resultado seria el mismo

areaSalida.append(i+"\t"+fact+"\n");

}


Espero me hayas entendido

[jomaruro]

Hola Omarchh:

Te recomiendo que utilices la etiqueta HIGHLIGHT para resaltar el código ¿Ves la diferencia?.

Código Java:

for(int i=1;i<=5;i++){           //Este for o el de abajo salen sobrando
 
for(int j=5;j>=1;j--)             //Este for deberia tener corchetes como estos {} por puro control
 
fact=fact*j;                       //puedes utilizar la i en vez de la j y el resultado seria el mismo
 
areaSalida.append(i+"\t"+fact+"\n");
 
}
 

Saludos.

[leoalvis]

Claro que si, es claro. Muchas gracias.