[Themanye]

I am receiving an error message saying illegal start of expression, beginning with "public double getTotal()" I dont know why i am getting this error maybe because i have a method inside a method? could someone please help and explain what is going wrong.

thank you.

public class astronomyNumbers1
{
 
  private double[] numbers;
  
   public astronomyNumbers1(String filename) throws IOException
	{
	 
      final int SIZE = 12;
		double[] numbers = new int[SIZE];
		int index = 0;
      
		
		// Create the necessary objects for file input.
      File file = new File(filename);
      Scanner inputFile = new Scanner(file);
		
		//read the file contents in to the array
		while (inputfile.hasNext() && index < numbers.length)
		{
		    numbers[index] = inputfile.nextInt();
		    index++;
		}
		
		
	public double getTotal()
   {
      double total = 0.0;   // Accumulator

      // Add up all the values in the array.
      for (double value : numbers)
         total += value;

      // Return the total.
      return total;
   }

   /**
    * The getAverage method returns the average of the 
    * elements in the array. 
    */

   public double getAverage()
   {
      return getTotal() / numbers.length;
   }

   /**
    * The getHighest method returns the highest value  
    * stored in the array.
    */

   public double getHighest()
   {
      // Store the first value in the array in
      // the variable highest.
      double highest = numbers[0];

      // Search the array for the highest value.
      for (int index = 1; index < numbers.length; index++)
      {
         if (numbers[index] > highest)
            highest = numbers[index];
      }

      // Return the highest value.
      return highest;
   }

   /**
    * The getLowest method returns the lowest value    
    * stored in the array.
    */

   public double getLowest()
   {
      // Store the first value in the sales array in
      // the variable lowest.
      double lowest = numbers[0];

      // Search the array for the lowest value.
      for (int index = 1; index < numbers.length; index++)
      {
         if (sales[index] < lowest)
            lowest = numbers[index];
      }

      // Return the lowest value.
      return lowest;
	}
		inputFile.close();
}
		}

[Dogstopper]

You cannot have methods inside of methods...it just does not work. If you are having scoping errors, try using instance variables.

[Themanye]

where is the method that i have inside the other method? and how do i fix it?

[MrLuke187]

In your code, all your methods are in your constructor. This won't work. Put them in the class-Body, not in the constructor.

Greetings: Luke

[softwareEngineer()]

What I see is a problem with not having a "for" loop at line 31; you can't just declare a for loop and leave it like that.

Is there something else you meant to do there?

[sakshamkum]

the for doesnot require block braces if there is only a single statement to be iterated

[MrLuke187]

sakshamkum, on 11 April 2011 - 04:20 PM, said:

the for doesnot require block braces if there is only a single statement to be iterated

Also, this is an advanced for-loop, the "for-each" in Java: Link

Greetings: Luke

[pbl]

Actually, you do not have a method inside a method
but you have code starting by

while (inputfile.hasNext() && index < numbers.length)

inside your instance variables declaration (not in a method or constructor)