多线程无法正常工作

Question

编辑 这篇文章涉及我在学校的家庭作业,指示我依靠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()".现在问题似乎是所有线程在显示器上都显示为完整,即使按钮在状态之间随机闪烁几分之一秒.

编辑

下面包含了大量编辑以回应评论

下面是我如何定义jobFlag,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;

下面是定义了Creature(target)的位置,其中boolean isWorking驻留在:

 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;

并且响应这里的评论是我如何显示线程的图片:

Answer 1

由于仍然缺少一些代码，我将首先列举一些假设；如果任何不成立，我下面的答案可能是不正确的。

1. 您提供的方法run()在Job类内部。
2. 您提到的“取消”按钮是stopJob JButton

如果您打算Job在第一次运行后重新启动相同的实例，那么您的基本问题是您的run()方法终止。您有一个while检查 的循环!killFlag，但是一旦该循环结束（即一旦作业被取消并且killFlag == true），就没有什么会导致它返回到开头并等待后续goFlag == true状态再次开始运行。（另外，请考虑方法stopJob中的子句是否actionPerformed()需要对 执行某些操作goFlag。）

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

我故意在上面的诊断中有点含糊，试图帮助你自己解决问题——这是最好的学习方式。:) 如果您需要更多详细信息，我可以尝试提供，只需在评论中LMK即可。