Thread: Need advice

Hi,
total newbie here. Started learning programming only 2 weeks ago. I need help with assignment. I need to create voting programme which will ask 20 students to pick their 1 favorite soccer team from 5 options. I set my for loop to run 20 times. Problem is if students type in invalid vote, it counts it, so at the end i'm getting only 19 valid votes. What i'm looking for is basically if student types in invalid vote(not 1 to 5) to give him option to vote again. Hopefully you guys will understand what i'm trying to do. This is what i got so far.

int RealMadrid = 0; int Valencia = 0; int Chelsea = 0; int Barcelona = 0; int ManchesterCity = 0;     for (int i = 0; i<20; i++) { // starts loop   String vote = JOptionPane.showInputDialog(null, " Please vote for your favorite soccer team " + "\n 1. Real Madrid" + "\n 2. Valencia" + "\n 3. Chelsea" + "\n 4. Barcelona" + "\n 5. Manchester City", "Team input", JOptionPane.QUESTION_MESSAGE);   if (vote.equals("1")) { RealMadrid++; JOptionPane.showMessageDialog(null,"Real Madrid", "Student vote", JOptionPane.INFORMATION_MESSAGE); } else if (vote.equals("2")) { Valencia++; JOptionPane.showMessageDialog(null,"Valencia", "Student vote", JOptionPane.INFORMATION_MESSAGE ); } else if (vote.equals("3")) { Chelsea++; JOptionPane.showMessageDialog(null, "Chelsea", "Student vote", JOptionPane.INFORMATION_MESSAGE); } else if (vote.equals("4")) { Barcelona++; JOptionPane.showMessageDialog(null, "Barcelona", "Student vote", JOptionPane.INFORMATION_MESSAGE ); } else if (vote.equals("5")) { ManchesterCity++; JOptionPane.showMessageDialog(null, "Manchester City", "Student vote", JOptionPane.INFORMATION_MESSAGE); }   else {JOptionPane.showMessageDialog(null, "Invalid vote", "Warning", JOptionPane.WARNING_MESSAGE);}     }// end of loop   JOptionPane.showMessageDialog(null," The number of students who like Real Madrid is " + RealMadrid, "Team results", JOptionPane.INFORMATION_MESSAGE); JOptionPane.showMessageDialog(null," The number of students who like Valencia is " + Valencia, "Team results", JOptionPane.INFORMATION_MESSAGE); JOptionPane.showMessageDialog(null," The number of students who like Chelsea is " + Chelsea, "Team results", JOptionPane.INFORMATION_MESSAGE); JOptionPane.showMessageDialog(null," The number of students who like Barcelona is " + Barcelona, "Team results", JOptionPane.INFORMATION_MESSAGE); JOptionPane.showMessageDialog(null," The number of students who like Manchester City is " + ManchesterCity, "Team results", JOptionPane.INFORMATION_MESSAGE);         if (RealMadrid > 10){ JOptionPane.showMessageDialog(null,"Students love Real Madrid ", "Team results", JOptionPane.INFORMATION_MESSAGE); } else { JOptionPane.showMessageDialog(null,"Students don't like Real Madrid ", "Team results", JOptionPane.INFORMATION_MESSAGE); }         if (Valencia > 10) { JOptionPane.showMessageDialog(null,"Students love Valencia ", "Team results", JOptionPane.INFORMATION_MESSAGE); } else { JOptionPane.showMessageDialog(null,"Students don't like Valencia ", "Team results", JOptionPane.INFORMATION_MESSAGE); }         if (Chelsea > 10) { JOptionPane.showMessageDialog(null,"Students love Chelsea ", "Team results", JOptionPane.INFORMATION_MESSAGE); } else { JOptionPane.showMessageDialog(null,"Students don't like Chelsea ", "Team results", JOptionPane.INFORMATION_MESSAGE); }         if (Barcelona > 10) { JOptionPane.showMessageDialog(null,"Students love Barcelona ", "Team results", JOptionPane.INFORMATION_MESSAGE); } else { JOptionPane.showMessageDialog(null,"Students don't like Barcelona ", "Team results", JOptionPane.INFORMATION_MESSAGE); }         if (ManchesterCity > 10) { JOptionPane.showMessageDialog(null,"Students love Manchester City ", "Team results", JOptionPane.INFORMATION_MESSAGE); } else { JOptionPane.showMessageDialog(null,"Students don't like Manchester City ", "Team results", JOptionPane.INFORMATION_MESSAGE); } * *

Please do not give me the solutions, just advice. I want to be able to do it myself.
Thanks very much.
If i broke any forum rules then i'm really sorry.

Recommended reading: The while and do-while Statements (The Java™ Tutorials > Learning the Java Language > Language Basics)

When it reaches the end of the loop, it increments the loop counter and goes back to the top to see whether to terminate the loop, right? Every time.

However...

Say you get an invalid response. What can you do about it? How about decrement the loop counter? And then continue to the end of the loop?


Or some such thing...

Cheers!

Z

Hi guys, it took me long time to come back to say thank you. I have been quite busy. Studying and full time job is quite difficult :-) Special thanks to Zaphod_b.

To Alexdan: I could use for loop only and no break statement :-)