(忽略下面的句子-我很愚蠢)。我有一种感觉,我正在调用EDT上的jProgressBar的更新(我知道这是错误的)。
@Action
public Task launchScenario() {
return new LaunchScenarioTask(getApplication());
}
private class LaunchScenarioTask extends org.jdesktop.application.Task<Object, Void> {
LaunchScenarioTask(org.jdesktop.application.Application app) {
super(app);
launchTestCaseButton.setEnabled(false);
progressBar.setIndeterminate(false);
statusPanel.validate();
statusPanel.repaint();
totalTests = scenarioRepeat * selectedSSIDS.length;
setProgress(0);
}
@Override
protected Object doInBackground() {
currentScenario = 1;
progressBar.setMaximum(totalTests);
progressBar.setMinimum(0);
try {
Thread.sleep((long) 500);
} catch (InterruptedException ignore) {
}
int ssidsToTest = selectedSSIDS.length;
//For every SSID, we iterate through and create an autoconnector object
for (int counter = 0; counter < ssidsToTest; counter++) {
try {
setMessage("Preparing " + selectedSSIDS[counter] + " test case...");
Thread.sleep((long) 2000);
} catch (InterruptedException ignore) {
}
setMessage("Launching ");
AutoConnector ac = new AutoConnector(selectedSSIDS[counter]);
for (int i = 0; i < scenarioRepeat; i++) {
if (ac.connected()) {
setMessage("Running test " + currentScenario + "/" + totalTests);
currentScenario++;
passedTests++;
} else {
currentScenario++;
failedTests++;
}
System.out.println("setting progress");
setProgress((currentScenario / totalTests) * 100);
progressBar.setString((currentScenario / totalTests) * 100 + "% complete");
}
}
try {
Thread.sleep((long) 500);
} catch (InterruptedException ignore) {
}
return null; // return your result
}
@Override
protected void succeeded(Object result) {
launchTestCaseButton.setEnabled(true);
setMessage("Complete. " + passedTests + " tests passed and "
+ failedTests + " tests failed.");
}
}
我知道一个事实(感觉有点糟糕),我调用更新(setProgress()
和setMessage()
在doInBackground()
方法,我有一个偷偷摸摸的怀疑,我不应该。& lt; & lt;
所以问题是,我应该把setProgress()
片段放在代码的哪里?这是否需要我用AutoConnector
类创建一个任务,看看我是否可以从那里更新jProgressBar ?
我以前使用过jProgressBars,但不是以这种方式使用NetBeans。
你的问题与你在问题开始时所说的正好相反。你状态:
我有一种感觉,我在EDT上调用更新到我的jProgressBar(我知道这是错误的)。
而事实正好相反。对JProgressBar的所有更新都应该在EDT的上进行,而您所做的恰恰相反。你不应该从SwingWorker的doInBackground(...)
方法中调用JProgressBar上的任何方法。
考虑
- 改变一个被监听的属性,比如SwingWorker的progress属性,并让JProgressBar在PropertyChangeListener中得到更新,PropertyChangeListener会监听这个属性的变化,或者
- 使用SwingWorker的publish/process方法对允许你将数据从SwingWorker传输到EDT上的GUI。
我自己,我更喜欢添加一个PropertyChangeListener到我的SwingWorker(在EDT),并从这个监听器中更新我的JProgressBar。我觉得与第一种方法相比,它允许更好的代码分离,具有更低的耦合和更高的内聚。