硬貨の数を変更する

Question

記載された変更量（1〜99）を満たすために必要な最小限の硬貨（四半期、十二分、ニッケルおよびペニー）を取得しようとしています。例：

Change: 93 

Quarters: 3 

Dimes: 1 

Nickels: 1 

Pennies: 3 

私がプログラムを実行すると、私は答えが得られません。私の質問は、上記のソリューションのように私のプログラムの出力をどのように見えるようにするのですか？プログラム内の方程式を変更して、必要な数値を取得していますか？私が正しいプログラムの唯一の部分は、記載された量を満たすのに必要な四分の一の量です。

package mincoins; 
import java.util.Scanner; 

public class MinCoins2 
{ 

    public static void main(String[] args) 

    { 
     int change = 0; 
     int quarters = 0; 
     int dimes = 0; 
     int nickels = 0; 
     int pennies = 0; 
     try (//creates a copy of Scanner class 
       Scanner in = new Scanner(System.in)) 
     { 
      System.out.println("Please enter amount of change(1-99)"); 

      change = in.nextInt(); 

      //loop for quarters 
      while (change >= 25) 
      { 
       change = change - 25; 
       quarters = quarters + 1; 
      } 
      while (change >= 10) 
      { 
       change = change - 10; 
       dimes = dimes + 1; 
      } 
      while (change >= 5) 
      { 
       change = change - 5; 
       nickels = nickels + 1; 
      } 
      pennies = change; 
      System.out.println("Change: " + change); 
      System.out.println("Quarters = " + change); 
      System.out.println("Dimes = " + change); 
      System.out.println("Nickels = " + change); 
      System.out.println("Pennies = " + change); 
     } 
    }//end of main 
}//end of class 

Answer 1

あなたは最後の4行にタイプミスがあります：

System.out.println("Change:" + change); 
System.out.println("Quarters= "+ change); 
System.out.println("Dimes=" + change); 
System.out.println("Nickels=" + change); 
System.out.println("Pennies=" + change); 

があります。このプログラムは、ここで私がこれまで持っているものだ1〜99

の任意の記載された金額のために働く必要がありますすべての印刷change

また、whileループ条件は、>ではなく、である必要があります。

最後に、変更を最後に印刷するには、whileループで変更するのでコピーする必要があります。

change = in.nextInt(); 
int changeOut = change; 
... 
System.out.println("Change:" + changeOut); 

Live Demo

Answer 2

主な問題は、あなたが使用している変数です。適切な場所にquarters、dimes、nickels、および変数を使用する必要があります。

もう1つ論理的なエラーがあります。ペニーカウントを構成するために残りのすべての変更を使い果たす必要があります。 （それが 0でなければならないので）あなたの出力（あなたが実際に必要とする1つ）を使用すると、余っの変化をカウントすることを強制しないので、あなたは完全に別のループを回避し、ちょうど直接割り当てることができpennieschangeへ：

pennies = change; 

...これにより残ったペニーの総数が正しい結果になります。

いくつかの一般的なクリーンアップでは、少し混乱させる可能性があるので、printステートメントでの割り当てを避けることもできます。代わりにSystem.out.printlnから独立したchange -= 25などのステートメントを使用してください。

Answer 3

明らかにタイプミスが、問題となっているにする必要がある：

System.out.println("Quarters = " + quarters); 
System.out.println("Dimes = " + dimes); 
System.out.println("Nickels = " + nickles); 
System.out.println("Pennies = " + change); 

しかし、私はこのコードはW.R.T.をどのように機能するかについての提案をしたいですこれは簡単にループすることなく行われるループ、例えば：

//loop for quarters 
while (change > 25) { 
    change = change - 25; 
    quarters = quarters + 1; 
} 

に：changeと25がint S、SO /は整数除算を行う予定であるため

quarters = change/25; 
change = change - quarters * 25; 

です。それは、変更に適合する四半期の最大数、つまり四捨五入されます。

さらに一歩行く、あなたはまた、各ステップで、残りの変更を取得するために%剰余演算子を使用することができます。

quarters = change/25; 
change = change % 25; 

dimes = change/10; 
change = change % 10; 

nickels = change/5; 
change = change % 5;