#include<iostream>

using namespace std;

int fakultät(int zahl)
{
	while (zahl>1)
	{
		int fak = 1;
		fak = fak * zahl;
		zahl--;
	}
	return fakultät(zahl);
}
int main()
{
	int zzahl;
	cin >> zzahl;
	cout << endl << fakultät(zzahl) << endl;
}

EDIT:
1. fak gibt es in der main-Funktion nicht. => Compiler-Fehler

2. Deine Funktion ruft sich immer wieder selbst auf. Das endet in einer Endlosrekursion. Kann ja so nicht funktionieren. Denke noch mal in Ruhe über deinen Ablauf nach.

3. In der main musst du deine Variable nicht zzahl nennen. Du scheinst davon auszugehen, dass der Name zahl nur einmal im Programm vorkommen darf. Das stimmt nicht. Eine lokale Variable zahl darf natürlich in der main existieren und hat nichts mit einer lokalen Variablen zahl in einer anderen Funktion zu tun.

Mehr nicht.

EDIT: Glaube sogar, dass ich weiß, was du meinst. Ich werd mal ne Runde drüber grüebln

Aufgabe ist es, die Fakultät als Rekursion anzugeben

Sorry, du berechnest die Fakultaet iterativ und rufts die Funktion rekursiv auf. Das ist nicht die Loesung.

#include<iostream>

using namespace std;

int fakultaet(int zahl)
{
	if (zahl>1)
		return zahl * fakultaet(zahl - 1);
	else
		return 1;
}
int main()
{
	int zahl;
	cin >> zahl;
	cout << endl << fakultaet(zahl) << endl;
}

ist bei mir jetzt rausgekommen nach ein wenig hin und her probieren.
Ergebnisse werden auch richtig zurückgegeben etc, alles wunderbar.
Hatte grade noch die Frage warum return 1 und nicht return 0, aber hat sich von selbst erklärt
Hoffe alle sind zufrieden damit
Kurzes Statement und/oder Verbesserungsvorschläge sind gerne gesehen.

EDIT: Umlaute sind jetz auch weg

ist bei mir jetzt rausgekommen nach ein wenig hin und her probieren.

Aha, du hast es also selbst gemacht.

xStrykex schrieb:

Hatte grade noch die Frage warum return 1 und nicht return 0, aber hat sich von selbst erklärt

Du hattest noch die Frage, warum du in deinem eigenen Code return 1 und nicht return 0 geschrieben hast? Na klar... Copy&Paste ftw! Hoch lebe das Internet!

Im Zweifelsfall bitte auch folgende Hinweise beachten:
C/C++ Forum :: FAQ - Sonstiges :: Wohin mit meiner Frage?

Dieses Posting wurde automatisch erzeugt.

EDIT: "unsigned" nicht zu vergessen

EDIT2:

#include<iostream>

using namespace std;

unsigned long long fakultaet(unsigned long long zahl)
{
	if (zahl>1)
		return zahl * fakultaet(zahl - 1);
	else
		return 1;
}
int main()
{
	unsigned long long zahl;
	cin >> zahl;
	cout << endl << fakultaet(zahl) << endl;

	cin.get();
}