Can someone look at my simple program and tell me what is wrong with it?

Actually, I would use a do while loop here. Do while works like this:

Code:

---

do {
    //code that repeats goes here
} while(expression); //repeat until expression is false.
//put code for what to do after the loop here.

---

The reason for preferring do while to an infinite loop and a break is that it is more clear to see when the loop terminates.

I did it! Thank you so much everyone for your help!