Why does this throw an InterruptedException?

Question

enter image description here

This code should compile, should it not? What am I doing wrong? I would like the code to briefly pause before displaying each number in the array.

public static void median(int odd[]) throws InterruptedException {

    Arrays.sort(odd);

    for (int i = 0; i < odd.length; i++) {
        System.out.println(odd[i]);
        Thread.sleep(500);
    }
    System.out.println("The median number of the previous list of numbers is: " + odd[5]);
}

Yeah, it compiles. Review a tutorial about checked exceptions. Those must be declared as thrown or caught. — Sotirios Delimanolis, Nov 25 '13 at 00:53
**Did** it throw the exception, or are you concerned that you have to still address the exception? — Makoto, Nov 25 '13 at 00:53
@Makoto I just want the thing to run! It wont. I updated the post with the error netbeans is giving me. — Christopher Baldwin, Nov 25 '13 at 00:56
Take a look at this: http://stackoverflow.com/questions/4386076/uncompilable-source-code-runtimeexception-in-netbeans — tbodt, Nov 25 '13 at 00:56
@tbodt: It'd be better to close the question as a duplicate instead, then. — Makoto, Nov 25 '13 at 00:57
@Makoto Not necessarily. See http://stackoverflow.com/a/20183224/1455016. — tbodt, Nov 25 '13 at 01:13

score 2 · Answer 1 · answered Nov 25 '13 at 00:57

I'm assuming that in your main you have something like

public static void main (String[] args) {
    int[] array  = new int[X];
    ...// populate array
    median(array);
}

Because median is a method declared as throwing a checked exception, you must either catch the Exception or rethrow it.

public static void main (String[] args) {
    int[] array  = new int[X];
    ...// populate array
    try {
        median(array);
    } catch (InterruptedException e) {
        // handle it
    }
}

or

public static void main (String[] args) throws InterruptedException {
    int[] array  = new int[X];
    ...// populate array
    median(array);
}

score 1 · Answer 2 · answered Nov 25 '13 at 00:58

1

It's better to use try catch block for Thread.sleep

Remove the throw exceptions and change

Thread.sleep(500);

to

try{Thread.sleep(500);}catch(Exception e){}

answered Nov 25 '13 at 00:58

JoeC

1,850
14
12

score 0 · Answer 3 · answered Nov 25 '13 at 00:56

0

Use Timer or TimerTask instead of making the thread sleep.

answered Nov 25 '13 at 00:56

iWumbo

135
1
8

score 0 · Answer 4 · answered Nov 25 '13 at 00:56

I'm not entirely convinced that the exception is thrown, but the reason you have to either declare it to be thrown, or catch it yourself, is due to it being a checked exception.

Because InterruptedException is declared as part of the signature of the method, you must address it in some way.

Why does this throw an InterruptedException?

4 Answers4