为什么同步无法正常工作？

Question

这是我的代码：

private int count = 0;

  public synchronized void increment() {
      count++;
  }

 public void doWork() throws InterruptedException {

    Thread t1 = new Thread(new Runnable() {
        public void run() {
            for (int i = 0; i < 5; i++) {
                increment();
                System.out.println(count+"  "+Thread.currentThread().getName());
            }}});

    Thread t2 = new Thread(new Runnable() {
        public void run() {
            for (int i = 0; i < 5; i++) {
                increment();
                System.out.println(count+"  "+Thread.currentThread().getName());
            }}});

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

这是我的输出：

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

我的理解increment
是synchronized。因此，应该首先increment一个号码，然后松开lock，然后给lock到线t1或t2。所以，一次应该有increment一个数字，对吧？

但是为什么我的代码一次是incrementing两个或三个数字？我做错什么了吗（我是新手）？

Answer 1

虽然count++;确实不是同步System.out.println(count+" "+Thread.currentThread().getName());的，但是它访问count变量。

即使您同步访问权限，它也无济于事，因为下一种情况仍然可能：

• 线程1增量
• 线程2增量
• 线程1的打印值 2
• 线程2的打印值 2

要解决此问题，您需要增加并在同一同步部分中打印。例如你可以把System.out.println(count+" "+Thread.currentThread().getName());进入increment的方法。