8

编辑 这篇文章与我的学校作业有关，该作业要求我依靠摇摆来显示我的线程和布尔标志以进行阻塞。

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

我的线程使用 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;

下面是定义布尔 isWorking 所在的 Creature (target) 的位置：

 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;

作为对评论的回应，这里有一张我如何显示线程的图片： 在此处输入图像描述

4

1 回答 1

2

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

  1. run()你给的方法在Job类里面。
  2. 您所指的“取消”按钮是stopJob JButton

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

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

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

于 2013-07-14T03:00:04.510 回答