关于java线程的问题，是死锁了吗？

Question

背景：今天朋友拿了一个问题找到我，我看来半天是没看出什么问题出来，有没有看出问题所在的，还望指教一下！谢谢！！

业务需求：在main方法中启动两个线程，第1个线程循环随机打印100以内的整数，直到第2个线程从键盘读取了“Q”命令。

问题：我正确的在控制台输入了q，可是子线程A并没有停止，按照逻辑，线程B输入Q的时候会调用线程A的set方法把loop的值改成false，但这个时候线程A中while的判断条件loop的值并没有改变，这是为什么了？

import java.util.Scanner;
/*·
   (1)在main方法中启动两个线程
   (2)第1个线程循环随机打印100以内的整数
   (3)直到第2个线程从键盘读取了“Q”命令。
*/
public class HomeWork01 {

    
    public static void main(String[] args) throws InterruptedException {
        A a = new A();
        B b = new B();
        a.start();
        b.start();
    }
}

class A extends  Thread{
    
    private boolean loop = true;

    public void setLoop(boolean loop) {
        this.loop = loop;
    }

    @Override
    public void run() {
        while (loop){
            System.out.println((int)(Math.random()*(100-0)));
            try {
                Thread.sleep(2000);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
    }
}

class B extends Thread{

    @Override
    public void run() {
        Scanner sca = new Scanner(System.in);
        while (true){
            System.out.println("输入Q结束:");
            char end = sca.next().toUpperCase().charAt(0);

            System.out.println("-----"+end);

            if (end == 'Q'){
                System.out.println("Q进来了----");
                new A().setLoop(false);
                System.out.println("循环退出");
                break;
            }
        }
    }
}

Answer 1

你在 main 函数里面创建了一个 A 线程的对象，并将其拉起执行。
然后，在输入Q之后的 B 线程中，又创建了一个 A 线程的对象。
前后创建的两个 A 线程对象，都是独立的内存对象，彼此之间没有任何关系。
setLoop(false) 这个值，不能从新创建的对象中传递到已经创建完的老的对象当中去，类和对象是具有封装属性的。

---

正确的方式，应该是：

1. 将 boolean loop 变量提取出来，让两个线程都能访问到这个变量，（扩大变量的作用域到两个线程类）。
2. 让B对象持有A对象，然后在输入Q之后调用a.setLoop(fasle)方法。
   上述两种方案，二选一即可。

---

另外，基于线程安全的考虑， boolean loop 最好加上 volatile 关键字。