GUI freezes with long running task in a SwingWorker Thread

414 views Asked by At

I have a long running task winch executes when the user clicks a button in a GUI application:

simplfyButton.addMouseListener(new MouseAdapter() {
            @Override
            public void mousePressed(MouseEvent e) {


                simplfyAllText();
            }//mouse evebt
        });

The simplfyAlltext() method performs some GUI calls then a long running task, so I have placed this is SwingWorker:

private void simplfyAllText() {

        /*
         * User has imported a text document for simplfying Depending on the size of the
         * document this could be time consuming So we will perform this on a
         * SwingWorker thread
         */

        // If user selects 'Simplfy' before any import is done
        if (!clearTextArea) {
            message_textArea.setText("");
            clearTextArea = true;
            displayMessage("You must import your text doucument to simplfy.", "Notice!");
            return;
        } else if (message_textArea.getText().length() <= 20) {
            displayMessage("You must import your text doucument to simplfy.", "Notice!");
            return;
        }

        // Set up stuff outside the worker thread
        exchangedText.setText("");
        progressBar.setIndeterminate(false);
        progressBar.setStringPainted(true);
        progressBar.setVisible(true);
        message_textArea.setEditable(false);

        // Create our worker to all heavy tasks off the event dispatch thread
        SwingWorker<String, Integer> sw = new SwingWorker<String, Integer>() {

            String simplifiedText = "";

            @Override
            protected String doInBackground() throws Exception {
                // Simplfy the text form message TA
                int size = message_textArea.getText().length();

                for (int i = 0; i < size; i++) {
                    publish(i);
                }
                simplifiedText = textSimplfier.simplyFullText(message_textArea.getText());
                return "finished";

            }

            @Override
            protected void process(List<Integer> chunks) {
                // define what the event dispatch thread
                // will do with the intermediate results received
                // while the thread is executing

                for (int val : chunks) {
                    progressBar.setValue(val);
                    countButton.setText(Integer.toString(val));
                    System.out.println("PROCESS : " + val);
                }

            }

            @Override
            protected void done() {
                // this method is called when the background
                // thread finishes execution
                outPut_TextArea.setText(simplifiedText);
                String exchanged = textSimplfier.getSwappedWords().toString();
                exchangedText.setText(exchanged);
                // remove 1/3 for : between words
                String wordsWithoutHTML = "";
                try {
                    wordsWithoutHTML = exchangedText.getDocument().getText(0, exchangedText.getDocument().getLength());
                } catch (BadLocationException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
                int noExchanged = ((wordsWithoutHTML.length() / 3) * 2);
                int noOfWords = message_textArea.getText().length();
                keyWord_TextField.setText(
                        "Complete!. " + noExchanged + "/" + noOfWords + " simplfied. Click /'Clear/' to continue.");

            }
        };

        sw.run();
    }

Even though the long running background task is run on a SwingWorker thread the GUI still frezzes during its execution, and the progress from the SwingWorker.process method does not appar to update the ProgressBar. Any input appreciated.

1

There are 1 answers

0
camickr On BEST ANSWER
sw.run();

You should be using:

sw.execute();

To start execution on another Thread.

Read the section from the Swing tutorial on Concurrency for more information and working examples.