是否通过使用同步安全地访问此变量？

Question

我有一个案例，其中 Java 类有一个包含同步块的超类。

Class SuperClassA {
     private Bitmap bmpA;

     protected abstract Bitmap createBitmap();

     public void run() {
          synchronized (this) {
               bmpA = createBitmap();
          }
     }

     // some other codes.
}

Class SubClassB extends SuperClassA {
     private Bitmap outBmpB;

     protected Bitmap createBitmap() {
          outBmpB = ..... // create and process "outBmpB".

          Bitmap bmp;
          bmp = ..... // create and process "bmp".
          return bmp;
     }

     public Bitmap getOutBmpB() {
          Bitmap tempBmp;
          synchronized (this) {
               tempBmp = outBmpB.clone();
          }
          return tempBmp;
     }

     // some other codes.
}

Class B 中的“getOutBmpB()”方法由一个线程运行，而 ClassB 中继承的“run()”方法由另一个线程运行。 ClassB 中实现的“createBitmap()”方法应该在“run()”方法内的同步块中运行。

我的问题是我不确定ClassB中新定义的类变量“outBmpB”是否被两个线程安全访问。我不确定“run（）”方法中的“同步（this）”块是否也会“锁定”仅在 ClassB 中定义的“outBmpB”变量？如果没有，那么我可以在“createBitmap()”实现中添加一个“同步（this）”块。例如

 protected Bitmap createBitmap() {
      synchronized (this) {
           outBmpB = ..... // create and process "outBmpB".
      }

      Bitmap bmp;
      bmp = ..... // create and process "bmp".
      return bmp;
 }

感谢您的任何建议。

劳伦斯

Answer 1

您不必必须同步它，因为它已经在超类中同步并且没有其他调用，但您应该这样做。您对createBitmap() 的实现依赖于超类的实现细节。 在您访问共享字段的每一点进行同步。

由于这个事实，您当前的子类的代码很容易出错！

虽然通过this 进行同步是有问题的，但通过其他人无法访问的私有对象进行同步是一个更好的主意。因此，您的代码不能被使用您的类对象的客户端破坏并在其上进行同步。

Answer 2

这不是关于“锁定”一个变量：你只是不能这样做。您的代码中确实有适当的锁定。但是，“线程安全”意味着“任何合理使用都是安全的”。由于您更改 outBmpB 的方法是 protected（即包成员和所有子级都可以使用），因此您最好显式同步。

作为一般规则，每次访问共享可变状态时，都必须确保自己持有锁。在你的情况下，你不确定。同时，Java 锁是可重入的，因此嵌套同步不会受到伤害——或许还能让您免于意外。

Answer 3

一般来说，最好锁定您正在使用的对象（bmpA 和 outBmpB 在您的情况下）而不是容器对象，除非您确实需要这样做（在这种情况下同步方法会更好）。

根据您的具体情况，synchronized (this) 块将相互排斥，所以是的，如果您不在其他任何地方调用 createBitmap()，它会起作用，所以最好是在createBitmap() 内部同步，而不是在run() 内部同步。