Восстановление состояния MapView при повороте и обратно

Фон

У меня есть более крупное приложение, в котором у меня было несколько проблем с новым API Карт Google. Я попытался описать это в другом вопросе, но поскольку это кажется слишком сложным, я решил начать новый проект, как можно более простой, и попытаться воспроизвести проблемы. Итак, вот оно.

Ситуация

Я использую Fragments и хочу поместить внутрь MapView. Я не хочу использовать MapFragment. Образец проекта, который я подготовил, может быть не очень красивым, но я постарался сделать его максимально простым, и он должен был содержать некоторые элементы (опять же упрощенные) из исходного приложения. У меня есть один Activity и мой пользовательский Fragment с MapView в нем, добавленный программно. Map содержит несколько точек/Markers. После нажатия на Marker отображается InfoWindow, а нажатие на него приводит к отображению следующего Fragment (с функцией replace()) в содержании.

Проблемы

У меня есть две проблемы:

  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];
            }
        };

    }
}

Если вам нужно взглянуть на любой другой файл - дайте мне знать. Здесь вы можете найти полный проект, вам просто нужно указать свой собственный КЛЮЧ API Карт в AndroidManifest.xml файл.

ИЗМЕНИТЬ

Мне удалось сделать пример еще проще и обновить код выше.


person Izydorr    schedule 26.04.2013    source источник


Ответы (3)


Вот мое решение первой проблемы:

Кажется, что Map пытается распаковать весь пакет, а не только собственную информацию, когда я вызываю mMap.onCreate(savedInstanceState), и у него проблемы с этим, если я использую свой собственный класс Parcelable. Решение, которое сработало для меня, заключалось в удалении моих дополнений из savedInstanceState, как только я их использовал - до того, как я вызову Map's onCreate(). Я делаю это с savedInstanceState.remove(MY_KEY). Еще одна вещь, которую мне нужно было сделать, это вызвать mMap.onSaveInstanceState() перед добавлением моей собственной информации в outState в функции Fragment's onSaveInstanceState(Bundle outState).

И вот как я справился со вторым:

Я упростил пример проекта до голых костей. Я добавлял необработанный Markers на карту, и если я заменил Fragment на карту другой, то после нажатия «назад» я все равно получал «нулевую» карту. Поэтому я сделал две вещи:

  1. сохранение CameraPosition в функции onPause() для восстановления в onResume()
  2. установка mMap на null в onPause(), поэтому, когда Fragment возвращается, Markers снова добавляется функцией addMapPoints() (мне пришлось немного изменить ее, так как я сохранял и проверял идентификаторы Markers).

Вот примеры кода:

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

Я понимаю, что это ненастоящие решения - просто переопределения, но пока это лучшее, что я могу сделать, и проект должен продолжаться. Если кто-то найдет более чистые исправления, я буду благодарен за то, что сообщил мне о них.

person Izydorr    schedule 02.05.2013
comment
Приветствую ваш ответ о том, что Карта пытается распаковать весь пакет, я реализовал ваше предложение много лет назад, но недавно добавил ViewPager к моему фрагменту, что снова вызвало BadParcelableException. Оказывается, сохраненное состояние ViewPager также нельзя распарсить, так как оно взято из библиотеки поддержки, поэтому я решил скопировать состояние карты в новый пакет вместо того, чтобы удалять все участки, о которых я знал, мой ответ здесь: stackoverflow.com/a/24651246/1007151 - person darnmason; 09.07.2014

Пришел сюда в поисках подсказок относительно onSaveInstance и NullPointerException. Я пробовал различные обходные пути, в том числе упомянутый @Izydorr, но безуспешно. Проблема была с FragmentPagerAdapter — адаптером ViewPager, в котором размещались мои фрагменты, в которые встраивались MapView. После того, как он был изменен на FragmentStatePagerAdapter, NPE исчезли. Вау!

person kellogs    schedule 18.01.2016

Реализует ваш фрагмент/активность с помощью GoogleMap.OnCameraChangeListener и метода переопределения onCameraChange, и вы можете сохранить новое положение камеры и onResume использовать

googleMap.animateCamera(CameraUpdateFactory.newCameraPosition(cameraPositionSaved));
person PeppinoL    schedule 24.11.2015
comment
Этот интерфейс теперь устарел. См. здесь: stackoverflow.com/questions/38727517/ - person Ruben2112; 21.08.2017