JProgressBar не работает должным образом

Так что мой JProgressBar Я настроил не работает так, как я хочу. Поэтому, когда я запускаю программу, она мгновенно переключается с 0 на 100. Я пытался с помощью ProgressMonitorЗадача, и попробовал SwingWorker но ничего, что я пробовал, не работает.

Вот моя программа:

int max = 10;
for (int i = 0; i <= max; i++) {
        final int progress = (int)Math.round(
            100.0 * ((double)i / (double)max)
        );

        EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                try {
                    Thread.sleep(1000);
                } catch (InterruptedException ex) {
                    Logger.getLogger(BandListGenerator.class.getName()).log(Level.SEVERE, null, ex);
                }
                jProgressBar2.setValue(progress);
            }
        });
    }

@MadProgrammer Вот моя попытка сделать рабочий свинг и записать каждое имя в документ и обновить индикатор выполнения. Программа достигает примерно 86 процентов и останавливается, никогда не создавая готовый документ. Программа создает пустой документ. Вот два метода, во-первых, объект SwingWorker, который я сделал:

public class GreenWorker extends SwingWorker<Object, Object> {
    @Override
    protected Object doInBackground() throws Exception {
        int max = greenList.size();
        XWPFParagraph tmpParagraph;
        XWPFRun tmpRun;
        FileInputStream file = 
                new FileInputStream(location + "GreenBandList.docx");
        gDocument = new XWPFDocument(OPCPackage.open(file));
        for (int i = 0; i < max; i++) {
            tmpParagraph = gDocument.getParagraphs().get(0);
            tmpRun = tmpParagraph.createRun();
            if (greenList.get(i).length() == 1) {
                    tmpRun.setBold(true);
                    tmpRun.setText(greenList.get(i));
                    tmpRun.setBold(false);
            } else {
                    tmpRun.setText(greenList.get(i));//Write the names to the Word Doc
            }
            int progress = Math.round(((float) i / max) * 100f);
            setProgress(progress);
        }
        return null;
  }        
}

И вот код для кнопки, которая запускает ее и имеет мое событие изменения свойства.

private void GenerateGreenList() throws IOException, InterruptedException {
    //Need to fix the bug that removes the Letter Header in Yellow Band list
    //********************************************************************\\

    //Delete the old list and make a new one
    File templateFile = new File(location + "\\backup\\GreenTemplate.docx");
    FileUtils.deleteQuietly(new File(location + "GreenBandList.docx"));
    FileUtils.copyFile(templateFile, new File(location + 
            "GreenBandList.docx"));
    //Get the New Entries
    String[] entries = jTextPane3.getText().split("\n");
    for (String s : entries) {
        if (s != null) {
            greenList.add(s);
        }
    }
    //Resort the list
    Collections.sort(greenList);
    //Write the names to the document
    GreenWorker worker = new GreenWorker();
    worker.addPropertyChangeListener(new PropertyChangeListener() {
        @Override
        public void propertyChange(PropertyChangeEvent evt) {
            if ("progress".equals(evt.getPropertyName())) {
                jProgressBar2.setValue((Integer) evt.getNewValue());
            }
        }
    });
    worker.execute();
    if (worker.isDone()) {
        try {
            gDocument.write(new FileOutputStream(new File(location + "GreenBandList.docx")));
            ////////////////////////////////////////////////////////////
        } catch (IOException ex) {
            Logger.getLogger(BandListGenerator.class.getName()).log(Level.SEVERE, null, ex);
        }
        JOptionPane.showMessageDialog(null, "Green Band List Created!");
        jProgressBar2.setValue(0);
    }
}

Я использовал слушателя изменения свойств из одного из ваших других постов, но я не совсем понимаю, что делает тот, который вы написали, или что он вообще делает?

2 ответа

Решение

Swing - это однопотоковая среда, то есть единый поток, который отвечает за обработку всех событий, происходящих в системе, включая события перерисовки. Если что-то заблокирует этот поток по какой-либо причине, он не позволит Swing обрабатывать любые новые события, включая события перерисовки...

Так что все это...

EventQueue.invokeLater(new Runnable() {
    @Override
    public void run() {
        try {
            Thread.sleep(1000);
        } catch (InterruptedException ex) {
            Logger.getLogger(BandListGenerator.class.getName()).log(Level.SEVERE, null, ex);        }
        jProgressBar2.setValue(progress);
    }
});

Постоянно приостанавливает поток диспетчеризации событий, не позволяя ему делать какие-либо обновления (или, по крайней мере, делать интервалы между ними)...

Также вероятно, что ваш внешний цикл был запущен из контекста EDT, а это означает, что пока он не существует, ничего в очереди событий не будет обработано. Все ваши запросы на перекрашивание будут объединены в один запрос на рисование и вуаля, мгновенно заполненный индикатор выполнения...

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

И прости меня, если мы не говорили об этом несколько раз раньше:P

Вы вызываете Thread.sleep внутри EvokeLater Это означает, что он работает в другом потоке, чем ваш цикл for. т. е. ваш цикл for завершается мгновенно (ну, однако, сколько времени занимает цикл от 1 до 100, что почти мгновенно).

Переехать Thread.sleep вне EvokeLater и это должно работать так, как вы собираетесь.

    int max = 10;
    for (int i = 0; i <= max; i++) {
        final int progress = (int)Math.round(
            100.0 * ((double)i / (double)max)
        );

        EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                jProgressBar2.setValue(progress);
            }
        });
        try {
            Thread.sleep(1000);
        } catch (InterruptedException ex) {
            Logger.getLogger(BandListGenerator.class.getName()).log(Level.SEVERE, null, ex);
        }
    }

Изменить: согласен с @MadProgrammer. Похоже, это просто иллюстративный вопрос, но вы должны убедиться, что все, что вы пытаетесь достичь здесь, вы используете SwingWorker за.

Другие вопросы по тегам