Hi Macca
Welcome to the ranch. A couple of things, first and most importantly, you never asked a question. Does your code compile? Does it run? do you get any compilation errors? Or an any runtime exceptions? Or does it run fine and not behave as you expect it to?
The first thing that jumps out is your while statement:
while (goOn == 'Y'|goOn == 'y')
The | is not the one you're looking for. You most likely want the ||, for the logical or. The single | is the bit operator 'or'.
Also, in the future, when you post code use the
UBB code tags to make it easier to read.