2

ご存知のように、Android 開発ガイド に「Location Aware」というダウンロード可能な優れたサンプルがあります。

このコードに書かれている Handler について質問があります。同じクラス内で Async Task と一緒に Handler を使用するのは正しいですか? このコードは Android 標準に従って正しく記述されていますか? (ところで、エラーが修正されれば、このプロジェクトは正常に動作します。)

非同期タスクのリファレンス: http://developer.android.com/reference/android/os/AsyncTask.html

public class LocationActivity extends FragmentActivity {
    .....
    private Handler mHandler;
    private boolean mGeocoderAvailable;
    ...

    @SuppressLint("NewApi")
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);

        if (savedInstanceState != null) {
            mUseFine = savedInstanceState.getBoolean(KEY_FINE);
            mUseBoth = savedInstanceState.getBoolean(KEY_BOTH);
        } else {
            mUseFine = false;
            mUseBoth = false;
        }
        mLatLng = (TextView) findViewById(R.id.latlng);
        mAddress = (TextView) findViewById(R.id.address);

        ..............

        // Handler for updating text fields on the UI like the lat/long and address.
        mHandler = new Handler() {
            public void handleMessage(Message msg) {
                switch (msg.what) {
                    case UPDATE_ADDRESS:
                        mAddress.setText((String) msg.obj);
                        break;
                    case UPDATE_LATLNG:
                        mLatLng.setText((String) msg.obj);
                        break;
                }
            }
        };
        // Get a reference to the LocationManager object.
        mLocationManager = (LocationManager) getSystemService(Context.LOCATION_SERVICE);
    }

    ...............
    @Override
    protected void onStart() {
        super.onStart();

        LocationManager locationManager =
                (LocationManager) getSystemService(Context.LOCATION_SERVICE);
        final boolean gpsEnabled = locationManager.isProviderEnabled(LocationManager.GPS_PROVIDER);

        ........

    private void setup() {
        Location gpsLocation = null;
        Location networkLocation = null;
        mLocationManager.removeUpdates(listener);
        mLatLng.setText(R.string.unknown);
        mAddress.setText(R.string.unknown);

        if (mUseFine) {
            mFineProviderButton.setBackgroundResource(R.drawable.button_active);
            mBothProviderButton.setBackgroundResource(R.drawable.button_inactive);

            gpsLocation = requestUpdatesFromProvider(
                    LocationManager.GPS_PROVIDER, R.string.not_support_gps);

            if (gpsLocation != null) updateUILocation(gpsLocation);
        } else if (mUseBoth) {

            mFineProviderButton.setBackgroundResource(R.drawable.button_inactive);
            mBothProviderButton.setBackgroundResource(R.drawable.button_active);

            gpsLocation = requestUpdatesFromProvider(
                    LocationManager.GPS_PROVIDER, R.string.not_support_gps);
            networkLocation = requestUpdatesFromProvider(
                    LocationManager.NETWORK_PROVIDER, R.string.not_support_network);

            ........

    private void doReverseGeocoding(Location location) {
        // Since the geocoding API is synchronous and may take a while.  You don't want to lock
        // up the UI thread.  Invoking reverse geocoding in an AsyncTask.
        (new ReverseGeocodingTask(this)).execute(new Location[] {location});
    }

    private void updateUILocation(Location location) {
        // We're sending the update to a handler which then updates the UI with the new
        // location.
        Message.obtain(mHandler,
                UPDATE_LATLNG,
                location.getLatitude() + ", " + location.getLongitude()).sendToTarget();

        // Bypass reverse-geocoding only if the Geocoder service is available on the device.
        if (mGeocoderAvailable) doReverseGeocoding(location);
    }

    private final LocationListener listener = new LocationListener() {

        @Override
        public void onLocationChanged(Location location) {
            // A new location update is received.  Do something useful with it.  Update the UI with
            // the location update.
            updateUILocation(location);
        }

        .........


    // AsyncTask encapsulating the reverse-geocoding API.  Since the geocoder API is blocked,
    // we do not want to invoke it from the UI thread.
    private class ReverseGeocodingTask extends AsyncTask<Location, Void, Void> {
        Context mContext;

        public ReverseGeocodingTask(Context context) {
            super();
            mContext = context;
        }

        @Override
        protected Void doInBackground(Location... params) {
            Geocoder geocoder = new Geocoder(mContext, Locale.getDefault());

            Location loc = params[0];
            List<Address> addresses = null;
            try {
                addresses = geocoder.getFromLocation(loc.getLatitude(), loc.getLongitude(), 1);
            } catch (IOException e) {
                e.printStackTrace();
                // Update address field with the exception.
                Message.obtain(mHandler, UPDATE_ADDRESS, e.toString()).sendToTarget();
            }
            if (addresses != null && addresses.size() > 0) {
                Address address = addresses.get(0);
                // Format the first line of address (if available), city, and country name.
                String addressText = String.format("%s, %s, %s",
                        address.getMaxAddressLineIndex() > 0 ? address.getAddressLine(0) : "",
                        address.getLocality(),
                        address.getCountryName());
                // Update address field on UI.
                Message.obtain(mHandler, UPDATE_ADDRESS, addressText).sendToTarget();
            }
            return null;
        }
    }

    ...............
4

2 に答える 2

2

はい。使用できます。それは間違いではありません。必要に応じて、それらを 2 つのクラスに分けることもできます。

于 2013-02-23T06:55:52.890 に答える
1

いいえ、外部オブジェクト インスタンスへの参照を取得する Java の匿名内部クラスを使用します。(この場合はハンドラー)

内部クラスを静的にし、PARENT_CLASS をコンストラクター パラメーターとして渡し、WeakReference を使用して参照を格納します。

これにより、ガベージ コレクター内の依存関係の循環が防止されます。

https://docs.oracle.com/javase/7/docs/api/java/lang/ref/WeakReference.html

WeakReference を使用すると、オブジェクトを簡単に「指す」ことができますが、そのオブジェクトに他の (強い) 参照がない場合は、ガベージ コレクションが行われます。(これは望ましい結果です) (RAM が限られている Android では特に重要です)

編集: これは非常に微妙なメモリ リークであり、特に探していない限り見つけるのは困難です。その理由は、Java がすべての非静的内部クラスへの暗黙的な参照を追加するためです (ラムダ表記について 100% 確信があるわけではありませんが、そうではないと思います... しかし、それを調べる必要があります...)。内部クラスのインスタンスを作成したインスタンス オブジェクトに...これにより、GC の参照の追跡にサイクルが発生し、両方のオブジェクトが不要なときにメモリに保持されるというメモリ リークが発生します。(これは、独自のスレッドを処理する場合により重要になることに注意してください...しかし、それでも始めることをお勧めします)

于 2016-05-22T13:02:21.767 に答える