I am trying to do Factorial with Recursion and BigIntegers but eclipse is complaining about the BigInteger. I know the program is supposed to be simple but it is giving me headache. Here is the code.
import java.util.Scanner;
import java.math.BigInteger;
public class Factorial
{
public static void main(String[] args)
{
Scanner input = new Scanner(System.in);
System.out.println("Enter integer");
BigInteger n = input.nextBigInteger();
System.out.println("Factorial of " + n + " is " + fact(n));
}
public static int fact(BigInteger n)
{
if(n ==0)
{
return 1;
}
else
{
return n * fact(n-1);
}
}
}
In addition to what @aix mentioned regarding invoking the arithmetics on
BigInteger
- I can also see another issue with this code.Your method signature is
This is problemantic - factorial grows fast, so you are very likely to overflow the result.
I think what you realy wanted is:
which makes much more sense, since the return value should probably be the
BigInteger
(since it grows fast) and not the parameter, or possibly - both of them.Java doesn't support operator overloading. So + and == can't be supported for user-defined classes with one exception that java.lang.String supports +.
I believe you can't simply use arithmetic operator onto BigInteger object. Try to use their methods for arithmetic processes such as comparing, subtracting, multiplying etc.
References are given here
BigInteger
does not support comparison using==
and multiplication using*
. Instead, you have to call the appropriate methods of theBigInteger
class (equals()
andmultipy()
).Also note that there exist
BigInteger.ZERO
andBigInteger.ONE
.Finally, the return type of your
fact
method should beBigInteger
and notint
. Whether you want the argument to be of typeBigInteger
orint
is up to you.