Re: I"m a new Java student and i've got my program working except for one problem help

From:

"judith" <jspurlock83@hotmail.com>

Newsgroups:

comp.lang.java.programmer

Date:

16 Oct 2006 13:03:49 -0700

Message-ID:

<1161029029.758699.277980@i42g2000cwa.googlegroups.com>

Now it's adding the numbers back up when i put in a i--; i can't figure
out what to do it's in the catch statement (InputMismatchException
e)catch(InputMismatchException e)
            {

            keyboard.nextLine();
            System.out.println("Error, please enter the number again.");
            error = true;
            i--;

            }
It's supposed to go back to Enter number 1 if there's an error like
entering a letter like i entered t

C:\>java program3JS
How many numbers do you want to enter?
3
Enter number 1
t
Error, please enter the number again.
Enter number 2
3
Enter number 3
3
The average is 2.0

judith wrote:

The instructer wrote it this way. He wrote everything except the catch
statements so i don't know what to do because i can't change what he
wrote i just need the prompt to go back to the same Enter number 1 or
Enter number 2 if i make an error can some one help Oliver thanks for
your suggestion
Oliver Wong wrote:

"judith" <jspurlock83@hotmail.com> wrote in message
news:1161019659.941282.204000@f16g2000cwb.googlegroups.com...

// Loop until there is no error
do
{
try
{ error = false;
System.out.println("How many numbers do you want to enter?");
n = keyboard.nextInt();
if (n <= 0 )
throw new Exception ("Number must be greater than 0.");

}
catch (Exception e)
{

String message = e.getMessage();
System.out.println(message);
error = true;

}
}
while (error);

    You're using exceptions for control flow, which is generally considered
bad form. See http://java.sun.com/docs/books/tutorial/essential/exceptions/

    This problem might be more manageable if you refactored it into several
smaller methods. If each method is very short, and does one thing, then it's
easier to see if the method contains errors, as opposed to one big method
which does lots of different things.

public static void main(String[] args) {
  int numberOfNumbers = getNumberOfNumbers();
  int sum = 0;
  for (int i = 0; i < numberOfNumbers; i++) {
    sum = sum + getIthNumber(i);
  }
  displaySumAverageAndOtherStatistics(sum, numberOfNumbers);
}

    - Oliver