trying to display month corresponds with temperature in Array type program

[leoshiner]

I have everything else working in this program but I cannot work out how to display what month the top temperature falls on.I have declared month as a variable and can get the program to request a different month for each input and it is outputting the top temp and the average temp just fine but not what month the top temperature is.any help would be greatly appreciated




package practical18;

import java.text.DecimalFormat;
import javax.swing.JOptionPane;


public class Pract18 {



public static void main(String[] args) {
DecimalFormat twoDigits=new DecimalFormat("0.00");

String strTemp;

double temp=0,topTemp=0,avg=0,total=0;

int counter,month=0;



double tempArray[]= new double [12];

for (counter=0;counter<tempArray.length;counter++){
strTemp=JOptionPane.showInputDialog("Enter temperature for month "+(counter+1));
temp=Double.parseDouble(strTemp);
tempArray[counter]=temp;}

for(counter=0;counter<tempArray.length;counter++){
if(tempArray[counter]>topTemp){
topTemp=tempArray[counter];


}

for(counter=0;counter<tempArray.length;counter++){
total=total+tempArray[counter];
}




}
avg=total/tempArray.length;


JOptionPane.showMessageDialog
(null,"the highest temp was "+twoDigits.format(topTemp)+" in month "+month+
" the average temperature is "+twoDigits.format(avg));






}



}

[Durden]

Where are you incrementing the month? Seems to stay at month=0 the entire time.

[leoshiner]

well I was going to increment it just after the line of code
topTemp=tempArray[counter];

but it still comes up with month 1 regardless

if I input the highest temperature say on the fifth month I want it to output fifth month at the end

[leoshiner]

I'm wondering would I need another for loop

[leoshiner]

or output month 5

[JosAH]

I'm wondering would I need another for loop

Nope, a single loop is enough; make 'month' the index of the month with the highest temperature; something like this:

month= -1; // no valid index yet.
for (int i= 0; i < months.length; i++) // loop over the months
   if (month == -1 || months[i] > months[month]) // higher?
      month= i; // remember highest index of temperature so far

kind regards,

Jos

[leoshiner]

Nope, a single loop is enough; make 'month' the index of the month with the highest temperature; something like this:

month= -1; // no valid index yet.
for (int i= 0; i < months.length; i++) // loop over the months
   if (month == -1 || months[i] > months[month]) // higher?
      month= i; // remember highest index of temperature so far

kind regards,

Jos

thanks that worked a treat as soon as I was able to get my head around it