Hi! Can anyone check this code and tell me why it doesn't work ?

Question

#include <iostream>
using namespace std;
/* this program is to find factorial of any number*/
int main() {
int factorial,n,i;
cout<< "enter any positive integer number:"<<endl;
cin>>n;
for (i=1;i<=n;i++)
{factorial*=i};

cout << "factorial of "<<n<<"="<<factorial<<endl;
return 0;
}

Answer 1

umm also.. for loop doesn't need semicolon to end it after curly brackets..
{factorial*=i;}

Answer 2

thaks

Answer 3

you forgot to initialize variable factorial with 1(in ur code factorial has garbage value)

Answer 4

it should be
{factorial *=I;} //semi colon should be inside the bracket

and you must declare value of factorial as 1
otherwise the some value of factorial will be used for multiplication