Thread: Help with Java code,

Write a program that stores the total rainfall for each of 12 months into an array of doubles. The program should have methods that return the following:

the total rainfall for the year.
the average monthly rainfall.
the month with the most rain.
the month with the least rain.

Here's my code so far.

import java.util.Scanner;		//Needed for Scanner class.
 
public class Rainfall
{
	public static void main(String[] args)
	{
		double[] months = new double[12];
		double[] rainfall = new double[12];
 
		Scanner scn = new Scanner(System.in);
 
		for (int index = 0; index < rainfall.length; index++)
		{
			System.out.print("rainfall for month #" + (index + 1) + ": ");
			rainfall[index] = scn.nextDouble();
		}
 
		for (int index = 0; index < rainfall.length; index++)
		{
			TotalRainfall(rainfall[index]);
			Average(rainfall[index]);
			Most(rainfall[index]);
			Least(rainfall[index]);
		}
	}
 
	public static void TotalRain(double rainfall)
	{
		double total = 0;
		for (int index = 0; index < rainfall.length; index++)
		{
			total += rainfall[index];
		}
		System.out.print("Total rainfall is: " + total);
	}
 
	public static double Avrge(double rainfall)
	{
		double total = 0;
		double average;
 
		for (int index = 0; index < rainfall.length; index++)
		{
			total += rainfall;
		}
		average = total / rainfall.length;
 
		System.out.println("average monthly rainfall is: " + average);
	}
 
	public static double Most(double rainfall)
	{
		int highest = rainfall[0];
		int month;
		for (int index = 1; index < rainfall.length; index++)
		{
			if (rainfall[index] > highest)
				{
					highest = rainfall[index];
					month = index;
				}
		}
 
		System.out.println("Highest rain happened during month #" + month);
	}
	public static double Least(double rainfall)
	{
		int lowest = rainfall[0];
		int month;
		for (int index = 1; index < rainfall.length; index++)
		{
			if (rainfall[index] < lowest)
			{
				lowest = rainfall[index];
				month = index;
			}
		}
 
		System.out.println("Least rain happened during month #" + month);
	}
}

Originally Posted by Norm

Please post any questions or problems you are having?
If there are errors, copy the full text of the messages and post them.

C:\Users\PSY\Desktop\Rainfall.java:38: error: double cannot be dereferenced
for (int index = 0; index < rainfall.length; index++)
^
C:\Users\PSY\Desktop\Rainfall.java:40: error: array required, but double found
total += rainfall[index];
^
C:\Users\PSY\Desktop\Rainfall.java:50: error: double cannot be dereferenced
for (int index = 0; index < rainfall.length; index++)
^
C:\Users\PSY\Desktop\Rainfall.java:54: error: double cannot be dereferenced
average = total / rainfall.length;
^
C:\Users\PSY\Desktop\Rainfall.java:61: error: array required, but double found
int highest = rainfall[0];
^
C:\Users\PSY\Desktop\Rainfall.java:63: error: double cannot be dereferenced
for (int index = 1; index < rainfall.length; index++)
^
C:\Users\PSY\Desktop\Rainfall.java:65: error: array required, but double found
if (rainfall[index] > highest)
^
C:\Users\PSY\Desktop\Rainfall.java:67: error: array required, but double found
highest = rainfall[index];
^
C:\Users\PSY\Desktop\Rainfall.java:76: error: array required, but double found
int lowest = rainfall[0];
^
C:\Users\PSY\Desktop\Rainfall.java:78: error: double cannot be dereferenced
for (int index = 1; index < rainfall.length; index++)
^
C:\Users\PSY\Desktop\Rainfall.java:80: error: array required, but double found
if (rainfall[index] < lowest)
^
C:\Users\PSY\Desktop\Rainfall.java:82: error: array required, but double found
lowest = rainfall[index];
^
12 errors

Tool completed with exit code 1


I'm using Textpad and these are the errors.

That really helped!

Although, I got the program to compile correctly, I'm having a logical error.

I can't get the program to grab which month had the most and least rain.

public static void main(String[] args)
	{
		double[] months = new double[12];
		double[] rainfall = new double[12];
 
 
 
		Scanner keyboard = new Scanner(System.in);
 
		for (int index = 0; index < rainfall.length; index++)
		{
			System.out.print("Enter rainfall for month #" + (index + 1) + ": ");
			rainfall[index] = keyboard.nextDouble();
		}
 
			TotalRainfall(rainfall);
 
			Average(rainfall);
 
			Most(rainfall, months);
 
			Least(rainfall, months);
	}
 
	public static void TotalRainfall(double[] rainfall)
	{
		double total = 0;
		for (int index = 0; index < rainfall.length; index++)
		{
			total += rainfall[index];
		}
		System.out.printf("\ntotal rainfall of the year is: %.1f" , total);
	}
 
	public static void Average(double[] rainfall)
	{
		double total = 0;
		double average;
 
		for (int index = 0; index < rainfall.length; index++)
		{
			total += rainfall[index];
		}
		average = total / rainfall.length;
 
		System.out.printf("\naverage monthly rainfall is: %.1f" , average);
	}
 
	public static void Most(double[] rainfall, double[] months)
	{
		double highest = rainfall[0];
		double month;
 
		for (int index = 1; index < rainfall.length; index++)
		{
			if (rainfall[index] > highest)
				{
					month = months[index];
				}
		}
			System.out.println("\nmonth with the most rain is " + month);
	}
	public static void Least(double[] rainfall, double[] months)
	{
		double lowest = 0;
		int month=0;
		for (int index = 1; index < rainfall.length; index++)
		{
			if (rainfall[index] < lowest)
			{
				lowest = rainfall[index];
				month = index;
			}
		}
 
		System.out.println("month with the least rain" + month);
	}
}

The output is perfectly fine.

When it comes to the most and least rain, it always shows month as month #0

I got it all done!

Basically what I did was, I changed the months to String variables and I could return it just perfectly!

Thank you Norm for you help,
Truly appreciated!