Synchronisierte Methode von Android Activity onCreate Methode

Question

Ist die folgende Situation schlechtes Design oder bin ich etwas wichtiges übersehen?

Hier ist der Code

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(); 
    } 
} 

Warum ist buildGoogleApiClientsynchronized, wenn es überhaupt ist nur von innen onCreate genannt?

Ich weiß, es kann nicht weh tun, aber das ist kaum der Punkt der Frage. Was ich wissen muss ist: Warum ist es notwendig?

bearbeiten

Quelle vollständigen Code: 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

In diesem Fall synchronized wird das Objekt für die Dauer des Anrufs sperren. Wenn auch andere Methoden synchronized deklariert sind, wird jeweils nur einer von ihnen ausgeführt. Wenn beispielsweise von einer synchronisierten Methode auf einen anderen Thread auf mGoogleApiClient zugegriffen wird, stellt die Synchronisierung sicher, dass der andere Thread entweder ein vollständig konstruiertes Objekt oder eine Nullreferenz erkennt.

Sogar damit können Sie immer noch einen Fall machen, dass es eine schlechte Implementierung ist. Ein besserer Ansatz wäre es, jede Methode für ein privates Objekt zu synchronisieren, sodass Code außerhalb der Klasse das Objekt nicht versehentlich sperren und unerwartetes Verhalten verursachen kann. OTOH, es kann argumentiert werden, dass synchronized Methoden für Beispiele netter sind, weil es weniger Codezeilen gibt und es ist einfach, dass die Synchronisierung für die gesamte Methode gilt.

In diesem speziellen Fall könnte volatile funktioniert haben, obwohl ich nicht sicher bin, wie das mit dem Builder-Muster spielt.

In jedem Fall müssen Sie mehr über die ... Abschnitte des Codes wissen, wenn Synchronisation erforderlich ist.