使用 C# 更改计算器

Question

I am trying to write a program that takes price and amount paid and outputs the change denominations. I am having issues with the following.

• I will also like only applicable change denominations to be printed (Ie, if change is 71, only show FithyBills =1, TwentyBills=1 and Bills =1 and ignore the rest). Currently, it shows all with other recorded as Ten Bills=0 and so on).

Here is the code

using System;

namespace ChangeCalulator
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.Write("Item Price: ");
            int price = Convert.ToInt32(Console.ReadLine());
            Console.Write("Amount Paid: ");
            int paid = Convert.ToInt32(Console.ReadLine());
            while (paid < pice)
            {
                Console.WriteLine("You have paid less than the price");
                Console.Write("Paid: ");
                paid = Convert.ToInt32(Console.ReadLine());

            }
            decimal changeamount = paid - price;

            Change change = new Change(changeamount);
            //This displays the change based on each denominantion
            Console.WriteLine($"Hundred SEK Bills: {change.HundredBills}");
            Console.WriteLine($"Fifty SEK Bills: {change.FiveBills}");
            Console.WriteLine($"Twenty SEK Bills: {change.TwentyBills}");
            Console.WriteLine($"Ten SEK Bills: {change.TenBills}");
            Console.WriteLine($"Five SEK Bills: {change.FiveBills}");
            Console.WriteLine($"One SEK: {change.Bills}");

            Console.ReadKey();
        }
    }
    //The Change class  will create a public integer variable for each denomination and store the values.
    public class Change
    {
        private int changeamount;

        public int HundredBills { get; }
        public int FiftyBills { get; }
        public int TwentyBills { get; }
        public int TenBills { get; }
        public int FiveBills { get; }
        public int Bills { get; }
        //This calculates the number of each denomination the amount equals
        public Change(decimal price)
        {
            HundredBills = (int)(changeamount / 100);
            price %= 100;

            FiftyBills = (int)(changeamount / 50);
            price %= 50;

            TwentyBills = (int)(changeamount / 20);
            price %= 20;

            TenBills = (int)(changeamount / 10);
            price %= 10;

            FiveBills = (int)(changeamount / 5);
            price %= 5;

            Bills = (int)(changeamount / 1);
            price %= 1;
        }
    }
}

Any help on this is greatly appreciated.

Answer 1

In the constructor public Change(decimal price) , you need to use price :

public Change(decimal price)
{
    HundredBills = (int)(price / 100);
    price %= 100;

    FiftyBills = (int)(price / 50);
    price %= 50;

    TwentyBills = (int)(price / 20);
    price %= 20;

    TenBills = (int)(price / 10);
    price %= 10;

    FiveBills = (int)(price / 5);
    price %= 5;

    Bills = (int)(price / 1);
    price %= 1;
}

You can then remove the unneeded changeamount field.

Working demo: https://dotnetfiddle.net/NbyZPc

As for not printing the 0 amounts, a simple if (theValue > 0) check around each WriteLine should accomplish that.