Find Minimum and Maximum values

I have a form that has people enter an amount, and it calculates a discount for the invoice and displays it, as well as total of all invoices, number of invoices, and average invoice amount. I need it to also display the smallest and largest invoices. I am still quite new to C#, so any hints would be greatly appreciated. Here's the code I have so far, not sure how to start on the last part, looking into using Math.Max and Math.Min.

Code:

---

public partial class frmInvoiceTotal : Form
        {
                public frmInvoiceTotal()
                {
                        InitializeComponent();
                }

        int numberOfInvoices = 0;
        decimal totalOfInvoices = 0m;
        decimal invoiceAverage = 0m;
       
        private void btnCalculate_Click(object sender, EventArgs e)
                {
                        decimal subtotal = Convert.ToDecimal(txtEnterSubtotal.Text);
                        decimal discountPercent = .25m;
            decimal discountAmount = Math.Round(subtotal * discountPercent, 2);
                        decimal invoiceTotal = subtotal - discountAmount;

            txtEnterSubtotal.Text = subtotal.ToString("c");
            txtDiscountPercent.Text = discountPercent.ToString("p1");
                        txtDiscountAmount.Text = discountAmount.ToString("c");
                        txtTotal.Text = invoiceTotal.ToString("c");

            numberOfInvoices++;
            totalOfInvoices += invoiceTotal;
            invoiceAverage = totalOfInvoices / numberOfInvoices;

            txtNumberOfInvoices.Text = numberOfInvoices.ToString();
            txtTotalOfInvoices.Text = totalOfInvoices.ToString("c");
            txtInvoiceAverage.Text = invoiceAverage.ToString("c");

            txtEnterSubtotal.Text = "";
                        txtEnterSubtotal.Focus();
                }

        private void btnClearTotals_Click(object sender, System.EventArgs e)
        {
            numberOfInvoices = 0;
            totalOfInvoices = 0m;
            invoiceAverage = 0m;

            txtNumberOfInvoices.Text = "";
            txtTotalOfInvoices.Text = "";
            txtInvoiceAverage.Text = "";

            txtEnterSubtotal.Focus();
        }

        private void btnExit_Click(object sender, EventArgs e)
                {
                        this.Close();
                }

        private void label10_Click(object sender, EventArgs e)
        {

        }

        }
}

---

With these lines:

Code:

---

numberOfInvoices++;
totalOfInvoices += invoiceTotal;
invoiceAverage = totalOfInvoices / numberOfInvoices;

---

You are keeping track of invoice-to-invoice parameters. To keep track of min and max, you can either do:

Code:

---

minInvoice = Math.Min(minInvoice, invoiceTotal);
maxInvoice = Math.Max(maxInvoice, invoiceTotal);

---

Or:

Code:

---

if( invoiceTotal < minInvoice ) {
        minInvoice = invoiceTotal;
}

if( invoiceTotal > maxInvoice ) {
    maxInvoice = invoiceTotal;
}

---

I tend to prefer the second style, but either is fine.

Though: if you wrote the rest of that code, you should be entirely competent to see solution for yourself. We're happy to help, of course, but I think you will find these sorts of solutions will become apparent if you just work through them. MSDN is your friend. Here are, for example, the documents for Math.Min an Math.Max.

I've also surrounded your code with [code] and [/code] tags; this will preserve formatting, making it easier (and more likely) for us to read your code.

Hope that helps!

Thanks for your help. Sorry for not formatting the code as preferred, as I said I'm still quite new to programming. I appreciate your time.

You can loop:

Code:

---

decimal[] invoices = { 1234.24m, 3456.64m, 500.75m, 468.90m, 797.95m, 3035.40m };
//Method1
decimal minValue = invoices[0];
for (int i = 1; i < invoices.Length; i++)
{
        if (invoices[i] < minValue)
        {
                minValue = invoices[i];
        }
}
Console.WriteLine(minValue); // 468.90

---

Perhaps you would want to check the number of invoices before indexing the first invoice, depending on how the rest of your program works.

You can use LINQ:

Code:

---

decimal[] invoices = { 1234.24m, 3456.64m, 500.75m, 468.90m, 797.95m, 3035.40m };
decimal minValue = invoices.Min();
Console.WriteLine(minValue); // 468.90

---

Same principle with finding the highest values.

So, with your help I was able to get much further. I declared the variables for minInvoice and maxInvoice. I added the lines of code using the math.min and math.max syntax. I then attached the 2 variables to my text boxes for minimum and maximum invoice. The code runs, but it just displays the subtotal in the min and max invoice fields. I will continue working on it. I appreciate your help thus far.

Code:

---

private void btnCalculate_Click(object sender, EventArgs e)
                {
                        decimal subtotal = Convert.ToDecimal(txtEnterSubtotal.Text);
                        decimal discountPercent = .25m;
            decimal discountAmount = Math.Round(subtotal * discountPercent, 2);
                        decimal invoiceTotal = subtotal - discountAmount;
            decimal minInvoice = decimal.MaxValue;
            decimal maxInvoice = 0m;

            txtEnterSubtotal.Text = subtotal.ToString("c");
            txtDiscountPercent.Text = discountPercent.ToString("p1");
                        txtDiscountAmount.Text = discountAmount.ToString("c");
                        txtTotal.Text = invoiceTotal.ToString("c");

            numberOfInvoices++;
            totalOfInvoices += invoiceTotal;
            invoiceAverage = totalOfInvoices / numberOfInvoices;
            minInvoice = Math.Min(minInvoice, invoiceTotal);
            maxInvoice = Math.Max(maxInvoice, invoiceTotal);

            txtNumberOfInvoices.Text = numberOfInvoices.ToString();
            txtTotalOfInvoices.Text = totalOfInvoices.ToString("c");
            txtInvoiceAverage.Text = invoiceAverage.ToString("c");
            txtMinInvoice.Text = minInvoice.ToString("c");
            txtMaxInvoice.Text = maxInvoice.ToString("c");
           
            txtEnterSubtotal.Text = "";
                        txtEnterSubtotal.Focus();
                }

---

What do you need further help on?

Code:

---

decimal subtotal = Convert.ToDecimal(txtEnterSubtotal.Text);

---

Here you should be using TryParse() though, this is a textBox control i'm presuming, therefore invalid input can result in a runtime exception if the input is not validated first.

Realize, that this is all the ToDecimal() function does, for the string param overload:

Code:

---

public static decimal ToDecimal(string value)
{
        if (value == null)
        {
                return 0M;
        }
        return decimal.Parse(value, CultureInfo.CurrentCulture);
}

---

A textBox's Text property, will NEVER be null, so really, all it's doing is calling decimal.Parse() for you, and without validation on the input. The most "nothingness" value a textBox can have is string.Empty.

The min and max invoice boxes are just displaying whatever the current invoice total is. Still working on what the problem could be.

Quote:

---

Originally Posted by Inigo Montoya

The min and max invoice boxes are just displaying whatever the current invoice total is. Still working on what the problem could be.

---

You're comparing the total against values that aren't relevant from a first glance.

Look at these lines:

Code:

---

decimal invoiceTotal = subtotal - discountAmount;

decimal minInvoice = decimal.MaxValue;
decimal maxInvoice = 0m;

minInvoice = Math.Min(minInvoice, invoiceTotal);
maxInvoice = Math.Max(maxInvoice, invoiceTotal);

---

minInvoice will ALWAYS be the invoiceTotal because minInvoice is as high as things get. There can be no larger decimal value than decimal.MaxValue, and you're comparing for the smallest of those 2 numbers. It doesn't make sense. It's the complete reverse with the Math.Max line, unless your invoiceTotal is a negative value, you can't get a smaller value than 0...

I see what you mean Ace. It was suggested I use MaxValue so the correct value is returned the first time. If I had declared it as 0, then the first time an invoice is entered it would always display 0 as the min invoice.

Quote:

---

Originally Posted by Inigo Montoya

I see what you mean Ace. It was suggested I use MaxValue so the correct value is returned the first time. If I had declared it as 0, then the first time an invoice is entered it would always display 0 as the min invoice.

---

That's why if you look at my looping method, I automatically set it to the first invoice, and then I start looping from there, without looking at the one I already assigned as the minimum value to find any values smaller than the first invoice that I assigned to that variable. Doing it that way, avoids you from having to assign the variable to a useless value that doesn't really do anything for you.

**It double posted for some reason, so i'll provide an analogy here:

It would be like picking a number from a bag that has a whole bunch of paper slips, and then taking each other piece of paper out of that bag individually comparing to the first one you took out, and keeping the smaller value from each comparison as you went along until no more slips of paper were left.

Logically, this makes much more sense, than for instance, what you're doing, which is almost like having an initial piece of paper, writing a ridiculously huge number on it. And using that as a stepping stone to compare the same way as described earlier in my post here. Being able to skip steps is always good in programming, so long as you get a good reliable result.

I haven't gotten as far as loops yet. Here's what I am supposed to do:

Use the Math.Min method to compare each invoice total to a variable that contains the smallest invoice total to that point. Then, replace the variable with the smaller of the two invoice totals. To make this work for the first invoice, you can initialize this variable to a large number like the MaxValue member of the Decimal class.

This is a homework thing? Sort out your knowns for what they want you to do then:

- Use Math.Min() to compare each invoice total to a variable that contains the smallest invoice total

I wouldn't suggest setting it to MaxValue if there's no reason. Unless it's possible to get an invoice of $0, you can set it to the first invoice total, then you have an immediate starting point to compare against future invoices.

You know what you're supposed to do, but I can't help and I won't write code for you especially if this is homework, so what are you stuck on then?

:thumb:

I don't expect anyone to write code for me, just looking for outside opinions as well. My classmates in this online course aren't posting any suggestions either, and neither is my instructor. I thank you for your help anyways.

It seems to be doing the first invoice right, it should display invoiceTotal in both the smallest and largest invoice fields the first time. Just can't work out how to get it to compare the next input to the previous one. We cover loops in the next chapter, so I'm guessing I shouldn't be using those yet.

Doesn't help I'm trying to do this with a head cold...brain is a little fuzzy.

You're entering another invoice in based on a button click event and a textbox i'm assuming? Therefore, these values you're storing for the min and max invoice totals will run out of scope if you're doing it that way. You need to store them outside of the button click event method.

Code:

---

private decimal maxInvoice;
private decimal minInvoice;

public void button_click()
{
}

---

In that kind of setup. That way those values stay in tact even after the code for the button click is finished executing. And you can compare to the new values in the textbox for the next button click event. If you haven't gotten to loops yet, I doubt you're at filestreams or serialization.

You were right Ace, declaring maxInvoice and minInvoice preserved the invoice data from one input to the next. Once I declared them outside the calculate button event, it started keeping track properly. In addition, I declared the minInvoice as MaxValue, otherwise it always had a 0 in the smallest invoice field. For anyone else needing such basic help, here's the lines I had to add:

Code:

---


private decimal maxInvoice;
private decimal minInvoice = Decimal.MaxValue;

---

Thanks again for all your help guys. I think that once I get into this a little better I won't need as much help.

You could even use a nullable type, however, they probably just want you to stick with the normal types. It would eliminate having to set it to the max value as a kind of "safety" mechanism for not staying at 0 for the minimum.

Good you got it working. :)

Inigo, Would you be able to post the full code you came up with that resulted in a working product, I am doing a quite similar project at the moment and would like to compare if possible.

Thanks in advance!

Quote:

---

Originally Posted by cdr61804

Inigo, Would you be able to post the full code you came up with that resulted in a working product, I am doing a quite similar project at the moment and would like to compare if possible.

Thanks in advance!

---

Post a new thread and I will help you out entirely. Comparisons don't allow you to make the best of your learning endeavors. :)