Please Find out the logical error

[nkalambe]

#include<iostream.h>
#include<conio.h>
class factorial
{
int no,fact;
public:
void getdata();
void calculate();
void display();
};
void factorial::getdata()
{
cout<<"Enter a no"<<endl;
cin>>no;
}
void factorial::calculate()
{
int fact=1;
for(int i=1;i<=no;i++)
{
fact=fact*i;
}}
void factorial::display()
{
cout<<"Factorial of entered number is "<<fact;
}
void main()
{
clrscr();
factorial f;
f.getdata();
f.calculate();
f.display();
getch();
}

[Rabbit]

You have two variables named fact. The local one overrides the class one.