为什么此程序只有1个线程在执行？

Question

为什么此程序只有1个线程在执行？
代码如下：

package demo;

public class AddSubDemo {

    public static void main(String[] args) {
        Resource resource = new Resource();
        new Thread(new AddRunnable(resource)).start();
        new Thread(new SubRunnable(resource)).start();
    }

}

class AddRunnable implements Runnable {

    private Resource resource;

    public AddRunnable(Resource resource) {
        this.resource = resource;
    }

    @Override
    public void run() {
        while (true) {
            try {
                resource.add();
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }

    }

}

class SubRunnable implements Runnable {

    private Resource resource;

    public SubRunnable(Resource resource) {
        this.resource = resource;
    }

    @Override
    public void run() {
        while (true) {
            try {
                resource.sub();
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }

    }

}

class Resource {
    private int num = 0;
    private boolean flag = true; // true：执行ADD；false：执行SUB

    public synchronized void add() throws InterruptedException {
        if (this.flag = false) {
            super.wait();
        }
        Thread.sleep(1000);
        this.num++;
        flag = false;
        System.out.println(Thread.currentThread().getName() + " " + this.num);
        super.notifyAll();
    }

    public synchronized void sub() throws InterruptedException {
        if (this.flag == true) {
            super.wait();
        }
        Thread.sleep(2000);
        this.num--;
        flag = true;
        System.out.println(Thread.currentThread().getName() + " " + this.num);
        super.notifyAll();
    }
}

日志如下：

Thread-0：1
Thread-0：2
Thread-0：3
Thread-0：4
Thread-0：5
Thread-0：6
Thread-0：7

Answer 1

以下内容由CHATGPT及阿里嘎多学长共同生成、有用望采纳：

---

从你提供的代码和日志来看，问题出在Resource类中的同步方法add和sub上。这两个方法都使用了wait和notifyAll来控制线程的执行流程，但是存在一个逻辑错误，导致只有一个线程能够执行。

问题在于add方法和sub方法中的条件判断：

public synchronized void add() throws InterruptedException {
    if (this.flag = false) { // 这里应该是 ==
        super.wait();
    }
    // ...
}

public synchronized void sub() throws InterruptedException {
    if (this.flag == true) { // 这里条件是正确的
        super.wait();
    }
    // ...
}

在Java中，==是用于比较两个值是否相等的，而=是赋值操作符。在add方法中，if (this.flag = false)实际上是将false赋值给flag，而不是比较flag是否等于false。因此，这行代码将始终执行super.wait()，导致调用add方法的线程进入等待状态，并且不会唤醒。

由于flag被错误地赋值为false，当sub方法执行时，if (this.flag == true)的条件判断为false，sub方法也会调用super.wait()，导致调用sub方法的线程也进入等待状态。

由于两个线程都因为条件判断错误而进入等待状态，并且没有其他代码来唤醒它们，所以只有一个线程（在这个例子中是Thread-0）能够继续执行并打印日志。

要解决这个问题，你需要将add方法中的条件判断改为比较操作符==，如下所示：

public synchronized void add() throws InterruptedException {
    if (this.flag == false) { // 修正为比较操作符
        super.wait();
    }
    // ...
}

修正后，两个线程应该能够交替执行add和sub方法，从而正确地更新num的值。同时，注意notifyAll()应该在条件满足后立即调用，以唤醒所有等待的线程，而不是在方法的最后。这样可以避免潜在的竞态条件。