這個程序的邏輯有什麼問題？

Question

我有這段代碼的問題。它是一個計算程序的一部分，它計算你在某些計算器上調用nPr的內容。它編譯沒有錯誤，但每次運行它並設置n = 0我收到一個看似無盡的錯誤消息列表。爲什麼不能打印出「錯誤！」並終止程序像我編程它？

{ 
    System.out.print("n = "); 
    int n = j.nextInt(); 
    System.out.print("r = "); 
    int r = j.nextInt(); 
    long n_fact = factorial(n); 
    if (((n - r) <= 0) || (n <= 0) || (r < 0)) 
    { 
     System.out.println("Error!"); 
     System.exit(0); 
    } 
    else 
    { 
     long nr_fact = factorial(n - r); 
     long nPr = n_fact/nr_fact; 
     NumberFormat f = NumberFormat.getNumberInstance(); 
     System.out.println("nPr = " + f.format(nPr)); 
    } 
} 

Answer 1

打電話給factorial(n)，然後檢查是否n > 0。移動線

long n_fact = factorial(n); 

到else塊內部。

Answer 2

移動這一行else塊：

long n_fact = factorial(n); 

，因爲它是越來越觸發even when n=0 or n < 0，因爲你是上面的調用，您不希望以後檢查n > 0。 我懷疑這個（n < = 0）是否在你的factorial()方法中正確處理。

備選地穿上n檢查和處理在factorial(n)方法即返回1，如果n = 0和錯誤輸出（擲excepion）如果n < 0;