if文のループでエラーが発生しますが、コンパイルされてエラーなしで終了し、期待される結果を表示しません。

Question

質問を解決している間（コメントにはコード内にあります）if文（結果を表示）は機能しません。エラーを理解するのを手伝ってください。それはコンパイルされて実行されますが、出力は表示されません。

/* 
* (Calculating the Value of π) Calculate the value of π from the infinite series 
     4  4  4  4  4 
π = 4 – --- + --- – --- + --- – ------ + ... 
     3  5  7  9  11 
Print a table that shows the value of π approximated by computing the first 200,000 terms of this 
series. How many terms do you have to use before you first get a value that begins with 3.14159? 
*/ 
public class ValueOfPi 
{ 
    public static void main(String[] args) 
    { 
     int i,count=0; 
     double pi=0.0,n=1.0,PI=3.14159; 
     for(i=0;i<200000;i++) 
     { 
      count++; 
      if(count%2!=0) 
      { 
       pi=pi+(4/n); 
      } 
      else if(count%2==0) 
      { 
       pi=pi-(4/n); 
      } 
      n=n+2; 
      if(pi==PI) 
      { 
       System.out.printf("terms=%d  PI=%f\n",count,pi); 
       break; 
      } 
     } 
    } 
} 

Answer 1

if文が機能しないのは、浮動小数点数が等しいかどうかを比較するためです。理由は、What Every Computer Scientist Should Know About Floating-Point Arithmeticページに説明されています。

OPの質問を読み返した後、私はまずpiの計算値を8文字以上の文字列に変換し、最初の7文字を "3.14159"と比較します。

Answer 2

最初の浮動小数点は不正確です - （負の）べき乗の近似値 - （3.14159も含む）であり、計算されたpiは指定された値よりも上または下です。

%nも使用します。\nはLinuxの行末であり、Windowsではなく、\r\nです。これにより、ラインバッファのフラッシングが出力されるのを防ぐことができました。 （わからない、私はLinuxを持っている）。

 if (Math.abs(pi - PI) < 1.0E-5) 
     { 
      System.out.printf("terms=%d  PI=%f%n",count,pi); 
      break; 
     } 

また、近似のためにMath.PIを使用することもできます。あなたは今度はPIよりも正確なpiが0.00001の不正確さを必要とする危険性があります。

==または小さすぎるepsマージンを使用すると、0.00000001のようにほぼ無限のループが発生する可能性があります。