2016-08-15 50 views
1

我做了SwingWorker的快速和脏实现来测试publish()和setProgress()方法。当我为setProgress()方法设置参数(如下面完整源代码中所述)时,所有内容都按预期工作。 但是如果我设置setProgress参数是这样的: setProgress((int) ((i/2000) * 100)); 或者这样 setProgress((int) (i * (100/2000)));SwingWorker中setProgress的无法解释的行为

以来的propertyChange不被解雇的进度不更新。

我严重不知道这是为什么,因为在我看来,我看不出有什么错数学有...

任何想法?

在此先感谢!

问候

package test; 

import java.awt.BorderLayout; 
import java.awt.Dimension; 
import java.util.LinkedList; 
import java.util.List; 
import javax.swing.JButton; 
import javax.swing.JCheckBox; 
import javax.swing.JFrame; 
import javax.swing.JPanel; 
import javax.swing.JProgressBar; 
import javax.swing.JScrollPane; 
import javax.swing.JTextPane; 
import javax.swing.SwingWorker; 
import javax.swing.text.BadLocationException; 
import javax.swing.text.StyledDocument; 

public class ConsoleTest extends SwingWorker<Void, String>{ 

private JTextPane console; 
private JProgressBar pb; 

public ConsoleTest(JProgressBar pb, JTextPane out){ 

    this.console = out; 
    this.pb = pb; 
} 

private static void createGUI(){ 

    JFrame container = new JFrame(); 
    container.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
    JTextPane console = new JTextPane(); 
    JButton start = new JButton("start"); 
    JButton reset = new JButton("reset"); 
    JPanel btnpane = new JPanel(); 
    btnpane.add(start); 
    btnpane.add(reset); 
    btnpane.add(new JCheckBox("Click me to proof UI is responsive")); 
    JProgressBar pb = new JProgressBar(0,100); 
    btnpane.add(pb); 
    container.add(btnpane, BorderLayout.SOUTH); 

    JScrollPane sp = new JScrollPane(console); 
    sp.setPreferredSize(new Dimension(800,800)); 
    container.add(sp,BorderLayout.NORTH); 

    start.addActionListener(e -> { ConsoleTest test = new ConsoleTest(pb, console); 
            test.addPropertyChangeListener(evt -> { if ("progress".equals(evt.getPropertyName())) { 
                      System.out.println("check"); 
                      test.getPG().setValue((int)evt.getNewValue()); 
                      }                     
                     }); 
            test.execute(); 
           }); 
    reset.addActionListener(e -> clear(console)); 
    container.pack(); 
    container.setVisible(true); 


} 

private static void clear(JTextPane console) { 

    console.setText(""); 
} 



public JProgressBar getPG(){ 

    return this.pb; 
} 

@Override 
protected Void doInBackground() throws Exception { 

    LinkedList<Integer> list = new LinkedList<>(); 
    for (int i = 0; i< 2000;i++){ 
     list.add(i); 
     if(((i+1) % 5) == 0) 
      publish(Integer.toString(i+1)); 
     setProgress((int) (i*100/2000)); 
     Thread.sleep(1);    
    } 
    return null; 
} 

@Override 
protected void process(List<String> chunks){ 

     StyledDocument doc = this.console.getStyledDocument(); 
     try { 
      for(String s: chunks) 
      doc.insertString(doc.getLength(),"- "+s+" elements added to the list\n", null); 
     } catch (BadLocationException e) { 
      e.printStackTrace(); 
     }  
} 


@Override 
protected void done() { 

    try { 
     System.out.println("DONE!!!"); 
    } 
    catch (Exception ignore) { 
    } 
} 

public static void main(String[] args) { 

    createGUI(); 
} 

}

+0

莫名其妙编辑器 “吃” 的muliply标志 – motaa

回答

3

你使用整数工作!

setProgress((int) ((i/2000)*100)); 

总是0因为(int)i/2000总是0

同样的问题

setProgress((int) (i*(100/2000))); 

100/2000总是0整数

尝试

setProgress((int) (((float)i/2000)*100)); 

setProgress((int) (i*((float)100/2000))); 

,也将努力

+0

holy钼!我怎么可能没有看到这一点....猜我在我的脑海中得到了一个虚空:) – motaa

0

你应该让(I * 100)/ 2000 公式是

(current_state * 100)/ final_state

+0

莫名其妙编辑吃了我的 “*” 的迹象。那个文档中的任何地方都是这样写的吗?从数学上讲, (i * 100)/ 2000与i *(100/2000) – motaa

+0

相同,因为您将int用作最终类型而不进行浮动投球100/2000始终为零,因此进度保持为0. – whyn0t