Java min max values

Question

Everything is working fine, I just need my output to say the quarter with the highest/lowest rainfall, not the actual values. I am not sure how to tie the quarter and the values together so that the output will be quarter 1, 2, 3 or 4.

import java.util.Scanner;

public class rainfall
{
   public static void main(String[] args) 
   { 
      Scanner scan = new Scanner(System.in); 
      double[] rainfall = new double[4]; 
      double totalRainfall = 0.0; 
      double max = 0, min = 0;


      for (int i=0; i < 4; i++) 
      { 
         System.out.print("Enter rainfall for quarter " + (i+1) + ": "); 
         rainfall[i] = scan.nextDouble(); 
         totalRainfall += rainfall[i]; 
         if (i == 0)
         {
            max = min = rainfall[i];
         }

         {
            if (rainfall[i] > max)
               max = rainfall[i];
            else if (rainfall[i] < min)
               min = (i + 1);
            //min = rainfall[i];     
         }
     }

   System.out.println("Total rainfall = "+totalRainfall); 
   System.out.println("Average rainfall = "+(totalRainfall / 4.0)); 
   System.out.println("Max quarter rainfall = "+ max);
   System.out.println("Min quarter rainfall = " + min);

   //System.out.println("Max quarter rainfall = "+ maxQuarter);
   //System.out.println("Min quarter rainfall = " + minQuarter);

   }//end main 
}//end class

Sort that array and print the first/last value of the array. http://stackoverflow.com/questions/8938235/java-sort-an-array — grizeldi, Mar 24 '14 at 13:54

Tyler · Answer 1 · 2014-03-24T14:13:17.490

2

Just store the indexes instead of the values:

if (rainfall[i] > rainfall[max])
    max = i;
else if (rainfall[i] < rainfall[min])
    min = i;

System.out.println("Max quarter rainfall = " + max + 1);
System.out.println("Min quarter rainfall = " + min + 1);

edited Mar 24 '14 at 14:13

answered Mar 24 '14 at 13:54

Tyler

17,669
10
51
89

1

Should be i + 1, since the indexes will be 0 to 3 but they want them to correspond to quarters 1 to 4. Or they can add the 1 on when they output. – Anthony Grist Mar 24 '14 at 13:56
Ah good point, and I agree that that should go in the print statement. – Tyler Mar 24 '14 at 13:59
How would i declare the min and max at the top of the code? Sorry, I am extremely new to java – user3455588 Mar 24 '14 at 14:29
You could initialize them as 0 (like you have done). This would mean that the min and max would default to the first quarter. – Tyler Mar 24 '14 at 14:32

Charles Goodwin · Answer 2 · 2014-03-24T17:35:53.093

You need to track maxQuarter and minQuarter (added as ints) as well as max and min. Like so:

import java.util.Scanner;

public class rainfall
{
   public static void main(String[] args) 
   { 
      Scanner scan = new Scanner(System.in); 
      double[] rainfall = new double[4]; 
      double totalRainfall = 0.0; 
      double max = 0, min = 0;
      // init to Q1 as that's the 1st tested
      int maxQuarter = 1, minQuarter = 1;


      for (int i=0; i < 4; i++) 
      { 
         System.out.print("Enter rainfall for quarter " + (i+1) + ": "); 
         rainfall[i] = scan.nextDouble(); 
         totalRainfall += rainfall[i]; 
         if (i == 0)
         {
            max = min = rainfall[i];
         }

         {
            if (rainfall[i] > max) {
               max = rainfall[i];
               maxQuarter = i + 1;
            }
            if (rainfall[i] < min) {
               min = rainfall[i];
               minQuarter = i + 1;
            }
         }
     }

   System.out.println("Total rainfall = "+totalRainfall); 
   System.out.println("Average rainfall = "+(totalRainfall / 4.0)); 
   System.out.println("Max quarter rainfall = "+ max);
   System.out.println("Min quarter rainfall = " + min);

   System.out.println("Max quarter rainfall = "+ maxQuarter);
   System.out.println("Min quarter rainfall = " + minQuarter);

   }//end main 
}//end class

With these changes, it runs fine for my basic tests:

Enter rainfall for quarter 1: 2
Enter rainfall for quarter 2: 4
Enter rainfall for quarter 3: 6
Enter rainfall for quarter 4: 7
Total rainfall = 19.0
Average rainfall = 4.75
Max quarter rainfall = 7.0
Min quarter rainfall = 2.0
Max quarter rainfall = 4
Minquarter rainfall = 1

I would have the text say something like `"Quarter with most rainfall is " + quarter + " with rainfall (actual):"` — Engineer2021, Mar 24 '14 at 13:59
I just edited his example. Not too interested in dressing it up for him. — Charles Goodwin, Mar 24 '14 at 13:59
so, if i do this way, it is always saying that quarter 4 is the max and min, no matter if quarter might have the most rainfall. Any idea on to store the correct variable in i? — user3455588, Mar 24 '14 at 14:27
You're doing something wrong then. I just tested it. It worked fine for my example (added to the answer). — Charles Goodwin, Mar 24 '14 at 17:35
Thanks, I just figured it out. I left a set of brackets out of the if statement. Thanks for the help! — user3455588, Mar 24 '14 at 18:22
Don't forget to a) sign up b) tick the answer c) try to answer some yourself! — Charles Goodwin, Mar 24 '14 at 20:59

score 0 · Answer 3 · answered Mar 24 '14 at 13:52

0

You should format your code (Ctrl+Shift+f in Eclipse).

Keep the i index in parallel to the min and max, something like imax, imin. Otherwise you lose the origin of the values like in your case.

answered Mar 24 '14 at 13:52

Christophe Roussy

16,299
4
85
85

Java min max values

3 Answers3