2つの値は商を生成しません

Question

私は現在学校への割り当てを行っていますが、Math.random（）を使用してランダムな値を取得するプログラムを作成する必要があります。 「頭」または「尾」。これを10回行う必要があります。次に、プログラムの出力が何パーセントになるか、何回出力されるかを調べる必要があります。ただし、正しく動作していません。それは常にヘッド/テールがトーセンスの0％であったことを出力します。誰かが理由を説明できますか？

public class HeadsTails 
{ 
    public static void main(String[] args) 
    { 
     int v; 
     double i; 

     int heads = 0, tails = 0; 

     double headsPercent, tailsPercent; 

     for(v = 1; v <= 10; ++v) 
     { 
      i = Math.random(); 
      if(i <= 0.5) 
      { 
       System.out.println("Heads"); 
       heads = heads + 1; 
      } 
      else if(i > 0.5) 
      { 
       System.out.println("Tails"); 
       tails = tails + 1; 
      } 
     } 
     headsPercent = (heads/10) * 100; 
     tailsPercent = (tails/10) * 100; 
     System.out.println("Heads were " + headsPercent + "% of the tosses."); 
     System.out.println("Tails were " + tailsPercent + "% of the tosses."); 
    } 
} 

私はまた、プログラムを正しく機能させるだけでなく、改善することができます。

Answer 1

問題は整数の数学です。 headsとtailsので、両方のint次のとおりです。

heads/10 

は... 9/10てint -land、1/10、2/10、内のすべての0をしているため、ほとんどの場合、あなたのケースで0になりますint結果を、得られます。あなたが他のものを手に入れる唯一の時間は、すべてのロールが頭部だった場合、またはすべてが尾部だった場合です。10/10は1です。数学を行う前にdoubleへ

出演：

headsPercent = ((double)heads/10) * 100; 
tailsPercent = ((double)tails/10) * 100; 

そのfixes the problemを行います。

---

サイド注1：あなたが+=、例えばを使用して変数に追加することができます。

heads += x; 

そしてもちろん、何を追加していること1で、あなただけのインクリメント演算子を使用することができ、接頭語のいずれか：

++heads; 

または接尾

heads++; 

（headsとtailsを更新するという点で、それはどの問題ではありません。）

サイド注2：あなたはelse if (i > 0.5)を必要としません。ロジックについて考えるなら、if (i <= 0.5) ... else ...です。つまり、if (i > 0.5)の部分は必要ありません。elseになると、iは間違いなく> 0.5になります。

そして、あなたはそれを行う場合、あなたももうiは必要ありません：あなたはintiger分裂を行っている

for(v = 1; v <= 10; ++v) 
{ 
    if(Math.random() <= 0.5) 
    { 
     System.out.println("Heads"); 
     ++heads; 
    } 
    else 
    { 
     System.out.println("Tails"); 
     ++tails; 
    } 
}