Exception handling with a do-while loop in Java

Question

The algorithm should take in 3 integers to an ArrayList. If the input is not an integer, then there should be a prompt. When I execute my code the catch clause is executed, but the program runs into a infinite loop. Could someone guide me into the right direction, I appreciate the help. :-D

package chapter_08;
import java.util.Scanner;
import java.util.List;
import java.util.ArrayList;

public class IntegerList {
    static List<Integer> numbers = new ArrayList<Integer>();

    public static void main(String[] args) {
        Scanner input = new Scanner(System.in);
        int counter = 1;
        int inputNum;

        do {
            System.out.print("Type " + counter + " integer: " );
            try {
                inputNum = input.nextInt();
                numbers.add(inputNum);
                counter += 1;
            }
            catch (Exception exc) {
                System.out.println("invalid number");
            }
        } while (!(numbers.size() == 3));
    }
}

MC Emperor · Accepted Answer · 2017-03-08 21:14:11Z

3

That is because when the next int is read using nextInt() and it fails, the Scanner still contains the typed contents. Then, when re-entering the do-while loop, input.nextInt() tries to parse it again with the same contents.

You need to 'flush' the Scanner contents with nextLine():

catch (Exception exc) {
    input.nextLine();
    System.out.println("invalid number");
}

Notes:

You can remove the counter variable, because you're not using it. Otherwise, you could replace counter += 1 by counter++.
You can replace while (!(numbers.size() == 3)) with while (numbers.size() != 3), or even better: while (numbers.size() < 3).
When catching exceptions, you should be as specific as possible, unless you have a very good reason to do otherwise. Exception should be replaced by InputMismatchException in your case.

edited Mar 8, 2017 at 21:14

answered Feb 16, 2017 at 19:17

MC Emperor

23.3k16 gold badges90 silver badges139 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

davidxxx · Accepted Answer · 2017-02-16 19:23:00Z

1

If inputNum = input.nextInt(); cannot be fit into an int and a InputMismatchException is raised, the input of the Scanner is not consumed.

So after the catch, it loops and it goes again here :

inputNum = input.nextInt();

with exactly the same content in the input.

So you should execute input.nextLine(); in the catch statement to discard the current input and allow a new input from the user.
Besides it makes more sense to catch InputMismatchException rather than Exception as other exception with no relation with a mismatch could occur and it would not be useful to display to the user "invalid number " if it is not the issue :

    catch (InputMismatchException e){
        System.out.println("invalid number ");
        input.nextLine();
    }

edited Feb 16, 2017 at 19:23

answered Feb 16, 2017 at 19:17

davidxxx

132k23 gold badges231 silver badges228 bronze badges

Comments

Youcef LAIDANI · Accepted Answer · 2017-02-16 19:13:23Z

0

You should to use a break; in your catch(){} like so :

try {
    inputNum = input.nextInt();
    numbers.add(inputNum);
    counter += 1;
} catch (Exception e) {
    System.out.println("invalid number ");
    break;
}

So if one input is not correct break your loop.

answered Feb 16, 2017 at 19:13

Youcef LAIDANI

60.3k21 gold badges111 silver badges178 bronze badges

Comments

Inus Saha · Accepted Answer · 2017-02-16 19:26:11Z

0

try changing

inputNum = input.nextInt();

to

String inputText=input.next();
inputNum = Integer.valueOf(inputText);

it works perfectly well.

answered Feb 16, 2017 at 19:26

Inus Saha

1,93813 silver badges18 bronze badges

Comments

Faisal · Accepted Answer · 2017-02-16 19:27:14Z

0

You need to move the scanner to the next line. Add this line of code below the error message in the catch section.

input.nextLine();

answered Feb 16, 2017 at 19:27

Faisal

4155 silver badges20 bronze badges

Collectives™ on Stack Overflow

Exception handling with a do-while loop in Java

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related