是否可以使用同步安全地访问该变量？

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.
}

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

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

 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 的方法是受保护的（即可供包成员和所有子项使用），因此最好显式同步。

作为一般规则，每次访问共享可变状态时，都必须确保持有锁。就你而言，你不确定。同时，Java 锁是可重入的，因此嵌套同步不会造成任何损害，而且可能会让您避免意外。

Answer 3

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

根据您的具体情况，synchronized (this) 块将相互排除，所以是的，如果您不在任何地方调用 createBitmap()，它就会起作用。 else，因此在 createBitmap() 内部同步比在 run() 内部同步要好得多。