java - 局部变量同步，比同步方法更好？

Question

我编写了一个围绕数据库查询的包装器，需要从不同的线程访问它。因此，我的应用程序恰好创建了该帮助程序类的一个实例，并通过 getter 返回它。

DbConnection dbc = app.getDatabaseConnection();
synchronized (dbc) {
  dbc.doSomething();
}

这段代码安全吗？如前所述here尽管在局部变量上同步，但它应该可以工作。只要保证对象实例相同，这是否正确？

使 DbConnection 的所有受影响的实例方法同步是否是更好的方法？

Answer 1

这不是好的设计。

不是通过在必要时同步其方法/ block 来使 DBConnection 类本质上是线程安全的，而是强制此类的所有客户端在每次需要时显式同步。因此，您不是将线程安全性封装在单个、明确的类中，而是将该责任分配给该类的所有客户端，从而使整个事情变得极其脆弱，并且潜在的错误极难发现。

也就是说，从多个线程使用单个数据库连接本身就是一个坏主意。