java - Android Activity onCreate 方法调用的同步方法

Question

以下情况是设计不当还是我忽略了一些重要的事情？

这是代码

public class MainActivity extends AppCompatActivity … {

  …

  @Override
  public void onCreate(Bundle savedInstanceState){
     …
     buildGoogleApiClient();
  }

  /**
   * Builds a GoogleApiClient. Uses the addApi() method to request the LocationServices API.
   */
    protected synchronized void buildGoogleApiClient() {
        mGoogleApiClient = new GoogleApiClient.Builder(this)
                .addConnectionCallbacks(this)
                .addOnConnectionFailedListener(this)
                .addApi(LocationServices.API)
                .build();
    }
}

为什么 buildGoogleApiClient 同步 只是从 onCreate 内部调用？

我知道这不会造成伤害，但这不是问题的重点。我需要知道的是:为什么有必要？

编辑

完整代码来源:https://github.com/googlesamples/android-play-location/blob/master/BasicLocationSample/app/src/main/java/com/google/android/gms/location/sample/basiclocationsample/MainActivity.java

Answer 1

在这种情况下，synchronized 将在调用期间锁定对象。如果还有其他方法也声明为 synchronized，那么在任何给定时间都只会执行其中一个方法。例如，如果从另一个线程上的同步方法访问 mGoogleApiClient，同步将确保另一个线程看到一个完全构造的对象，或者一个空引用。

即便如此，您仍然可以证明这是一个糟糕的实现。更好的方法是让每个方法在私有(private)对象上同步，这样类外部的代码就不会无意中锁定对象并导致意外行为。 OTOH，可以说 synchronized 方法对于示例来说更好，因为代码行更少，而且很明显同步适用于整个方法。

在这种特殊情况下，volatile 可能会起作用，但我不确定它如何与构建器模式一起发挥作用。

无论如何，您需要更多地了解代码的 ... 部分，以了解是否需要同步。