使用静态对象锁不起作用？

Question

我是Concurrency的新手，我想了解同步块：

public static int count1 = 0;
public static Object lock1 = new Object();

public static void add(){
        synchronized (lock1) {
            count1++;
        }
}

我的问题是关于lock1的，它不起作用，当该方法开始打印颜色时，它会随机打印它们，所以我认为问题出在同步块中，因为我看了一些有关此的教程，所有人都说锁定对象必须是静态的，因此不会发生干扰，但是在这里我看不到，为什么？

这是打印每个线程颜色的方法：

public static void compute(){
        String color = null;
        switch (Thread.currentThread().getName()) {
            case "First Count Down":
                color = TextColors.ANSI_YELLOW;
                break;
            case "Second Count Down":
                color = TextColors.ANSI_MAGENTA;
                break;
        }
        for (int i=0;i<100;i++) {
            System.out.println(color + Thread.currentThread().getName() + "is Running");
            //add();
            add();
        }
    }

这是线程：

public static void main(String[] args) {

        Thread t1 = new Thread(new Runnable() {
            @Override
            public void run() {
                compute();
            }
        });
        t1.setName("First Count Down");

        Thread t2 = new Thread(new Runnable() {
            @Override
            public void run() {
                compute();
            }
        });
        t2.setName("Second Count Down");

        t1.start();
        t2.start();

        try{
            t1.join();
            t2.join();

        }catch (InterruptedException io){
            io.printStackTrace();

        }

        System.out.println("Count1 = " + count1 + " Count2 = " + count2);

    }

对不起，如果我的英语不好，我不是母语人士，请事先感谢

Answer 1

首先，我想现在我明白了您的问题。您正在尝试打印出自己的行，而又不干扰其他线程的行。

为了实现这一点，您必须“保护”代码的一部分，该部分打印出一行，而不是同时从另一线程中打印出另一行。

可以通过同步这些代码行来完成保护。

您当前仅同步该计数器的加法。

您的锁（您正在锁定的对象，该新Object（）的唯一实例）是静态的，您无需在代码中对其进行更改，因此它必须可以工作。

public static Object lock1 = new Object();

您可以将变量设置为final以使其不可变，但是目前这不是问题。我还是建议。

锁定意味着，如果有任何其他线程登陆（执行）同一行代码（同步块的开头），则只有在阻塞线程放弃其锁定之前，它们才会得到执行。只有当他们持有并要求完全相同的锁时，这才是正确的。而且，由于您仅使用完全相同的新Object（）实例，因此您的锁定应该可以。

当前，您的代码已经建立，add（）方法基本上会等到其中一个线程累加一个。

如果要更改它以便分别打印出行，请尝试像这样同步“行打印块”：

    synchronized (lock1) {
        System.out.println(color + Thread.currentThread().getName() + "is Running");
        add();
    }

并且让计数不同步。

private static void add(){
    count1++;
}

这会起作用，但是大多数情况下，您不想让您的add（）方法不同步。可以从其他线程执行，谁知道。但是对于您而言，这些更改将有所帮助。