我有要复制到另一个位置的文件列表,我想使用JProgressBar监视传输进度。

JProgressBar构造函数仅将int用作边界,而File.length返回一个long。我知道我可以将long转换为int并失去一些准确性,但这是正确的方法吗?

我没有包含任何代码,因为这实际上不是语法或编码问题,我不确定是否将长整数转换为int以用于进度条是否足够正确或准确。

最佳答案

我真的没有看到关于进度条边界的问题,仅仅是因为实际进度是一个从0到100的百分比值。您应该始终能够计算处理后的字节与字节总数之间的比例,这就是实际的进度。

我认为正确的问题应该是这样的:如何根据文件长度和要传输的字节总数正确更新进度条的进度?

考虑以下示例:

import java.awt.event.ActionEvent;
import java.beans.PropertyChangeEvent;
import java.beans.PropertyChangeListener;
import java.io.File;
import java.util.List;
import javax.swing.AbstractAction;
import javax.swing.Action;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JProgressBar;
import javax.swing.SwingUtilities;
import javax.swing.SwingWorker;

public class Demo {

    private void createAndShowGUI() {

        final JProgressBar progressBar = new JProgressBar();
        progressBar.setStringPainted(true);
        progressBar.setString("");

        final SwingWorker<Void,String> worker = new SwingWorker<Void, String>() {
            @Override
            protected Void doInBackground() throws Exception {

                long totalSize = 0l;

                File[] files = new File(System.getProperty("user.dir")).listFiles();
                for (File file : files) {
                    totalSize += file.length();
                }

                long transferred = 0l;

                for (File file : files) {
                    Thread.sleep(250); // Just for animation purpose
                    transferred += file.length();
                    int progress = (int) (transferred * 100l / totalSize);
                    setProgress(progress);

                    String text = String.format("%1s%%: %2s / %3s bytes", progress, transferred, totalSize);
                    publish(text);
                }

                return null;
            }

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

        worker.addPropertyChangeListener(new PropertyChangeListener() {
            @Override
            public void propertyChange(PropertyChangeEvent evt) {
                if ("progress".equals(evt.getPropertyName())) {
                    progressBar.setValue((Integer)evt.getNewValue());
                }
            }
        });

        Action startAction = new AbstractAction("Start") {
            @Override
            public void actionPerformed(ActionEvent e) {
                worker.execute();
                setEnabled(false);
            }
        };

        JPanel content = new JPanel();
        content.add(progressBar);
        content.add(new JButton(startAction));

        JFrame frame = new JFrame("Demo");
        frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
        frame.add(content);
        frame.pack();
        frame.setLocationByPlatform(true);
        frame.setVisible(true);

    }

    public static void main(String[] args) {
        SwingUtilities.invokeLater(new Runnable() {
            @Override
            public void run() {
                new Demo().createAndShowGUI();
            }
        });
    }
}


因为transferred * 100l / totalSize始终是介于0到100之间的值,所以您可能会失去小数点精度,但是您绝对不会失去将其强制转换为int来设置进度条值的精度。

10-05 22:37
查看更多