需要同步的对象上的同步

Question

我希望仅通过类Engine返回类的一个实例Format1：

 public final class Engine {

     private static Format format;

     public Engine(final Param1 param1, final Param2 param2) {
         Engine.format = new FormatV1(param1);   // FormatV1 implements Format
     }

     static Format getFormat(int condition) {
         switch(condition) {
            case 1 : return format;
         }
     }
 }

这样是否可以确保通过getFormat只返回一个正确的Format实例？

另外，调用getFormat（）的客户端需要在格式对象（绝对没有状态）上执行方法，但是将ByteBuffer传递给该方法（此ByteBuffer可能会被修改）：

 class Client 
 {
     public static void main(String[] args) {

          Format obj = Engine.getFormat(1); // some param
          ByteBuffer buffer;                // read data from disk

          synchronized (obj) {

                 long[] result = obj.method(buffer);

                 if (result == null) {
                      // do something
                 }
          }
     }
 }

这里/这样的同步结构是否确保可串行性？

PS：我对Java还是很陌生，就我所知，我也可以使方法同步，但我认为关键部分也应包含检查（结果== null），因此我决定使用同步（obj）。 ）构造。 如果这是一个愚蠢的问题，请原谅我，但我想确认我的怀疑。

Answer 1

当前，您正在Engine构造函数中创建Format实例，这很可能不是您想要的（您的当前示例将引发NullPointerException。

其次，您有2个同步问题。 首先是格式实例的正确发布。 您目前不这样做。 您应该使用类初始化（在类创建时实例化格式实例）或使格式成员易失，以便正确发布格式实例。

其次，如果您的格式实例具有可变状态，则应使FormatV1方法本身同步。 在使用对象之前使调用者在对象上进行同步是一个非常糟糕的设计。

Answer 2

您创建了Engine类的构造函数来初始化静态变量，而是使用对该方法进行同步来创建静态初始化器。

public static synchronized void initFormat(....) {
   if (format == null) {
      ... create it
   } else {
   .... your chocie: throw an exception or do nothing 
   }
}