Вопрос факториал числа в c++

[Alex1875]

почему не работает программа, где ошибка?
обязательно нужно что бы вводимое число не превышало 12.

#include
using namespace std;

int main()
{
int n;
int x=1;
cout<<"Vvedite n: ";
cin>>n;
while (n<=12)
{
for (int i=1; i<n+1; ++i)
x*=i;
}
cout<<n<<"!"<<" = "<<x<<endl;
system("pause");
}

 

[Sneil]

а не проще ли int fact(int i){i?return fact(i-1)*i:return 1;}

 

[Sdas]

а чем классический вариант не нравится? int fact(int x) { if(x != 0 && x > 0) return x * fact(x-1); else return 1; } Если без рекурсии int temp = 1; int fact_var; cin >> fact_var; while (fact_var > 0) { temp *= (fact_var--); /* temp = temp * fact_var; fact_var = fact_var - 1;*/ }

 

[антабус]

Хз

 

[1234554321]

Зачем этот цикл? while (n<=12) Других ошибок нет. Цикл for лучше написать так: for (int i=2; i <= n; ++i)

 

[Whispering]

Легче и удобнее сделать отдельную функцию для факториала, а условие прописать в основной программе
[THIDE="0"]#include <iostream>
using namespace std;

long double fact(int N)
{
if(N < 0) // если отрицательное число
return 0; // возвращаем 0
if (N == 0) // если ввели 0
return 1; // возвращаем факториал от 0
else
return N * fact(N - 1); //рекурсия
}

int main()
{
int N;
cout << "Enter digit: ";
cin >> N;
if (N>=13) cout << "N=<13"; else // условие
cout << "Factorial " << N << " = " << fact(N) << endl << endl; // fact(N) - call function
return 0;
}[/THIDE]