通过`get(0,TimeUnit.Microseconds)检查`Future`状态是一个好主意吗?

HBa*_*ase 6 java concurrency future executorservice

我有一个Future,我想找出它的状态.我想到的是一个代码,如:

try {
    // Is that a good idea? Counting on exceptions looks weird.
    future.get(0, TimeUnit.MICROSECONDS);
    this.status = DONE;
} catch (InterruptedException e) {
    Thread.currentThread().interrupt();
    throw Throwables.propagate(e);
} catch (ExecutionException e) {
    this.status = FAILED;
} catch (TimeoutException e) {
    this.status = RUNNING;
} catch (CancellationException e) {
    this.status = CANCELED;
}
Run Code Online (Sandbox Code Playgroud)

看起来FutureTask会尝试持有一个锁,如果它可以获得锁将检查Future的状态.所以这似乎是一个好主意.

我在这里缺少陷阱吗?

Dun*_*nes 3

正如评论中所建议的,只需用于Future.isDone检查运行状态。但是,您仍然需要调用get()以确定它是否成功完成,检查是否有异常。