SwingWorker устанавливает флаг после завершения своей работы

Я собрал приложение, которое открывает текстовые файлы и позволяет пользователям редактировать их (например, текстовый редактор).

Некоторые текстовые файлы могут быть сколь угодно большими, поэтому их открытие может занять некоторое время. Я добавил индикатор выполнения, чтобы информировать пользователя о том, что что-то происходит на самом деле, и я использую Swing Worker для выполнения фактической загрузки файла, давая ему ссылку на текстовую область для вывода всего текста.

У меня также есть флаг в основном приложении под названием isFileLoaded, который является истинным, если файл открыт, и ложным в противном случае. В идеале рабочий процесс Swing должен установить это значение после завершения загрузки файла и выполнения необходимой обработки.

Я написал Swing Worker как отдельный класс, поэтому он не вложен в мой основной класс Frame, который содержит всю логику графического интерфейса, главным образом потому, что я не люблю определять классы внутри классов исключительно из эстетических соображений. Таким образом, в настоящее время я передаю ссылку на весь фрейм рабочему процессу свинга и позволяю ему установить значение флага.

Это хороший способ сделать что-то? Есть ли лучшие способы?


java swing swingworker
person MxLDevs    schedule 11.06.2013    source источник

Ответы (2)


arrow_upward
3
arrow_downward

Рассмотрите возможность добавления PropertyChangeListener, который содержит ссылку на ваш Frame (в этом отношении вполне подойдет анонимный внутренний класс) и который прослушивает свойство "state". Значение события будет равно StateValue.DONE, когда SwingWorker завершится.

Вот полностью рабочий пример:

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.beans.PropertyChangeEvent;
import java.beans.PropertyChangeListener;
import java.util.List;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JProgressBar;
import javax.swing.SwingUtilities;
import javax.swing.SwingWorker;
import javax.swing.SwingWorker.StateValue;
import javax.swing.UIManager;
import javax.swing.UnsupportedLookAndFeelException;

public class TestSwingWorker {

    private JProgressBar progressBar;

    protected void initUI() {
        final JFrame frame = new JFrame();
        frame.setTitle(TestSwingWorker.class.getSimpleName());
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        JButton button = new JButton("Clik me to start work");
        button.addActionListener(new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent e) {
                doWork();
            }
        });
        progressBar = new JProgressBar(0, 100);
        frame.add(progressBar, BorderLayout.NORTH);
        frame.add(button, BorderLayout.SOUTH);
        frame.pack();
        frame.setVisible(true);
    }

    private boolean someFlag;

    protected void doWork() {
        SwingWorker<Void, Integer> worker = new SwingWorker<Void, Integer>() {
            @Override
            protected Void doInBackground() throws Exception {
                for (int i = 0; i < 100; i++) {
                    // Simulates work
                    Thread.sleep(10);
                    publish(i);
                }
                return null;
            }

            @Override
            protected void process(List<Integer> chunks) {
                progressBar.setValue(chunks.get(chunks.size() - 1));
            }

            @Override
            protected void done() {
                progressBar.setValue(100);
                progressBar.setStringPainted(true);
                progressBar.setString("Done");
            }
        };
        worker.getPropertyChangeSupport().addPropertyChangeListener("state", new PropertyChangeListener() {

            @Override
            public void propertyChange(PropertyChangeEvent evt) {
                if (StateValue.DONE.equals(evt.getNewValue())) {
                    someFlag = true;
                }
            }
        });
        worker.execute();
    }

    public static void main(String[] args) throws ClassNotFoundException, InstantiationException, IllegalAccessException,
            UnsupportedLookAndFeelException {
        UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
        SwingUtilities.invokeLater(new Runnable() {
            @Override
            public void run() {
                new TestSwingWorker().initUI();
            }
        });
    }
}
person Guillaume Polet    schedule 11.06.2013
comment
Я не знаком с слушателями, хотя после прочтения некоторых руководств по Swing похоже, что если я действительно хочу получить отзывчивые приложения, мне нужно начать с ними работать. Можете ли вы предоставить несколько фрагментов, показывающих, где я буду добавлять слушателя и как с ним будет работать свинг-воркер? - person MxLDevs; 11.06.2013

arrow_upward
1
arrow_downward

Вы должны немного реструктурировать свой код, чтобы избежать использования всего фрейма, который действительно не очень чистый (но если он работает, кого это волнует).

Если вы хотите быть более крутым с точки зрения дизайна, вы должны использовать модель:

class FileModel
{
    boolean isLoading;
    // getter and setter that notifies
}

и передайте только эту модель своему рабочему, и после этого установите флаг.

person Pragmateek    schedule 11.06.2013
comment
Наличие объекта, совместно используемого несколькими компонентами, кажется хорошей идеей, поскольку я могу единообразно получать доступ к данным из различных объектов и обеспечивать дополнительный уровень абстракции. - person MxLDevs; 11.06.2013
comment
В этом суть паттерна MVC и всех его производных: отдельные задачи: вид с одной стороны, логика и данные с другой. :) - person Pragmateek; 11.06.2013