Not able to use Multi Catch from Java Effectively

Question

I really want to use features from Java-1.7. One of this feature is "Multi-Catch". Currently I have the following code

try {
    int Id = Integer.parseInt(idstr);

    TypeInfo tempTypeInfo = getTypeInfo(String.valueOf(Id));

    updateTotalCount(tempTypeInfo);
} catch (NumberFormatException numExcp) {
    numExcp.printStackTrace();
} catch (Exception exception) {
    exception.printStackTrace();
}

I want to remove the two catch blocks from the above code, and instead use single catch like below:

try {
    int Id = Integer.parseInt(idstr);

    TypeInfo tempTypeInfo = getTypeInfo(String.valueOf(Id));

    updateTotalCount(tempTypeInfo);
} catch (Exception | NumberFormatException ex) { // --> compile time error
    ex.printStackTrace();
}

But the above code is giving compile time error:

"NumberFormatException" is already caught by the alternative Exception.

I understood the above compile time error but what is the replace for my first block of code.

score 12 · Answer 1 · answered Sep 02 '15 at 05:36

NumberFormatException is a subclass of Exception. Saying that both catch blocks should have the same behavior is like saying that you don't have any special treatment for NumberFormatException, just the same general treatment you have for Exception. In that case, you can just omit its catch block and only catch Exception:

try {
    int Id = Integer.parseInt(idstr);

    TypeInfo tempTypeInfo = getTypeInfo(String.valueOf(Id));

    updateTotalCount(tempTypeInfo);
} catch (Exception exception) {
    exception.printStackTrace();
}

Moishe Lipsker · Answer 2 · 2015-09-02T05:45:44.933

3

The compiler is telling you that

} catch (Exception ex) {

will also catch NumberFormatException exceptions because java.lang.NumberFormatException extends java.lang.IllegalArgumentException, which extends java.lang.RuntimeException, which ultimately extends java.lang.Exception.

edited Sep 02 '15 at 05:45

answered Sep 02 '15 at 05:37

Moishe Lipsker

2,974
2
21
29

score 2 · Answer 3 · answered Sep 02 '15 at 05:37

2

The types in multi-catch must be disjoint and java.lang.NumberFormatException is a subclass of java.lang.Exception.

answered Sep 02 '15 at 05:37

akhil_mittal

23,309
7
96
95

score 2 · Answer 4 · answered Nov 15 '16 at 06:01

To add on to Mureinik's solution:

If you would like to differentiate the error handling for each of the subclasses you could use instanceof within the catch block something like:

FileNotFoundException is subclass of IOException

catch (IOException e) {
            if (e instanceof FileNotFoundException)
            {
                System.out.println("FileNotFoundException");
            }
            else if(e instanceof IOException)
            {
                System.out.println("IOException");
            }

        }

You can directly use multiple `catch` blocks to catch different types of exceptions in order of specific to general instead of using an `instanceof ` for each type of exception. — Raj Rajeshwar Singh Rathore, Nov 26 '18 at 07:11

score 1 · Answer 5 · answered Sep 02 '15 at 05:37

In this case multi-catch is not required because NumberFormatException is derived from Exception. You can simply catch only Exception to get them both. If you need another handling for NumberFormatException than for other exceptions, you must use the example you posted first.

score 1 · Answer 6 · edited Sep 02 '15 at 05:38

1

you can use

    try {
    int Id = Integer.parseInt(idstr);

    TypeInfo tempTypeInfo = getTypeInfo(String.valueOf(Id));

    updateTotalCount(tempTypeInfo);
  } catch (Exception exception) {
    exception.printStackTrace();
  }

edited Sep 02 '15 at 05:38

Am_I_Helpful

18,735
7
49
73

answered Sep 02 '15 at 05:37

Prashant

1,002
13
29

Thanks for the edits @Am_I_Helpful. In spirit of the edit you made, You_are_helpful :) – Prashant Sep 02 '15 at 16:39
Thanks for acknowledging me with "You_are_helpful"! Thanks again. And a +1 – Am_I_Helpful Sep 02 '15 at 16:53

score 0 · Answer 7 · answered Sep 02 '15 at 05:48

0

Exception is the parent class of all exceptions and ideally (Preferred approach - Best Coding Practice), you should never catch Exception unless you are not sure what is going to be thrown at Runtime in try block.

Since, in your code you are doing NumberFormat operation, which is a child class of Exception, you should not catch Exception (unless other 2 methods may throw unchecked exception) and instead, use:

try {
    int Id = Integer.parseInt(idstr);
    TypeInfo tempTypeInfo = getTypeInfo(String.valueOf(Id));\
    updateTotalCount(tempTypeInfo);
} catch (NumberFormatException npe) {
    npe.printStackTrace();
}

answered Sep 02 '15 at 05:48

Sategroup

955
13
29

Thank you @stategroup, I know that the Exception is super class , but i don't know other type of exceptions at run time, thats why my code has NumberFormatexception(expected) and excetpion(unexpected). – M.S.Naidu Sep 02 '15 at 06:24
@M.S.Naidu, To ensure that your code won't throw any other exception, you can check other methods that you are calling. If you are not doing any IO operations, class cast, etc. (that might throw RunTime exception) then you don't need to catch Exception. The reason Java has different exception level is so that you can have fine control over your code. This way, you can catch an exception and choose to ignore it or re-throw it – Sategroup Sep 02 '15 at 06:31

Not able to use Multi Catch from Java Effectively

7 Answers7

Linked