Каков наилучший подход к обработке исключений, выданных в отдельном потоке?

Question

Я работаю над проектом J2ME, который порождает рабочие потоки для многочисленных задач, таких как загрузка содержимого HTTP. Базовая схема потоков аналогична большинству Java-приложений - здесь есть основной поток пользовательского интерфейса, а рабочие потоки создаются, чтобы делать что-то за кулисами. Мой вопрос заключается в том, как лучше всего обрабатывать исключения, возникающие в рабочих потоках?

Я обычно придерживаюсь обоснования дизайна, что большинство исключений должны быть максимально возможными. Когда я пишу однопоточные приложения, я обычно просачиваю исключения до уровня пользовательского интерфейса, а затем сообщаю о них в диалоговом окне ошибок пользователю. Есть ли подобная практика для многопоточных приложений? Самым интуитивно понятным для меня является перехват исключений в Thread.run (), а затем вызов вызова invokeLater в потоке пользовательского интерфейса, чтобы сообщить об этом в диалоге. Проблема, которую я вижу здесь, заключается в том, что за пределами рабочего потока преждевременно отмирает, этот подход на самом деле не уведомляет поток пользовательского интерфейса об ошибке. Я не вижу четкого способа выбросить исключение, если можно так выразиться, в потоках.

Спасибо, Andy

Answer 1

Вы НЕ ДОЛЖНЫ вставлять код пользовательского интерфейса в ваши рабочие!

/**
 * TWO CHOICES:
 * - Monitor your threads and report errors,
 * - setup a callback to do something.
 */
public class ThreadExceptions {

    /** Demo of {@link RunnableCatch} */
    public static void main(String[] argv) throws InterruptedException {
        final Runnable bad = new NaughtyThread();
        // safe1 doesnt have a callback
        final RunnableCatch safe1 = new RunnableCatch(bad);
        // safe2 DOES have a callback
        final RunnableCatch safe2 = new RunnableCatch(bad, new RunnableCallback() {
            public void handleException(Runnable runnable, Exception exception) {
                System.out.println("Callback handled: " + exception.getMessage());
                exception.printStackTrace();
            }

        });
        final Thread t1 = new Thread(safe1, "myThread");
        final Thread t2 = new Thread(safe2, "myThread");
        t1.start();
        t2.start();
        t1.join();
        t2.join();
        if (safe1.getException() != null) {
            System.out.println("thread finished with exceptions");
            safe1.getException().printStackTrace();
        }
        System.out.println("done");
    }


}

/** Throws an exception 50% of the time */
class NaughtyThread implements Runnable {
    public void run() {
        try {
            if (Math.random() > .5) {
                throw new RuntimeException("badness");
            }
        } finally {
            System.out.println("ran");
        }
    }
}

/** Called when an exception occurs */
interface RunnableCallback {
    void handleException(Runnable runnable, Exception exception);
}

/**
 * Catches exceptions thrown by a Runnable,
 * so you can check/view them later and/or
 * deal with them from some callback.
 */
class RunnableCatch implements Runnable {

    /** Proxy we will run */
    private final Runnable _proxy;

    /** Callback, if any */
    private final RunnableCallback _callback;

    /** @guarded-by(this) */
    private Exception _exception;

    public RunnableCatch(final Runnable proxy) {
        this(proxy, null);
    }

    public RunnableCatch(final Runnable proxy, RunnableCallback target) {
        _proxy = proxy;
        _callback = target;
    }

    public void run() {
        try {
            _proxy.run();
        } catch (Exception e) {
            synchronized (this) {
                _exception = e;
            }
            if (_callback != null) {
                _callback.handleException(_proxy, e);
            }
        }
    }

    /** @return any exception that occured, or NULL */
    public synchronized Exception getException() {
        return _exception;
    }
}

Answer 2

Другой вариант, отличный от того, что дал Stuph, - устанавливать исключения в локальном потоке. Если до того, как оно будет очищено, произойдет другое исключение, произойдет подтверждение По крайней мере, это дает кому-то шанс заметить исключение и обработать его.