AndroidアクティビティonCreateメソッドによって呼び出された同期メソッド

Question

次の状況が悪い設計ですか、何か重要なことを見落としていますか？それは今までにのみonCreate内部から呼ばれていたときにここで

コードが

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​​のですか？

私はそれが傷つけることはできないことを知っていますが、それはほとんど問題のポイントではありません。私が知る必要があるのは：なぜそれが必要ですか？

編集

完全なコードのソース：この場合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

、​​は、呼の継続時間のオブジェクトをロックします。​​と宣言された他のメソッドがある場合は、それらのうちの1つだけが任意の時点で実行されます。たとえば、別のスレッドの同期メソッドからmGoogleApiClientにアクセスすると、同期により、他のスレッドが完全に構築されたオブジェクトまたはnull参照を参照するようになります。

それでも、実装が不適切である場合があります。より良いアプローチは、各メソッドをプライベートオブジェクト上で同期させることです。そのため、クラス外のコードは誤ってオブジェクトをロックして予期しない動作を引き起こすことはありません。 OTOHでは、コードの行数が少なく、同期がメソッド全体に適用されることが明白なので、​​のメソッドは例の方が優れていると主張できます。

この特定のケースでは、volatileが機能している可能性がありますが、どのようにビルダーのパターンに対応しているか分かりません。

いずれにしても、同期が必要かどうかを知るには、コードの...セクションの詳細を知る必要があります。