Question

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

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

我在主应用程序中也有一个名为isFileLoaded的标志，如果有文件打开则为true，否则为false。理想情况下，swing工作者应在完成加载文件并进行任何需要处理的操作后设置该值。

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

这是一种做事的好方法吗？还有更好的方法吗？

Answer 1

考虑改为添加一个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();
            }
        });
    }
}

Answer 2

你应该重新调整你的代码一点，以避免使用整个框架，这确实不是很干净（但是如果它工作的人关心）。

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

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

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

SwingWorker在完成工作后设置标志

2 个答案: