Fakultät aus Zahl bilden

[Poines]

Hallo Forum,

ich versuche im Folgenden die Fakultät aus einer Zahl zu bilden, komme aber nicht zum gewollten Ergebnis. Ziel ist die Fakultät von 5 zu bilden. Es wäre gut wenn mir dabei jemand helfen könnte

public class Functions {

	
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		
		int ergebnis = 5;
		ergebnis = faculty(ergebnis);
		
		System.out.println(ergebnis);
	
		}
	
	public static int faculty(int n){
		
		
		
		for(int zahl = 0; zahl > n; zahl++){
			
		n = (n * (n-1));
			
		}
		
		
		return n;
		
	}
	



	}

 

[Flown]

Da happerts ja an einigen Stellen bei dir:

heißt jetzt, dass der Schleifenkopf schon nicht stimmt:

Wenn angenommen n = 10 ist =>

Eintritt in die Schleife:
zahl = 0
Abbruchbedingung 0 > 10 -> Schleife wird übersprungen.

Des weiteren ist die Folge für n=5:

1 * 2 * 3 * 4 * 5 = 120

Eine einfache for-Schleife liefert dir die Werte wenn du sie von i = 1; i < 6 laufen lässt. Dann multiplizierst du diese i-Werte zu deinem Ergebnis (wobei 0! = 1 ist!).

[TIPP]Das Ergebnis kann schon mit 1 initialisiert werden.[/TIPP]

 

[Poines]

Erstmal schonmal Danke.

Die For-Bedingungen lauten dann also:

for(int zahl = 1; zahl < n; zahl++)

oder?

Aber wie genau drücke ich die Fakultät dann aus?

 

[gescom]

Wenn du es nicht rekursiv machen willst, speicherst du das zwischenergebnis in einer variablen? Bei jedem iterationsschritt multiplizierst du und gibst dann Ergebnis zurück.

 

[Poines]

Ok ich habs. Vielen Dank

 

[Flown]

Also um das abzuschließen hier der Code (falls das jemand mal suchen sollte):

Ich habe für die Berechnungen BigInteger genommen, da bei kleinen Eingaben schon große Werte rauskommen können:

Spoiler: Faculty

public static BigInteger faculty(int n) {
	BigInteger result = BigInteger.ONE;
	for (int i = 1; i <= n; i++) {
		result = result.multiply(BigInteger.valueOf(i));
	}
	return result;
}