SwingWorker done 方法使用 get() 抛出取消异常

Question

我遇到了为我的 gui 创建停止/启动 jbuttons 的问题，经过大量谷歌搜索后，我意识到我需要多线程。进一步阅读后，我发现了 swingworker 类，并且我设法让我的 GUI 响应 STOP 按钮。

现在我的问题是这样的

doinbackground() 方法执行一段代码，在无限 while 循环中捕获数据包，条件为 (!isCancelled)，一旦它被取消（STOP 按钮执行 worker.cancel()），它会返回一个 ArrayList理论上，我应该能够使用 get() 在 done() 方法中获取数据包。正确的？但是当我尝试这样做时，我得到了一个 CancellationException，这让我现在发疯了。

任何帮助都将受到高度评价！ 谢谢

编辑：obj 是在类外部声明的用于存储返回值的 ArrayList。

这是我通过 START jbutton 执行的代码

private void jButton5ActionPerformed(java.awt.event.ActionEvent evt) {


    final ArrayList packet_list = new ArrayList();
    obj.clear();

    try {
        worker = new SwingWorker<ArrayList,Integer>(){//initialze swingworker class


            @Override
            protected void done(){

                try {  

                    obj = get();

                }
                catch (InterruptedException ex) {
                    Logger.getLogger(NewJFrame3.class.getName()).log(Level.SEVERE, null, ex);
                } catch (ExecutionException ex) {
                    Logger.getLogger(NewJFrame3.class.getName()).log(Level.SEVERE, null, ex);
                }

            }

            //opens up stuff required to capture the packets    
            NetworkInterface   [] devices = JpcapCaptor.getDeviceList();
            int index = (jComboBox5.getSelectedIndex()-1);
            JpcapCaptor captor =JpcapCaptor.openDevice(devices[4], 65535, false, 20); 

            @Override
            protected ArrayList doInBackground(){  

                while(!isCancelled()){

                    try {
                        Packet packets = captor.getPacket(); //captures packets

                        if (packets != null)  //filters out null packets
                        {

                            //System.out.println(packets);
                            packet_list.add(packets); //adds each packet to ArrayList
                        }
                        Thread.sleep(100);

                    } catch (InterruptedException ex) {
                        return packet_list;
                    }
                }
                return packet_list;

            }


        };

        worker.execute();
    } catch (IOException ex) {
        Logger.getLogger(NewJFrame3.class.getName()).log(Level.SEVERE, null, ex);
    }
}                                 


The stop button simply executes 

worker.cancel(); no errors there. and this is the swingworker declaration

 private SwingWorker<ArrayList,Integer> worker;

Answer 1

cancel 不只是设置isCancelled 标志让您在闲暇时阅读。那将几乎没有用。 如果它还没有启动，它会阻止任务启动，如果它已经在运行，它可能会主动中断线程。因此，获得CancellationException 是自然的结果取消正在运行的任务。

为了进一步说明这一点，isCancelled 上的 Javadoc 指出：

如果此任务在正常完成之前被取消，则返回 true。

因此，如果返回 true，那么您的任务将无法正常完成。您不能取消任务并期望它照常继续。

SwingWorker 文档说“在后台线程中执行冗长的 GUI 交互任务的抽象类”。但是，对于 GUI 和应用程序生命周期，“冗长”的定义是不同的。 100 毫秒的任务对于 GUI 来说非常长，最好由 SwingWorker 完成。 10 分钟的任务对于SwingWorker 来说太长了，因为它有一个有限的线程池，你可能会耗尽。从您的问题描述来看，您确实做到了 - 一个可能运行时间很长的任务。因此，您应该创建一个适当的后台线程，而不是使用SwingWorker。

在该线程中，您将拥有一个 AtomicBoolean 或只是一个 volatile boolean 标志，您可以从 EDT 手动设置。然后线程可以将事件连同结果一起发布到 EDT。

代码：

class PacketCaptureWorker implements Runnable {
    private volatile boolean cancelled = false;
    public void cancel() {
        cancelled = true;
    }
    public void run() {
        while (!cancelled) {
            //do work
        }
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                //Use the result of your computation on the EDT
            }
        });
    }
}

new Thread(new PacketCaptureWorker()).start();

Answer 2

我尝试在 swingworker 线程的 while 循环中使用 volatile 布尔值，而不是使用 worker.cancel()，它运行良好。 （至少在表面上）我也设法创建了一个普通的后台线程，这也很有效：D 非常感谢你让我头疼不已！想知道两者中最好的方法是什么。

跟进，我必须使 volatile 布尔值可用于整个类，因为我必须为线程类创建 2 个单独的实例，一个使用 START，另一个使用 STOP。显然，两个不同的实例并不针对变量的同一个实例。这是不好的做法吗？