我正在编写一个实用程序类来实现秒表的行为.这个类的一个重要特性是它被设计为线程安全的.我使用一个名为lock的私有final字段进行同步.该类中的两个方法称为reset和start,它们分别重置秒表并启动它.它们被实现为：

public void reset() {
    synchronized (lock) {
        beginTime = 0;
        lapIndex = 0;
    }
}

public void start() {
    synchronized (lock) {
        if (beginTime == 0) {
            beginTime = System.nanoTime();
            laps[lapIndex++] = beginTime;
        }
    }
}

最近,我想到了添加一个名为restart的额外便捷方法,它会重置并启动秒表.我希望它的行为类似于原子操作,所以我的想法是将其实现为：

public void restart() {
    synchronized(lock) {
        reset();
        start();
    }
}

但是,reset和start方法已在锁定时同步,因此调用restart会在同一对象上同步两次.可能会出现任何问题吗？是否多次同步同一对象的行为定义明确？有必要吗？我已经运行了代码,因为它似乎工作正常,但我担心我可能会遗漏一些微线程,这是多线程常见的.

解决方法:

同步两次没问题.该线程已经拥有了监视器,因此额外的同步并没有真正做多少.这是必要的,否则你的restart()方法可能会被调用reset()和start()之间的另一个线程中断.

避免双重同步的一种方法是使restart(),reset()和start()同步,然后委托给非同步的内部方法.

public void reset() {
    synchronized (lock) {
        _reset();
    }
}

public void start() {
    synchronized (lock) {
        _start();
    }
}

public void restart() {
    synchronized(lock) {
        _reset();
        _start();
    }
}

private void _reset() {
    beginTime = 0;
    lapIndex = 0;
}

private void _start() {
    if (beginTime == 0) {
        beginTime = System.nanoTime();
        laps[lapIndex++] = beginTime;
    }
}