Probably the biggest problem in your code is that all of your logic seems to be jammed into the
main() method. Around here, we think that
MainIsAPain (←click that, it's a link). Your main() method should be small and it should be given the sole task of controlling the start and end of your program. All other logic not related to starting and ending the program should be delegated to either objects or other more focused methods.
One other thing to note: You only ever need to associate ONE Scanner variable with
System.in. In your code, you create multiple Scanner objects that are associated with System.in, which is unnecessary since there's only one System.in object in a running Java program. I would recommend declaring a Scanner that's associated with System.in like this instead:
Then, throughout your whole program, you just use the globally available
keyboard object to get input from the keyboard.
It's best to think of Scanners as being associated with their input source. Normally, there's only one keyboard on a computer, so if you want to scan the keyboard for input, you associate ONE Scanner object with System.in, which is associated to the keyboard device by default.
If you have String value that you want to scan, you'd associate a separate Scanner to that. If you have a file you want to scan, you'd associate a Scanner to that. If you have multiple files or multiple Strings that you want to scan for input, you'd associate one Scanner
for each file or string. That is, if you wanted to scan File1.txt and File2.txt, you'd create one Scanner with the
new Scanner("File1.txt") and assign that to one Scanner variable and then do
new Scanner("File2.txt") and assign that to another Scanner variable.
Does that make sense?
