web-dev-qa-db-ja.com

回転時と戻る時のMapViewの状態の復元

背景

新しいGoogle Maps APIでいくつかの問題を抱えていた/抱えている大きなアプリケーションがあります。私はそれを説明しようとしました 別の質問で しかし、それは複雑すぎるように思えたので、可能な限り簡単に新しいプロジェクトを開始し、問題を再現しようと決めました。ここにあります。

状況

Fragmentsを使用しており、MapViewを内部に配置したい。 MapFragmentを使いたくありません。用意したサンプルプロジェクトはあまり美しくないかもしれませんが、できるだけシンプルにしようとしましたが、元のアプリの要素を(再び単純化して)含める必要がありました。 Activityが1つあり、Fragmentを含むカスタムMapViewがプログラムで追加されています。 Mapには、いくつかのポイント/ Markersが含まれます。 Markerをクリックすると、InfoWindowが表示され、クリックすると、コンテンツに次のFragmentが(replace() functionとともに)表示されます。

問題

私には2つの問題があります。

  1. Map付きのMarkersが表示される場合、画面の回転によりClass not found when unmarshallingカスタムMyMapPointクラスのエラー-理由とその意味がわかりません。

  2. MarkerをクリックしてからInfoWindowをクリックします。この後、ハードウェアの戻るボタンを押します。これで、Mapが表示されますが、Markersはなく、0,0ポイント。

コード

主な活動

public class MainActivity extends FragmentActivity {

    private ArrayList<MyMapPoint> mPoints = new ArrayList<MyMapPoint>();

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);     

        if (savedInstanceState == null) {
            mPoints.add(new MyMapPoint(1, new LatLng(20, 10), 
                "test point", "description", null));            
            mPoints.add(new MyMapPoint(2, new LatLng(10, 20), 
                "test point 2", "second description", null));

            Fragment fragment = MyMapFragment.newInstance(mPoints);
            getSupportFragmentManager().beginTransaction()
                .add(R.id.contentPane, fragment).commit();
        }
    }
}

activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<FrameLayout 
    xmlns:Android="http://schemas.Android.com/apk/res/Android"
    Android:id="@+id/contentPane"
    Android:layout_width="fill_parent"
    Android:layout_height="fill_parent" />

map_fragment.xml

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout
    xmlns:Android="http://schemas.Android.com/apk/res/Android"
    Android:layout_width="match_parent"
    Android:layout_height="match_parent"
    Android:orientation="vertical" >

    <com.google.Android.gms.maps.MapView
        xmlns:map="http://schemas.Android.com/apk/res-auto"
        Android:id="@+id/map"
        Android:layout_width="match_parent"
        Android:layout_height="match_parent" />

</LinearLayout>

MyMapFragment

public class MyMapFragment extends Fragment implements
    OnInfoWindowClickListener {

    public static final String KEY_POINTS = "points";

    private MapView mMapView;
    private GoogleMap mMap;
    private HashMap<MyMapPoint, Marker> mPoints = 
        new HashMap<MyMapPoint, Marker>();

    public static MyMapFragment newInstance(ArrayList<MyMapPoint> points) {
        MyMapFragment fragment = new MyMapFragment();
        Bundle args = new Bundle();
        args.putParcelableArrayList(KEY_POINTS, points);
        fragment.setArguments(args);
        return fragment;
    }

    @Override
    public void onSaveInstanceState(Bundle outState) {
        super.onSaveInstanceState(outState);
        mMapView.onSaveInstanceState(outState);
        MyMapPoint[] points = mPoints.keySet().toArray(
            new MyMapPoint[mPoints.size()]);
        outState.putParcelableArray(KEY_POINTS, points);
    }

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        if (savedInstanceState == null) {
            Bundle extras = getArguments();
            if ((extras != null) && extras.containsKey(KEY_POINTS)) {
                for (Parcelable pointP : extras.getParcelableArrayList(KEY_POINTS)) {
                    mPoints.put((MyMapPoint) pointP, null);
                }
            }
        } else {
            MyMapPoint[] points = (MyMapPoint[]) savedInstanceState
                .getParcelableArray(KEY_POINTS);
            for (MyMapPoint point : points) {
                mPoints.put(point, null);
            }
        }
    }

    @Override
    public View onCreateView(LayoutInflater inflater, 
        ViewGroup container, Bundle savedInstanceState) {
        View layout = inflater.inflate(R.layout.map_fragment, container, false);
        mMapView = (MapView) layout.findViewById(R.id.map);
        return layout;
    }

    @Override
    public void onActivityCreated(Bundle savedInstanceState) {
        super.onActivityCreated(savedInstanceState);
        mMapView.onCreate(savedInstanceState);
        setUpMapIfNeeded();
        addMapPoints();
    }

    @Override
    public void onPause() {
        mMapView.onPause();
        super.onPause();
    }

    @Override
    public void onResume() {
        super.onResume();
        setUpMapIfNeeded();
        mMapView.onResume();
    }

    @Override
    public void onDestroy() {
        mMapView.onDestroy();
        super.onDestroy();
    }

    public void onLowMemory() {
        super.onLowMemory();
        mMapView.onLowMemory();
    };

    private void setUpMapIfNeeded() {
        if (mMap == null) {
            mMap = ((MapView) getView().findViewById(R.id.map)).getMap();
            if (mMap != null) {
                setUpMap();
            }
        }
    }

    private void setUpMap() {
        mMap.setOnInfoWindowClickListener(this);
        addMapPoints();
    }

    private void addMapPoints() {
        if (mMap != null) {
            HashMap<MyMapPoint, Marker> toAdd = 
                new HashMap<MyMapPoint, Marker>();
            for (Entry<MyMapPoint, Marker> entry : mPoints.entrySet()) {
                Marker marker = entry.getValue();
                if (marker == null) {
                    MyMapPoint point = entry.getKey();
                    marker = mMap.addMarker(point.getMarkerOptions());
                    toAdd.put(point, marker);
                }
            }
            mPoints.putAll(toAdd);
        }
    }

    @Override
    public void onInfoWindowClick(Marker marker) {
        Fragment fragment = DetailsFragment.newInstance();
        getActivity().getSupportFragmentManager().beginTransaction()
            .replace(R.id.contentPane, fragment)
            .addToBackStack(null).commit();
    }

    public static class MyMapPoint implements Parcelable {
        private static final int CONTENTS_DESCR = 1;

        public int objectId;
        public LatLng latLng;
        public String title;
        public String snippet;

        public MyMapPoint(int oId, LatLng point, 
            String infoTitle, String infoSnippet, String infoImageUrl) {
            objectId = oId;
            latLng = point;
            title = infoTitle;
            snippet = infoSnippet;
        }

        public MyMapPoint(Parcel in) {
            objectId = in.readInt();
            latLng = in.readParcelable(LatLng.class.getClassLoader());
            title = in.readString();
            snippet = in.readString();
        }

        public MarkerOptions getMarkerOptions() {
            return new MarkerOptions().position(latLng)
                .title(title).snippet(snippet);
        }

        @Override
        public int describeContents() {
            return CONTENTS_DESCR;
        }

        @Override
        public void writeToParcel(Parcel dest, int flags) {
            dest.writeInt(objectId);
            dest.writeParcelable(latLng, 0);
            dest.writeString(title);
            dest.writeString(snippet);
        }

        public static final Parcelable.Creator<MyMapPoint> CREATOR = 
            new Parcelable.Creator<MyMapPoint>() {
            public MyMapPoint createFromParcel(Parcel in) {
                return new MyMapPoint(in);
            }

            public MyMapPoint[] newArray(int size) {
                return new MyMapPoint[size];
            }
        };

    }
}

他のファイルを見る必要がある場合はお知らせください。 ここ 完全なプロジェクトを見つけることができます。独自のMaps APIキーをAndroidManifest.xmlファイル。

[〜#〜] edit [〜#〜]

私はこの例をさらにシンプルにし、上記のコードを更新しました。

26
Izydorr

最初の問題に対する私の修正は次のとおりです。

Mapは、mMap.onCreate(savedInstanceState)を呼び出すときにそれ自身の情報だけでなく、すべてのバンドルを開梱しようとしているようで、カスタムParcelableクラスを使用している場合は問題があります。私のために働いた解決策は、使用してすぐにsavedInstanceStateから余分なものを削除することでした-MapのonCreate()を呼び出す前に。 savedInstanceState.remove(MY_KEY)でそれを行います。もう1つやらなければならないことは、自分の情報を_Fragment's_ mMap.onSaveInstanceState()関数のoutStateに追加する前にonSaveInstanceState(Bundle outState)を呼び出すことでした。

次に、2番目の方法を示します。

サンプルプロジェクトを必要最低限​​に単純化しました。生のMarkersをマップに追加していましたが、Fragmentを別のマップに置き換えた場合、「戻る」をクリックした後も「null」マップが表示されました。だから私は2つのことをしました:

  1. onPause()関数でCameraPositionを保存して、onResume()で復元します
  2. onPause()mMapをnullに設定すると、Fragmentが戻ったときに、MarkersaddMapPoints()関数によって再び追加されます(Markers idを保存して確認してから少し変更する必要がありました)。

コードサンプルは次のとおりです。

_private CameraPosition cp;
_

...

_public void onPause() {
    mMapView.onPause();
    super.onPause();

    cp = mMap.getCameraPosition();
    mMap = null;
}
_

...

_public void onResume() {
    super.onResume();
    setUpMapIfNeeded();
    mMapView.onResume();
    if (cp != null) {
        mMap.moveCamera(CameraUpdateFactory.newCameraPosition(cp));
        cp = null;
    }
}
_

onResume()のカメラの位置を更新するには、手動でマップを初期化する必要がありました。私はsetUpMap()でそれをしました:

_private void setUpMap() {
    try {
        MapsInitializer.initialize(getActivity());
    } catch (GooglePlayServicesNotAvailableException e) {
    }
    mMap.setOnInfoWindowClickListener(this);
    mMap.setOnMapLongClickListener(this);
    addMapPoints();
}
_

これらは実際のソリューションではないことを認識しています-単にオーバーライドしますが、今のところできる限り最高であり、プロジェクトを続行する必要があります。誰かがよりクリーンな修正を見つけた場合、それらについて私に知らせてくれて感謝します。

31
Izydorr

onSaveInstanceおよびNullPointerExceptionに関するヒントを求めてここに来ました。 @Izydorrが言及したものを含むさまざまな回避策を試しましたが、運がありませんでした。問題はFragmentPagerAdapter-ViewPagersを埋め込むフラグメントをホストしていたMapViewのアダプターにありました。 FragmentStatePagerAdapterに変更した後、NPEはなくなりました。ふう!

1
kellogs

GoogleMap.OnCameraChangeListenerでフラグメント/アクティビティを実装し、メソッドonCameraChangeをオーバーライドして、新しいカメラの位置を保存し、onResumeを使用します

googleMap.animateCamera(CameraUpdateFactory.newCameraPosition(cameraPositionSaved));
0
PeppinoL