swing Java SwingWorker挂起
我正在调试一些使用SwingWorker编写的代码,以执行数值计算和GUI更新的混合。SwingWorker使用以下堆栈跟踪挂起:
Full thread dump Java HotSpot(TM) Client VM (14.3-b01 mixed mode, sharing):
"SwingWorker-pool-3-thread-4" prio=6 tid=0x07fd7c00 nid=0x143c waiting on condition [0x0a33f000]
java.lang.Thread.State: TIMED_WAITING (sleeping)
at java.lang.Thread.sleep(Native Method)
at sum.ee.ui.modelviewer.ModelViewer$ModelAnimator.doInBackground(ModelViewer.java:940)
at sum.ee.ui.modelviewer.ModelViewer$ModelAnimator.doInBackground(ModelViewer.java:877)
at javax.swing.SwingWorker$1.call(SwingWorker.java:274)
at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:303)
at java.util.concurrent.FutureTask.run(FutureTask.java:138)
at javax.swing.SwingWorker.run(SwingWorker.java:313)
at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
at java.lang.Thread.run(Thread.java:619)
"SwingWorker-pool-3-thread-3" prio=6 tid=0x07fd7000 nid=0x11a8 waiting for monitor entry [0x0a2af000]
java.lang.Thread.State: BLOCKED (on object monitor)
at java.awt.Component.resize(Component.java:2044)
- waiting to lock <0x24b936a0> (a java.awt.Component$AWTTreeLock)
at java.awt.Component.setSize(Component.java:2035)
at java.awt.Component.resize(Component.java:2069)
at java.awt.Component.setSize(Component.java:2060)
at javax.swing.JViewport.setViewSize(JViewport.java:1038)
at javax.swing.ViewportLayout.layoutContainer(ViewportLayout.java:183)
at java.awt.Container.layout(Container.java:1421)
at java.awt.Container.doLayout(Container.java:1410)
at jsyntaxpane.components.LineNumbersRuler.updateSize(LineNumbersRuler.java:109)
at jsyntaxpane.components.LineNumbersRuler.removeUpdate(LineNumbersRuler.java:203)
at javax.swing.text.AbstractDocument.fireRemoveUpdate(AbstractDocument.java:243)
at jsyntaxpane.SyntaxDocument.fireRemoveUpdate(SyntaxDocument.java:118)
at javax.swing.text.AbstractDocument.handleRemove(AbstractDocument.java:608)
at javax.swing.text.AbstractDocument.remove(AbstractDocument.java:576)
at javax.swing.JEditorPane.setText(JEditorPane.java:1493)
at sum.ee.ui.SourceCodePanel.clearSourcePane(SourceCodePanel.java:256)
at sum.ee.ui.SourceCodePanel.access$100(SourceCodePanel.java:47)
at sum.ee.ui.SourceCodePanel$1.stateChanged(SourceCodePanel.java:209)
at sum.ee.ui.VisualizationAggregator.fireStateChanged(VisualizationAggregator.java:300)
at sum.ee.ui.VisualizationAggregator.update(VisualizationAggregator.java:97)
at sum.ee.ui.modelviewer.ModelViewer$ModelAnimator.doInBackground(ModelViewer.java:918)
at sum.ee.ui.modelviewer.ModelViewer$ModelAnimator.doInBackground(ModelViewer.java:877)
at javax.swing.SwingWorker$1.call(SwingWorker.java:274)
at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:303)
at java.util.concurrent.FutureTask.run(FutureTask.java:138)
at javax.swing.SwingWorker.run(SwingWorker.java:313)
at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
at java.util.concurrent.ThreadPoolExecutor
$Worker。运行(ThreadPoolExecutor.java:908) 在爪哇。朗。丝线。运行(Thread.java:619)
我的理解是GUI工作不应该在doInBackground()内部完成,而应该在done()中完成。我做了一个简单的实验,将doInBackground()中的所有代码都移到了done()中,但仍然不起作用。关于我能做些什么来解决这个问题,大家能给我一些建议吗?代码如下所示:
protected Void doInBackground() {
isAnimating = true;
resetButtonBackgrounds();
backgroundColor = new Color(175, 255, 175); // Soft Green
JToggleButton b = null;
for (final int index : modelIndices) {
if (index == modelIndices.get(modelIndices.size() - 1)) {
backgroundColor = defaultBackgroundColor;
}
if (!keepTrace) {
// Resetting the backgrounds is necessary to have
// individual display of the changing elements due to
// the fact that there can be multiple nodes per
// source line. The reset works in combination
// with updating from ModelViewer.this (as opposed
// to the 'this' of ModelAnimator instances) due
// to not sending an event to itself. Furthermore,
// if the event was sent from ModelAnimator, the model
// indices are recalculated, causing a jump when multiple
// element source lines are encountered.
resetButtonBackgrounds();
}
aggregator.modelIndex(index);
aggregator.update(ModelViewer.this);
b = getButtonByIndex(index);
scrollRectToVisible(b.getBounds());
ModelViewer.this.repaint();
try {
StaticTools.sleepAtLeast(sleepTimeMilliseconds);
} catch (final InterruptedException ex) {
// continue with thread
}
}
isAnimating = false;
if ( b != null) {
Color orig = b.getBackground();
Color blink = Color.PINK;
Color current = orig;
for (int i = 0; i < 100; i++) {
try {
Thread.sleep(100);
} catch (InterruptedException ex) {
}
if (current == orig) {
current = blink;
} else {
current = orig;
}
b.setBackground(current);
ModelViewer.this.repaint();
}
}
return null;
}
另一个线索是有两个SwingWorker线程执行。它们可以运行相同的线程吗
更新:以下是执行SwingWorker的代码:
公共最终无效动画(更新之间的最终长延迟为毫秒, 最后的模型索引列表, 最后的布尔竞速, 最终列表属性更改侦听器){
ModelAnimator modelAnimator =
new ModelAnimator(delayBetweenUpdatesMilliseconds, modelIndices,
keepTrace);
for (final PropertyChangeListener listener : propertyChangeListeners) {
modelAnimator.addPropertyChangeListener(listener);
}
modelAnimator.execute();
}
# 1 楼答案
这是因为没有遵守Swing EDT规则
SwingWorker的目的是在UI事件发生时执行繁重的非UI任务,否则会阻塞UI,然后在最后更新UI
所以你要在doInBackground()内进行举重;然后,swing一旦完成,就会在EDT上调用done(),您可以使用get()检索结果
这里的问题是,您正在SwingWorker创建的新线程中进行GUI工作。这可能会导致死锁和并发问题
这包括创建所述GUI对象,即使您已经在EDT上,这些对象也应该是可运行的
行动,例如:
应该封装在带有InvokeandWait的Runnable中。实际上修改GUI本身的东西尤其需要在它们自己的可运行程序中,甚至如果你已经在swing事件中调度线程响应按钮的按下或更改,你就有可能处理你已经在处理的对象
例如,swing在A上工作并锁定,让你工作,让你在B上工作试图锁定并在A上工作