Factorial console program

hey guys i know all of can help me here. Program is about user enters a number at run time and output should give the factorial.
thanx in advance.
I feel error is really simple but couldnt sort it out.


import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.io.IOException;
public class NumberFactorial
{
public static void main(String args[])
{
int number;
try
{
BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
System.out.println("Enter the number");
number = Integer.parseInt(br.readLine());

int factorial = number;
for(int i =(number-1);i>1;i--)
{
factorial=factorial*i;
}
}
catch(NumberFormatException ne)
{
System.out.println("Number Format Exception"+ne);
System.exit(0);
}
catch(IOException ioe)
{
System.out.println("Input Exception Error"+ioe);
System.exit(0);
}
System.out.println("Factorial of number is:" +factorial);
}
}

Code:

---

System.out.println("Factorial of number is:" +factorial);

---

just move this line a bit up into the try block of ur code.

please use code tags.. when u post ur code.

Your formula is wrong: by definition 0! == 1.

kind regards,

Jos

Quote:

---

Your formula is wrong: by definition 0! == 1.

---

Code:

---

try
{
BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
System.out.println("Enter the number");
number = Integer.parseInt(br.readLine());
int factorial=1;
while(number>0)
{
        factorial = factorial * number;
        number--;
}
System.out.println("Factorial of number is:" +factorial);
}

---

ur try block must look something like this.

How about a simple recursive method?

Code:

---

int factorial(int n) {
  if(n == 0) return 1;
  return factorial(n-1)*n;
}

---

Quote:

---

Originally Posted by m00nchile

How about a simple recursive method?

Code:

---

int factorial(int n) {
  if(n == 0) return 1;
  return factorial(n-1)*n;
}

---

---

Great; factorial(-1)

kind regards,

Jos

It was just a reccomendation, I know it should check for values that it can't calculate.