Hi again mushkyn

The program is correct, you just have a semicolon where it shouldn't be:

Make this:

for(i=1; i<n; i++);

this:

for(i=1; i<n; i++)

What was happening is the loop was completing by executing nothing, it was the same as doing this:

for(i=1; i<n; i++)
{
}
insert(x, i); 

The loop continues until i is no longer less than n. This logically means i would be equal to n when the loop finishes. Normally this doesn't matter because the int used is declared only for the duration of the loop, but in your case, because of the semi-colon, you were accidently using the final value of 5.

Gungnir, on 28 March 2013 - 02:49 AM, said:

Each to their own of course. I simply suggest the debugger because it generally lets you see the state of local variables at the time of execution.

The OP sticks a breakpoint on all the lines that are significant and where it could error out and they can see the value of x and j before the line even executes.

mushkyn, on 28 March 2013 - 02:55 AM, said:

Then it's as Gungnir quite correctly said. You've got an array of size 5, so the indexes available are 0,1,2,3,4

5 doesn't exist.

mushkyn, on 28 March 2013 - 02:55 AM, said:

I'm not familiar with BlueJ I'm afraid, but if you Google it, i'm sure they'll be some tutorials.

Quote

Or preferably use the one already built into the IDE. If you're not using an IDE, then you should.

If you're using netbeans, here's a quick tut on the debugging tool:

http://www.cs.uga.ed...gerTutorial.htm

You can Google for an eclipse one if you need.

Hi ngroebner

Take a look at this SO thread:

http://stackoverflow...e-after-nextxxx

Calling the doGet() method from doPost() is bad practice and goes against how HTTP works.

You should response.sendRedirect instead to create a new GET request, which will call the doGet automatically.