因子編程挑戰混淆

Question

我最近完成了一個編程挑戰，挑戰我們獲得2015的階乘。在計算階乘之後，您將總和中的所有數字（在本例中爲BigInteger）進行求和。

完成後，我取得的結果是：23517

然而，當我提交給了教授他說正確的結果是：26532

我似乎無法找到問題根本。我已經交叉檢查了多個來源，到目前爲止，一切都似乎表明我是對的。任何人都可以找到我的代碼出錯的地方？

import java.math.BigInteger; 

class toymeister { 

    public static void main(String[] args){ 
     System.out.println("The sum is: " + SumOf(FactorialOf(2015))); 
    } 

    public static BigInteger FactorialOf(int x){ 
     BigInteger Answer, intConvert; 

     intConvert = new BigInteger("0"); 
     Answer = new BigInteger("1");  

     for(int i=1; i<=x;i++){ 
      intConvert = intConvert.valueOf(i); 
      Answer = Answer.multiply(intConvert); 
     } 

     System.out.println("The Factorial Of " + x + " is: " + Answer); 

     return Answer; 
    } 

    public static BigInteger SumOf(BigInteger y){ 
     BigInteger Sum = new BigInteger("0"); 

     while(y.compareTo(BigInteger.ZERO) > 0) { 
      Sum = Sum.add(y.mod(BigInteger.TEN)); 
      y = y.divide(BigInteger.TEN); 
     } 

     return Sum; 
    } 
} 

Answer 1

答案23517 is correct。不過，你可以簡化代碼：

public static BigInteger factorialOf(int x) { 
    BigInteger answer = BigInteger.ONE; 
    for(int i = 1; i <= x; i++) { 
     answer = answer.multiply(BigInteger.valueOf(i)); 
    } 
    System.out.println("The Factorial Of " + x + " is: " + answer); 
    return answer; 
} 

public static int sumOf(BigInteger y) { 
    String digits = y.toString(); 
    int sum = 0; 
    for(int i = 0, n = digits.length(); i<n; i++) sum += digits.charAt(i)-'0'; 
    return sum; 
} 

，或者使用Java 8：

public static BigInteger factorialOf(int x) { 
    return IntStream.rangeClosed(1, x).mapToObj(BigInteger::valueOf) 
     .reduce(BigInteger::multiply).orElse(BigInteger.ONE); 
} 
public static int sumOf(BigInteger y) { 
    return y.toString().chars().map(i -> i-'0').sum(); 
}