David Landup
David Landup

Reputation: 168

How to cancel done() in a SwingWorker thread?

I am making a program which should be able to stop the whole process on the click of a button.

I have used sw.cancel(true); to do so, however, the SwingWorker method protected void done() is still operating.

How can I on a push of a button cancel the whole thing? Not just the doInBackground() method?

Here's the whole relevant code below for those who are interested:

sw = new SwingWorker() {
    protected Object doInBackground() throws Exception {
        //Pravljenje timera
        t = new Timer(0, new ActionListener() {
            public void actionPerformed(ActionEvent e) {
            }
        });

        //Provera stanja checkboxova
        if(jCheckBox2.isSelected()) {
            try {
                int delay =(int) jSpinner2.getValue();
                jCheckBox1.setSelected(false);
                Thread.sleep(delay*60000);
            } catch (InterruptedException ex) {
                Logger.getLogger(App_Gui.class.getName()).log(Level.SEVERE, null, ex);
            }
        }

        if(jCheckBox1.isSelected()) {
            jCheckBox2.setSelected(false);   

            Date delay2 = (Date) jSpinner1.getValue();
            userCal = Calendar.getInstance();
            System.out.println("delay2: " + delay2);
            userCal.setTime(delay2);
            System.out.println("userCal:" + userCal);
            Calendar sysCal = Calendar.getInstance();
            System.out.println("sysCal: " + sysCal);
            timerAtDate = 
                (((int) userCal.get(Calendar.HOUR_OF_DAY) - (int) sysCal.get(Calendar.HOUR_OF_DAY)) * 60 * 60 * 1000 +
                ((int) userCal.get(Calendar.MINUTE) - (int) sysCal.get(Calendar.MINUTE)) * 60 * 1000 +
                ((int) userCal.get(Calendar.SECOND) - (int) sysCal.get(Calendar.SECOND)) * 1000);
            Thread.sleep(timerAtDate);
        }

        return null; 
    }

    @Override
    protected void done() {
        System.out.println("Done!");

        t.start();
        t.setRepeats(false);
        JDialog dialog = new JDialog();
        dialog.setLocation(700, 300);
        dialog.setSize(600, 400);
        dialog.setVisible(true);
        try {
            Thread.sleep(jSlider1.getValue());
        } catch (InterruptedException ex) {
            Logger.getLogger(App_Gui.class.getName()).log(Level.SEVERE, null, ex);
        }
        System.out.println("Zapravo gotov");
        dialog.getContentPane().setBackground(jLabel2.getBackground());
        dialog.setModal(true);
        Assignment_Tajmer_Aplikacija.f.setVisible(false); 
    }
};

sw.execute();

private void jSlider1StateChanged(javax.swing.event.ChangeEvent evt) {
    // TODO add your handling cosadde here:
    jSlider1.addChangeListener(new ChangeListener() {
        public void stateChanged(ChangeEvent e) {
            JSlider source = (JSlider) e.getSource();
            //System.out.println(source.getValue());
        }
    });
}                                     

private void jButton4ActionPerformed(java.awt.event.ActionEvent evt) {
    // TODO add your handling code here:
    JColorChooser jcc = new JColorChooser();
    Color c = jcc.showDialog(null, "Choose background color", Color.yellow);
    jLabel2.setForeground(c);
    jLabel2.setBackground(c);
    jLabel2.setText("Color: " + c.getRGB() + "(RGB)");
}                                        

private void jCheckBox2ActionPerformed(java.awt.event.ActionEvent evt) {
    // TODO add your handling code here:
    jCheckBox1.setSelected(false);
}                                          

private void jButton2ActionPerformed(java.awt.event.ActionEvent evt) {
    // TODO add your handling code here:
    sw.cancel(true);
}                                        

private void jCheckBox1ActionPerformed(java.awt.event.ActionEvent evt) {
    // TODO add your handling code here:
    jCheckBox2.setSelected(false);
}                                          

Upvotes: 0

Views: 425

Answers (2)

Kayaman
Kayaman

Reputation: 73558

Use the isCancelled() method in the beginning of done() to see if you should do the operations or not.

Upvotes: 1

Lee
Lee

Reputation: 738

You can't cancel it but you can short-circuit it so that it finishes early. Use some shared state flag between your threads to signal.

Upvotes: 0

Related Questions