Drawing Program

Question

Guys I’m having trouble solving this problem, could someone help me? The problem of study is this:

In a small country on the planet Cyber, the current currency is B$. This currency has banknotes of B$50.00, B$10.00, B$5.00 and B$1.00. You have been hired to implement the ATM withdrawal system, and to do so you must always release as few notes as possible for a certain amount requested. Your algorithm will have the input value to be removed from the cash and will issue the total of each note needed to compose the requested value (so that this total is the minimum possible). (2,0 points). The algorithm must be closed when the value to be removed is 0(zero). Note: No serve may exceed B$1000 bits. Example: Value of Withdrawal: 650,00 Notes: 6 100,00 banknotes and a 50,00 banknote

And I did it:

int cedCinq = 50;
int numCedCinq;
int cedDez = 10;
int numCedDez;
int cedCinc = 5;
int numCedCin;
int cedUm = 1;
int numCedUm;`

double dinheiro = 1000;
double saque, resto;

Console.WriteLine("Digite o valor que deseja sacar: ");
saque = double.Parse(Console.ReadLine());

if (saque > dinheiro)
{
    Console.WriteLine("Valor de limite excedido, não pode ser sacado esse valor!");
}
else if (saque == 0)
{
    Console.WriteLine("Saque realizado!");
}

numCedCinq = Convert.ToInt32(saque / cedCinq);
resto = saque % cedCinq;
numCedDez = Convert.ToInt32(resto / cedDez);
resto = saque % cedDez;
numCedCin = Convert.ToInt32(resto / cedCinc);
resto = saque % numCedCin;
numCedUm = Convert.ToUInt16(resto / cedUm);
resto = saque % numCedUm;

Console.WriteLine("Do valor que foi solicitado " + saque + " foram usadas essas cédulas - Cinquenta: " + numCedCinq +
" Dez: " + numCedDez + " Cinco: " + numCedCin + " Um: " + numCedUm);
Console.ReadLine();`

How can I solve this in a much easier way?

Answer 1

Well, I made a code with comments for you. There are other ways to do it, obviously, but I think this one is very simple and can help you.

I commented on the code rather than explain here because while writing I was already commenting. If there is any doubt, comment that I will help you.

// Aqui listamos todas os valores de cédulas que estarão disponíveis
static readonly decimal[] valorCedula  = { 50.0M, 10.0M, 5.0M, 1.0M };

static void Main(string[] args)
{
    // O uso correto para valores monetários seria DECIMAL. Existem threads aqui na SOpt com informações sobre
    decimal dinheiro = 1000M;
    decimal saque = 0.0M;

    saque = decimal.Parse(Console.ReadLine());

    if(saque <= 0.0M || saque >= dinheiro)
        Console.WriteLine("Valor de limite excedido, não pode ser sacado esse valor!");
    else if (saque == 0.0M)
        Console.WriteLine("Saque realizado!");
    else
    {
        // Criamos uma lista de cédulas que iremos receber do tamanho máximo de cédulas disponíveis
        // Cada índice vai ser respectivo a quantas cédulas de valorCedula que será dado
        // Ou seja , cedulas[0] corresponderá a nota de 50, cedulas[1] a de 10
        int[] cedulas = new int[valorCedula.Length];

        // Fazemos um loop entre todos os valores de cédulas disponíveis
        for(int i = 0; i < valorCedula.Length; i++)
        {
            // Adicionamos no respectivo índice o valor de quantas cédulas vamos ter 
            cedulas[i] = Convert.ToInt32(Math.Floor(saque / valorCedula[i]));

            // Diminuímos do valo total, você pode ter uma variável auxiliar neste caso
            saque -= (cedulas[i] * valorCedula[i]);
        }

        // Escrevemos em tela
        for (int i = 0; i < cedulas.Length; i++)
            Console.WriteLine($"Cedula de { valorCedula[i] }: { cedulas[i] }");
    }

    Console.ReadLine();
}

See working on .NET Fiddle