Strange unreachable statement error...

[silafirion]

hey guys, I've got a bizzare error here and it's got me stumped. Any and all input would be very helpful.

/*assign5.java - swaping values in an array
 *Gabriel Miller
 *12-10-10
 */
 
public class assign5 //Declairing the public class
{
	public static void main(String[]args) //Begining the main method
	{
			 add(2, 3);
			 System.out.println(" ");
			 int[] numbers = {1, 2, 3, 4}; //Setting up the array
			 printArray(numbers);
			 System.out.println(" ");
			 swap(numbers, 0, 1);
			 printArray(numbers);
			 System.out.println(" ");
			 swap(numbers, 4, 5);
			 swap(numbers, 0, 3);
			 swap(numbers, 2, 1);
			 swap(numbers, 2, 3);
			 printArray(numbers);
	}
	
	public static void add(int x, int y)
	{
		System.out.println(x + y);
	}
	
	public static void printArray(int[] numbers)
	{
		 for (int i = 0; i < numbers.length; i++)
		    {
		        System.out.println(numbers[i]);
		    }

	}
	
	public static void swap(int[] numbers, int pos1, int pos2)
	{
		if ((pos1 > numbers.length) || (pos2 > numbers.length));
		{
			System.out.println("Invalid parameters.  Cannot compute.");
			System.out.println(" ");
			return;
		}
		
		int n = 0;
		int o = 0;
		n = numbers[pos1];
		o = numbers[pos2];
		numbers[pos1] = o;
		numbers[pos2] = n;
		
	}
}

When I try to compile it, it's giving me this error:
[CODE]
assign5.java:48: unreachable statement
int n = 0;
^
[\CODE]

Any idea why it could be giving it to me? I thought I was just declaring a new int variable for temporary use... I need some help!!

[pbrockway2]

if ((pos1 > numbers.length) || (pos2 > numbers.length)); // <-- because of the ; ...
{
    System.out.println("Invalid parameters.  Cannot compute.");
    System.out.println(" ");
    return;
}
int n = 0; // <-- ...you never get here

Basically the code says "if whatever then do nothing, then execute a block which ends in return, then assign 0 to n". You never get to the assignment because you always return inside the block.

[mine0926]

remove ; in line of if at swap method

	public static void swap(int[] numbers, int pos1, int pos2)
	{
		if ((pos1 > numbers.length) || (pos2 > numbers.length))
		...
		
	}

[mine0926]

EDIT: Did not see the last reply.

[silafirion]

Oh gosh... I never even noticed that I did that. Thanks guys.

[mine0926]

Please mark the thread as solved using thread tool.