0

我正在编写一个进行一些音频分析的应用程序(用作吉他调音器,并添加和弦估计和其他一些东西),但我在使用 GUI 时遇到了一些问题。第一个问题是我有一个按钮,单击该按钮应更改其显示的文本。当它被点击时,文本并没有改变,但是应该改变它的代码行肯定已经到达了。

另一件事是我需要一个 SwingWorker 重复执行(完成后重新启动)并每次更新 GUI。由于我的代码目前是,我有一个循环重复执行我的 SwingWorker,但这导致 GUI 变得无响应,因为它在 EDT 中运行(大概)。让 SwingWorker 重复执行的最佳方法是什么?我应该创建一个新线程来运行循环,还是其他?

我的内部 ActionListener 类的代码如下:

private class TunerListener implements ActionListener {
    private boolean firstUpdate = true;
    private volatile boolean executing = false;

    private final SwingWorker<Void, Void> tunerWorker = new SwingWorker<Void, Void>() {

        @Override
        protected Void doInBackground() {
            model.update();
            return null;
        }

        @Override
        protected void done() {
            if (!this.isCancelled()) {
                prev.setText(model.getPrev());
                currentNote.setText(model.getNote());
                next.setText(model.getNext());
                frequency.setText(model.getFrequency());
                switch (model.getOffset()) {
                    case -2:
                        light_2.setIcon(onRed);
                        light_1.setIcon(off);
                        light0.setIcon(offBig);
                        light1.setIcon(off);
                        light2.setIcon(off);
                        break;
                    case -1:
                        light_2.setIcon(off);
                        light_1.setIcon(onRed);
                        light0.setIcon(offBig);
                        light1.setIcon(off);
                        light2.setIcon(off);
                        break;
                    case 0:
                        light_2.setIcon(off);
                        light_1.setIcon(off);
                        light0.setIcon(onGreen);
                        light1.setIcon(off);
                        light2.setIcon(off);
                        break;
                    case 1:
                        light_2.setIcon(off);
                        light_1.setIcon(off);
                        light0.setIcon(offBig);
                        light1.setIcon(onRed);
                        light2.setIcon(off);
                        break;
                    case 2:
                        light_2.setIcon(off);
                        light_1.setIcon(off);
                        light0.setIcon(offBig);
                        light1.setIcon(off);
                        light2.setIcon(onRed);
                        break;
                }
            }
        }

    };

    @Override
    public void actionPerformed(ActionEvent ae) {

        if (ae.getActionCommand().equals("tune")) {
            if (!executing) {
                tune.setText("Stop Tuning");
                executing = true;

                while (executing) {
                    tunerWorker.execute();
                    firstUpdate = false;
                }
                firstUpdate = true;
            } else {
                tune.setText("Start Tuning");
                executing = false;
                tunerWorker.cancel(true);
                firstUpdate = true;
            }
        }

    }
}

编辑:按钮文本问题现在似乎已经修复,但我仍然无法让这个 SwingWorker 正常工作。我已经尝试完全删除 while 循环并让它从它的 done() 方法中重新执行,但这似乎并没有帮助问题......

4

2 回答 2

6

SwingWorker只设计为执行一次。” 而是控制工人的执行和生命周期,如本所示。

于 2012-05-05T11:39:36.523 回答
5

免责声明:多线程不是我最擅长的领域......

您的两个问题实际上是同一件事,例如,您的 GUI 变得无响应,因为 EDT 正忙于运行 while 循环。这意味着它不能用新的文本值重新绘制按钮,也不能对用户输入做出反应。

此外,每个 SwingWorker 实例只能运行一次,因此execute()在循环中多次调用只会运行一次。

我建议创建一个 TunerWorker 对象并将循环放入其中,然后在需要启动循环时创建一个新对象。像这样:

class TunerListener implements ActionListener {

private TunerWorker tw = null;

@Override
public void actionPerformed(ActionEvent ae) {

    if (ae.getActionCommand().equals("tune")) {
        if (tw == null || tw.isDone()) {
            tune.setText("Stop Tuning");
            executing = true;

            tw = new TunerWorker();
            tw.execute();

        } else {
            tune.setText("Start Tuning");
            executing = false;
            tw.cancel(true);
            }
        }
    }
}

final class TunerWorker extends SwingWorker<Void, Void> {

    @Override
    protected Void doInBackground() {
        while (!this.isCancelled()) {
            model.update();
        }        
        return null;
    }    

    @Override
    protected void done() {
        if (!this.isCancelled()) {
            //Removed this code to make the example prettier...
        }
    }
}

哦,我不确定你要做什么,firstUpdate所以我把它从例子中删除了。希望弄清楚如何将其放回原处不会太难。

编辑:哎呀,该代码实际上并没有工作。现在应该修好了。

于 2012-05-05T11:42:25.430 回答