多线程无法正常工作

时间:2013-07-13 22:10:59

标签: java multithreading swing

修改 这篇文章涉及我在学校的家庭作业,指示我依靠swing来显示我的线程和布尔标志来阻止。

我的应用程序创建了一堆“作业”对象,每个对象都包含一个线程。每个工作都属于一个生物。一个生物可以拥有多个作业,但在任何特定时刻只能执行其中一个作业。

我的线程使用2个布尔标志来判断它是否应该运行名为“killFlag”和“goFlag”。它将它所属的生物指定为“目标”。每个目标都有一个布尔值“isWorking”来表示它是否忙于另一个工作。

这是每个作业应运行的线程:

public void run() {
    long time = System.currentTimeMillis();
    long startTime = time;
    long stopTime = time + 1000 *  (long)( jobTime );
    double duration = stopTime - time;



    synchronized (this.target) {
        while (this.target.isWorking) {
            status = 'w';
            showStatus(); // hmmmmmmmm
            try {
                this.target.wait();
            } catch (InterruptedException e) {
            }
        }

        this.target.isWorking = true;
    }

    while (time < stopTime && !killFlag) {
        try {
            TimeUnit.MILLISECONDS.sleep(100);
        } catch (InterruptedException e) {
        }

        if (goFlag) {
            status = 'p';
            showStatus();
            time += 100;
            this.showProgress.setValue((int)(((time - startTime) / duration) * 100));
        } else {
            status = 'r';
            showStatus();
        }
    }//End While loop here

        showProgress.setValue(100);
        status = 'c';
        showStatus();
        synchronized (target) {
            target.isWorking = false;
            target.notifyAll();

    }
}

起初我认为它是target.notifyAll()因为它正在抛出一个IllegalMonitorStateException,但当我将它注释掉时,线程会构造对象,但是当我在GUI中查看它们时,80%的是显示完整而没有任何与我的任何关系,其他20%表示该生物正忙着。

起初我以为这是因为我过早地弹出了杀戮旗帜但是当我将它降低或移除时,症状仍然存在。我现在已部署,这里没有程序员哈哈,你能提供的任何建议都意味着世界。

为了确保我在下面提供足够的信息,我使用的方法是与线程交互。以下方法使用一个按钮,该按钮根据线程是否正在运行而发生变化。

public void showStatus() { //switch that changes status of button used to start / pause / display status of thread
    switch (this.status) {
        case 'r' :
            startJob.setEnabled(true);
            startJob.setText ("Run");
            break;
        case 'p' :
            startJob.setEnabled(true);
            startJob.setText("Pause");
            break;
        case 'w' :
            startJob.setEnabled(false);
            startJob.setText("Working");
            break;
        case 'c' :
            startJob.setEnabled(false);
            startJob.setText("Job Complete");
            break;
    }
}

private class theHandler implements ActionListener {//Listener for Button mentioned above
    public void actionPerformed (ActionEvent event) {
        if (event.getSource() == startJob) {
            if (goFlag) {
                goFlag = false;
            } else {
                goFlag = true;
                killFlag = false;
            }
        } else if (event.getSource() == stopJob) {
            if (killFlag) {
                //do nothing
            } else {
                killFlag = true;
                status = 'r';
            }
        }
    }
}

这让我很伤心,我一直在四处寻找解决这个问题6个小时。

修改

根据MadProgrammer的评论调整我的代码后,修复了“target.notifyAll()”。现在问题似乎是所有线程在显示器上都显示为完整,即使按钮在状态之间随机闪烁几分之一秒。

修改

以下内容包含了大量针对评论的修改

下面是我定义作业类的方法,其中定义了killFlag,goFlag等。

  class Job extends Item implements SearchableByName, Runnable {
int                                         index;
String                                      name;
int                                         creature;
double                                      jobTime;
Creature                                    target;
boolean                                     goFlag = false;
boolean                                     killFlag = false;
char                                        status;
JButton                                     startJob;
JButton                                     stopJob;
JProgressBar                                showProgress;
JPanel                                      p1;

下面是定义了生物(目标)的位置,其中布尔值是工作所在:

 class Creature extends Entity implements SearchableByName, SearchableByType, Runnable {
int                                     party;
int                                     empathy;
int                                     fear;
int                                     carryCapacity;
Float                                   age;
Float                                   height;
Float                                   weight;
boolean                                 isWorking = false;

回应这里的评论是我如何显示线程的图片: enter image description here

1 个答案:

答案 0 :(得分:2)

由于某些代码仍然缺失,我将首先列举一些假设;如果有任何不成立,我的答案可能不正确。

  1. 您提供的run()方法位于Job类内。
  2. 您引用的“取消”按钮是stopJob JButton
  3. 如果您打算在第一次运行后重新使用相同的Job实例进行重新启动,那么您的基本问题是您的run()方法终止。你有一个while循环来检查!killFlag,但是一旦该循环结束(即一旦作业被取消并且killFlag == true),就没有任何东西会导致它回到开始并等待后续的goFlag == true状态再次开始运行。 (另外,请考虑stopJob方法中的actionPerformed()条款是否需要对goFlag执行某些操作。)

    另一方面,如果您打算创建一个新的Job实例来表示重新启动的作业,那么您还没有显示任何可以执行此操作的代码。

    我上面的诊断有点模糊不清,试图帮助你自己解决问题 - 这是最好的学习方法。 :)如果您需要更多细节,我可以尝试提供它们,只需在评论中提供LMK。