1

我整理了一个应用程序，可以打开文本文件并允许用户编辑它们（例如：文本编辑器）

有些文本文件可以任意大，因此打开它们需要一些时间。我添加了一个进度条来通知用户实际发生的事情，并且正在使用 swing worker 来执行实际的文件加载，给它一个文本区域的引用来转储所有文本。

我在主应用程序中还有一个标志，isFileLoaded如果有文件打开，则为 true，否则为 false。理想情况下，swing worker 应该在完成加载文件并完成它需要做的任何处理之后设置该值。

我将 swing worker 编写为一个单独的类，因此它没有嵌套在包含所有 GUI 逻辑的主 Frame 类中，主要是因为我不喜欢纯粹出于审美原因在类中定义类。因此，我目前正在将整个 Frame 的引用传递给 swing worker 并让它设置标志的值。

这是做事的好方法吗？有没有更好的方法？

4

2 回答 2

3

考虑添加一个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();
            }
        });
    }
}
于 2013-06-11T14:58:23.673 回答
1

您应该稍微重构您的代码以避免使用确实不是很干净的整个框架（但如果它有效，谁在乎）。

如果你想从设计的角度更酷，你应该使用模型：

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

并仅将此模型传递给您的工人，并在完成后设置标志。

于 2013-06-11T14:58:43.377 回答