syncd(obj)不起作用，为什么结果可以是-1或0?

Question

syncd(obj) 不起作用，为什么结果可以是 -1 或 0？ 当我使用同步(这个)时，它会没问题 那么为什么会发生这种情况

public class RunnableSales {
public static void main(String[] args) {
Ticket1 t = new Ticket1();
Thread t1 = new Thread(t);
Thread t2 = new Thread(t);
Thread t3 = new Thread(t);
Thread t4 = new Thread(t);
t1.start();
t2.start();
t3.start();
t4.start();
}
}
class Ticket1 implements Runnable {
private int tick = 10;
public void run() {
Object obj = new Object();
while (true) {
synchronized (obj) {
if (tick > 0) {
try {
Thread.sleep(10);
} catch (InterruptedException e) {
}
System.out.println(Thread.currentThread().getName() + "sale:" + tick--);
} else {
break;
}
}
}
}
}

控制台打印为：

Thread-1sale:10
Thread-2sale:10
Thread-3sale:10
Thread-0sale:9
Thread-2sale:8
Thread-3sale:7
Thread-1sale:8
Thread-0sale:8
Thread-1sale:6
Thread-0sale:3
Thread-2sale:5
Thread-3sale:4
Thread-1sale:2
Thread-0sale:1
Thread-2sale:0
Thread-3sale:-1

syncd(obj) 不起作用，为什么结果可以是 -1 或 0？ 当我使用同步(这个)时，它会没问题 那么为什么会发生这种情况

Answer 1

查看代码，发现每次调用run()都会生成一个新的Object obj，该仅用于此调用的同步。因此，实际上没有同步正在进行。修复非常简单：Object obj实例字段。您应该将其设置为final不允许重新分配(这可能会扰乱同步)。

固定版本如下所示：

public class Test {
public static void main(String[] args) {
Ticket1 t = new Ticket1(4);
Thread t1 = new Thread(t);
Thread t2 = new Thread(t);
Thread t3 = new Thread(t);
Thread t4 = new Thread(t);
t1.start();
t2.start();
t3.start();
t4.start();
try{
t1.join();
t2.join();
t3.join();
t4.join();
} catch (InterruptedException e){
}
}
}
class Ticket1 implements Runnable {
private int tick = 10;
private final Object obj = new Object();
public void run() {
while (true) {
synchronized (obj) {
if (tick > 0) {
try {
Thread.sleep(10);
} catch (InterruptedException e) {
}
System.out.println(Thread.currentThread().getName() + "sale:" + tick--);
} else {
break;
}
}
}
}
}

我添加了代码来显式等待线程终止。应始终包含此代码，以确保在主线程退出之前完成所有生成的线程。输出如下所示：

Thread-3sale:10
Thread-3sale:9
Thread-3sale:8
Thread-3sale:7
Thread-3sale:6
Thread-3sale:5
Thread-3sale:4
Thread-3sale:3
Thread-3sale:2
Thread-3sale:1

请注意，最终输出显示的值为1。关于您的程序，这是正确的，因为在tick减少到0后没有输出。

Answer 2

你以为 obj 就像锁一样。
但是当你启动 4 线程时，它们会调用 run 方法，并且每个 obj 都是在每个线程中创建的，所以上面的代码没有锁的意思。
只有刻度在 4 个线程中共享。
如果要正常工作，则需要将 obj 声明为类成员变量。

Answer 3

如注释中所述，您需要将锁定对象设置为类级变量，而不是在 run() 方法中实例化一个新变量。这样，访问此类的每个线程都将使用相同的锁定对象实例，而不是 run() 方法中新创建的实例。

法典

static class Ticket1 implements Runnable{
private final int tick = 10;
private final Object obj = new Object();
public void  run() {
while(true)
{
synchronized(obj)
{
if(tick>0)
{
try {
Thread.sleep(10);
} catch (InterruptedException e) {
}
System.out.println(Thread.currentThread().getName()+"sale:"+tick--);
}
else {
break;
}
}
}
}
}

乌普图特

Thread-0sale:10
Thread-0sale:9
Thread-0sale:8
Thread-0sale:7
Thread-0sale:6
Thread-0sale:5
Thread-3sale:4
Thread-3sale:3
Thread-3sale:2
Thread-3sale:1