我有一个控制器类,如下所示:
public class Controller {
private Driver driver;
public void method1() {/**/}
public void method2() {/**/}
public void method3() {/**/}
}
每种方法都使用driver
来执行某些操作。 然而,method2
是一个运行时间较长的进程,需要锁定driver
,因为如果其他方法在method2
运行时使用该driver
,则会发生错误。
现在按照我目前的设置方式,我已经在控制器的调用类中实现了具有公平调度的ReentrantLock
,该负责线程安全,如下所示:
public class Caller {
private static final Lock LOCK = new ReentrantLock(true);
private Controller controller;
public void startStuff() { // Runs in multiple threads.
...
...
LOCK.lock();
try {
controller.method1();
} finally {
LOCK.unlock();
}
...
...
LOCK.lock();
try {
controller.method2();
} finally {
LOCK.unlock();
}
}
}
相反,我想使Controller
类线程安全,而不是让任何调用类处理它。我假设这样做的方法是在每个方法的开头和结尾实现 try-finally 块,但我在类中有大约 15 个方法。这真的是这样做的方法吗?这似乎是一吨样板代码:
public class Controller {
private static final Lock LOCK = new ReentrantLock(true);
private Driver driver;
public void method1() {
LOCK.lock();
try {
/**/
} finally {
LOCK.unlock();
}
}
public void method2() {
LOCK.lock();
try {
/**/
} finally {
LOCK.unlock();
}
}
public void method3() {
LOCK.lock();
try {
/**/
} finally {
LOCK.unlock();
}
}
}
我不能只向每种方法添加synchronized
,因为我需要公平的计划才能使应用程序正常工作,并且使用ReentrantLock
可以完美地工作。
我如何使用ReentrantLock
使Controller
类线程安全?
您可以将锁定代码包装到一个块中,并将实际的代码实现传递给它,以便删除重复/样板。 类似于以下内容(Java 7):
public void method1() {
withLock(new LockableMethod() {
public void run() {
/** your method1 code here **/
}
});
}
private void withLock(LockableMethod lockableMethod) {
LOCK.lock();
try {
lockableMethod.run();
} finally {
LOCK.unlock();
}
}
如果您使用的是Java 8,则只需传入Lambda即可删除更多样板文件:
public void method1() {
withLock(() -> {
/** your method1 code here **/
}
);
}
private void withLock(Consumer<T> consumer) {
LOCK.lock();
try {
consumer.accept();
} finally {
LOCK.unlock();
}
}