Computing Month & Monthly Sales in Java

Question

computeHighestMonth(monthlySales)

This method receives the monthly sales array as an argument. This method will search and compare the values of the monthly sales array for the highest value. The method will return the index (or location in the array) of the month with the highest value.

I had it so that it would display the highest sales, but I can't figure out how to include the month name.

public static double computeHighestMonth(double[] monthlySales)
{
    double highestSales = 0;
    for (int index = 0; index < monthlySales.length; index++)
    {
        if (monthlySales[index] > highestSales)
        highestSales = monthlySales[index];
    }
    return highestSales;
}

What's the I/P you are giving and what's the O/P you are getting with this code means format. — SatyaTNV, Jul 13 '15 at 07:32
Not an answer to the question, but since you are talking about sales and use doubles, I suggest looking into `BigDecimals`; `Floating points` are inaccurate. [link1](http://stackoverflow.com/questions/6320209/javawhy-should-we-use-bigdecimal-instead-of-double-in-the-real-world); [link2](http://java-performance.info/bigdecimal-vs-double-in-financial-calculations/). — Kevin Cruijssen, Jul 13 '15 at 07:59

score 0 · Answer 1 · answered Jul 13 '15 at 07:31

You should keep not only the highestSales but the index value where it is stored. That means you should declare an integer which should be update everytime

monthlySales[index] > highestSales

is true. This way you would keep which is the "month number", which you should afterwards convert to the string you need.

Hope this helps!

score 0 · Answer 2 · answered Jul 13 '15 at 07:34

You also need to save the index:

public static double computeHighestMonth(double[] monthlySales)
{
    double highestSales = 0;
    int month = 0;
    for (int index = 0; index < monthlySales.length; index++)
    {
        if (monthlySales[index] > highestSales){
            highestSales = monthlySales[index];
            month = index + 1;
       }
    }
    System.out.print("The salary was highest on: ");
    switch(month){
        case 1: System.out.println("January");
        case 2: System.out.println("February");
        etc.
    }
        return highestSales;
}

score 0 · Answer 3 · answered Jul 13 '15 at 08:06

Simply instead of returning the highestSales you just need to return the index of this value, and then using this index you can provide the month and the highest value:

public static int computeHighestMonth(double[] monthlySales) {
    double highestIndex = 0;
    for (int index = 0; index < monthlySales.length; index++) {
        if (monthlySales[index] > monthlySales[highestIndex])
        highestIndex = index;
    }
    return highestIndex;
}

And then use the returned value to get the month and its value, like this:

highestIndex = computeHighestMonth(monthlySales);
System.out.println("the highest value is: "+monthlySales[highestIndex]+" of the month "+highestIndex+1);

score 0 · Answer 4 · answered Jul 13 '15 at 08:09

0

Would be easier if your monthlySales array would be a List<Double>, because you can then just use

List<Double> monthlySales = ...;
Double highestSales  = Collections.max(monthlySales);

answered Jul 13 '15 at 08:09

René Link

48,224
13
108
140

clD · Answer 5 · 2015-07-13T08:32:12.103

This will give you the month as a String i.e. January, February etc. Depending on your requirnments instead of you of returning a double you could return a key value pair or a String e.g. month + " had the highest sales " + highestSales

    public static double computeHighestMonth(double[] monthlySales)
    {
        double highestSales = 0;
        String month = "";

        DateFormatSymbols dfs = new DateFormatSymbols();            

        for (int index = 0; index < monthlySales.length; index++)
        {
            if (monthlySales[index] > highestSales) {
                highestSales = monthlySales[index];
                month = dfs.getMonths()[index];
            }
        }

        System.out.println(month);

        return highestSales;
    }

Computing Month & Monthly Sales in Java

5 Answers5