Illegal start

Hi all,

Does anyone know why i keep getting illegal start of type when i type System.out.println("The Largest Number is " + maxValue(numbers));
when i put // infront of it as a comment it compiles fine although i do not get the output.

Code :

---

import java.util.Scanner;
import java.io.*;
import java.io.IOException;
import java.util.*;
import java.util.Arrays;
 
 
public class arrayNum
{
 
       public static void main (String args[]) throws IOException{
 
 
                Scanner input = new Scanner(System.in);
                int array[] = new int[5];
                int count=0;
                int numbers;
 
 
 
 				System.out.println("Please enter 5 numbers");
 
 
                {
                while (count < 5)
                {
				//the input you get must be store in the array
				array[count] = input.nextInt();
						count++;
 
 
                }
				}
																}
 
				System.out.println("The Largest Number is " + maxValue(numbers));
 
                	public static int getmaxValue(int[] numbers)
            	{
						int maxValue = numbers[1];
						for(int i=1;i<numbers.length;i++)
						{
							if(numbers[i] < maxValue)
							{
								maxValue = numbers[i];
						}
							}
						return maxValue;
				}
 
 
 
 
 
 
			//	int max = array[5];
		//		for(int i = 1; i < 5; i++)
		//		 {
		//		 if(array[i] > max)
		//		 {
 		//		 max = array[i];
		//		 }
		//		 }
				// print the value instead of printing
 
 
			//	return max;
}

---

Your help is greatly appreciated!

Just a quick scan through it. As far as I can tell it isn't inside any method so try moving it before the } above it. Also you don't need the { before the while. Let us know how that works out.

Thanks,

I have removed the } in the while loop although it now compiles but does not give me the output :(
The out put should be showing the largest number that the user has entered 5 times.

I just realised you need the { brackets for the while loop otherwise it will keep looping.

So it is working now or not?

When I said to take out the { I meant the one before the while loop because there's no point to it and it isn't closed either.

Code :

---

 { <---This one
                while (count < 5)
                {

---

Code :

---

package javaapplication1;
 
 
import java.util.Scanner;
import java.io.*;
import java.io.IOException;
import java.util.*;
import java.util.Arrays;
 
 
 
class Main
{
 
 
 
       public static void main (String args[]) throws IOException
       {
 
 
                Scanner input = new Scanner(System.in);
                int array[] = new int[5];
                int count=0;
                int numbers;
 
 
 
 				System.out.println("Please enter 5 numbers");
 
 
                {
                while (count < 5)
                {
				//the input you get must be store in the array
				array[count] = input.nextInt();
						count++;
 
 
                }
                System.out.println("The Largest Number is " + getmaxValue(array));
				}
																}
 
 
 
                	public static int getmaxValue(int[] numbers)
            	{
						int maxValue = numbers[0];
						for(int i=1;i<numbers.length;i++)
						{
							if(numbers[i] > maxValue)
							{
								maxValue = numbers[i];
						}
							}
						return maxValue;
				}
 
 
 
 
 
 
			//	int max = array[5];
		//		for(int i = 1; i < 5; i++)
		//		 {
		//		 if(array[i] > max)
		//		 {
 		//		 max = array[i];
		//		 }
		//		 }
				// print the value instead of printing
 
 
			//	return max;
}

---

You should have to import the pakage of javaapplication1 and also your statement was written outside the main().Also, your getmaxvalue() was not for maximum but for minimum and you took the first index of array as the max value as you should have to take the zero index in the beginning. Well, its ok now. Enjoy.

First, you don't need to have parenthesis before the while loop. So remove that parenthesis and the matching closing parenthesis after closing the while loop.

Secondly, I think u should modify your array instantiation to something like:

int[] array; // Declare the array

array = new int[5]; // Instantiate the array

Refer to Arrays (The Java™ Tutorials > Learning the Java Language > Language Basics) for details.

I hope this helps.

Thanks alot, helped out heaps!