Карта SupportMapFragment исчезает при повороте

Question

Я работаю с приложением на основе карты, которое создает вид альбома, который включает карту и веб-вид. Эта проблема возникает во время поворота и работает нормально с com.android.support:support-v4:19.1, но не работает после обновления до 22.2.0.

Карта отображается так, как и ожидалось на начальном экране, но после поворота карта больше не отображается. Я подтвердил, что GoogleMap, контейнерный вид (который содержит карту) и SupportMapFragment действительны.

Во время отладки я решил проверить представление SupportMapFragment, чтобы увидеть, может ли это быть причиной. После поворота вызов getView() в SupportMapFragment возвращает значение null (я подозреваю, поэтому карта не отображается, но я не понимаю, почему она действительна изначально и равна нулю после поворота).

Я в растерянности, потому что, насколько я могу судить, у меня есть действительный SupportMapFragment, действительный GoogleMap и представление контейнера с соответствующим размером (в данном случае 680x800). Как я уже говорил выше, в предыдущей версии, которую я использовал 19.1, этот код работал как положено.

Ниже приведены соответствующие фрагменты кода:

Создание SupportMapManager находится в классе EventReportAlbumActivity

public class EventReportAlbumActivity extends FragmentActivity
{
   @Nullable protected SupportMapFragment m_mapFragment;

   @NotNull
   public SupportMapFragment getMapFragment ()
   {
     if( m_mapFragment == null )
     {
        GoogleMapOptions options = new GoogleMapOptions();
        options.compassEnabled( false );
        options.rotateGesturesEnabled( false );
        options.scrollGesturesEnabled( false );
        options.tiltGesturesEnabled( false );
        options.zoomControlsEnabled( false );
        options.zoomGesturesEnabled( false );

        m_mapFragment = SupportMapFragment.newInstance( options );
        m_mapFragment.setRetainInstance( true );
     }
     return m_mapFragment;
  }
}

Фактический экземпляр Fragment, который отображает карту и веб-представление

public class EventAlbumItemFragment extends Fragment
{
   @Nullable
   protected SupportMapFragment m_mapFragment;

  public static EventAlbumItemFragment newInstance ( )
  {
     final EventAlbumItemFragment fragment = new EventAlbumItemFragment();
     fragment.setRetainInstance( true );

     return fragment;
  }

    @Override
    public View onCreateView ( @NotNull LayoutInflater inflater,
                               @NotNull ViewGroup container,
                               @Nullable Bundle savedInstanceState )
    {
       final View view = inflater.inflate( R.layout.event_album_item, container, false );

       View mapContainer = view.findViewById( R.id.mapview_container );

       // make map fragment view invisible until we configure it so the coordinates will be correct
       mapContainer.setVisibility( View.INVISIBLE );

       // other code removed for brevity...
   }

  /**
   * Setter for the map fragment and its UI
   *
   * @param mapFragment The map fragment which should be added to this fragment's view, or null to remove the current fragment
   */
  public void setMapFragment ( @Nullable SupportMapFragment mapFragment )
  {
     final FragmentManager fragmentManager = getChildFragmentManager();

     if( m_mapFragment != null && m_mapFragment.isAdded() )
     {
        // Remove the existing map manager
        fragmentManager.beginTransaction()
           .remove( m_mapFragment )
           .commit();
     }

     if( mapFragment != null )
     {
        // Add the map view to the current view
        fragmentManager.beginTransaction()
           .add( R.id.mapview_container, mapFragment, MAP_FRAGMENT_TAG )
           .commit();
     }

     fragmentManager.executePendingTransactions();

     m_mapFragment = mapFragment;
     configureMap();
  }

  protected void configureMap ()
  {
     if ( m_mapFragment != null && m_eventMapFeature != null && m_eventMapFeature.haveValidLocation() == true )
     {
        final EventAlbumItemFragment localThis = this ;

        m_mapFragment.getMapAsync( new OnMapReadyCallback()
        {
           @Override
           public void onMapReady( GoogleMap googleMap )
           {
              final View mapContainer = getView().findViewById( R.id.mapview_container );
              final int mapWidth = mapContainer.getWidth();
              final int mapHeight = mapContainer.getHeight();

              // we have our map and the coordinates should be correct. Make map visible and proceed
              mapContainer.setVisibility( View.VISIBLE );

              // This line below crashes because view is NULL after rotation. 
              // Left here to illustrate what I believe to be the root cause of the issue
              // m_mapFragment.getView().setVisibility( View.VISIBLE );

              // Remove all overlays from the map
              googleMap.clear();

              final DisplayMetrics displayMetrics = getResources().getDisplayMetrics();

              final float zoom = (float)GeoUtils.getZoomForMetersWide( MAP_MINIMUM_METER_SPAN,
                      mapWidth / displayMetrics.scaledDensity,
                      m_eventMapFeature.getCoordinate().latitude );

              // Change the map bounds
              CameraUpdate cameraUpdate;

              // draw some items on the map
              final LatLngBounds bounds = m_eventMapFeature.getBounds();

              cameraUpdate = CameraUpdateFactory.newLatLngZoom( bounds.southwest, zoom );

              googleMap.moveCamera( cameraUpdate );
           }
        } );
     }
  }

  /** Assign the map fragment variable on resume. This is necessary for orientation change events */
  @Override
  public void onResume ()
  {
     super.onResume();

     // If the map fragment hasn't been set, but this is the current item, get the map fragment from the parent
    final EventReportAlbumActivity activity = (EventReportAlbumActivity)getActivity();

      if( m_mapFragment == null && this == activity.getCurrentItemFragment() )
      {
         setMapFragment( activity.getMapFragment() );
      }

     //
     configureMap();
  }

}

ОБНОВЛЕНИЕ: setMapFragment выше также падает, когда есть несколько страниц и страницы пользователя в альбоме после поворота

После еще нескольких испытаний я также протестировал этот код, когда в альбоме несколько страниц. Без вращения, я могу просматривать альбом без проблем. Однако, если я поворачиваю устройство и пытаюсь пролистать альбом, приложение вылетает. Снова, используя версию 19 библиотеки поддержки, этот код работает как ожидалось. Это начало происходить только после обновления до версии 22.

в методе setMapFragment() в EventAlbumItemFragment:

fragmentManager.executePendingTransactions();

Я получаю следующую трассировку стека:

java.lang.IllegalStateException: Could not execute method of the activity
    at android.view.View$1.onClick(View.java:4020)
    at android.view.View.performClick(View.java:4780)
    at android.view.View$PerformClick.run(View.java:19866)
    at android.os.Handler.handleCallback(Handler.java:739)
    at android.os.Handler.dispatchMessage(Handler.java:95)
    at android.os.Looper.loop(Looper.java:135)
    at android.app.ActivityThread.main(ActivityThread.java:5254)
    at java.lang.reflect.Method.invoke(Native Method)
    at java.lang.reflect.Method.invoke(Method.java:372)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:903)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:698)
Caused by: java.lang.reflect.InvocationTargetException
    at java.lang.reflect.Method.invoke(Native Method)
    at java.lang.reflect.Method.invoke(Method.java:372)
    at android.view.View$1.onClick(View.java:4015)
    at android.view.View.performClick(View.java:4780)
    at android.view.View$PerformClick.run(View.java:19866)
    at android.os.Handler.handleCallback(Handler.java:739)
    at android.os.Handler.dispatchMessage(Handler.java:95)
    at android.os.Looper.loop(Looper.java:135)
    at android.app.ActivityThread.main(ActivityThread.java:5254)
    at java.lang.reflect.Method.invoke(Native Method)
    at java.lang.reflect.Method.invoke(Method.java:372)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:903)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:698)
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.Object java.util.ArrayList.set(int, java.lang.Object)' on a null object reference
    at android.support.v4.app.FragmentManagerImpl.makeInactive(FragmentManager.java:1192)
    at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:1099)
    at android.support.v4.app.FragmentManagerImpl.removeFragment(FragmentManager.java:1235)
    at android.support.v4.app.BackStackRecord.run(BackStackRecord.java:710)
    at android.support.v4.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1501)
    at android.support.v4.app.FragmentManagerImpl.executePendingTransactions(FragmentManager.java:490)
    at crc.carsapp.fragments.MapAlbumItemFragment.setMapFragment(MapAlbumItemFragment.java:78)
    at crc.carsapp.listeners.OnMapAlbumScrollListener.onPageSelected(OnMapAlbumScrollListener.java:46)
    at crc.carsapp.listeners.OnEventAlbumViewScrollListener.onPageSelected(OnEventAlbumViewScrollListener.java:34)
    at android.support.v4.view.ViewPager.dispatchOnPageSelected(ViewPager.java:1786)
    at android.support.v4.view.ViewPager.scrollToItem(ViewPager.java:568)
    at android.support.v4.view.ViewPager.setCurrentItemInternal(ViewPager.java:552)
    at android.support.v4.view.ViewPager.setCurrentItemInternal(ViewPager.java:513)
    at android.support.v4.view.ViewPager.setCurrentItem(ViewPager.java:505)
    at crc.carsapp.activities.AlbumActivity.scrollToPrevious(AlbumActivity.java:133)
    at java.lang.reflect.Method.invoke(Native Method)
    at java.lang.reflect.Method.invoke(Method.java:372)
    at android.view.View$1.onClick(View.java:4015)
    at android.view.View.performClick(View.java:4780)
    at android.view.View$PerformClick.run(View.java:19866)
    at android.os.Handler.handleCallback(Handler.java:739)
    at android.os.Handler.dispatchMessage(Handler.java:95)
    at android.os.Looper.loop(Looper.java:135)
    at android.app.ActivityThread.main(ActivityThread.java:5254)
    at java.lang.reflect.Method.invoke(Native Method)
    at java.lang.reflect.Method.invoke(Method.java:372)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:903)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:698)

Answer 1

Итак, было 2 проблемы с приведенным выше кодом. Во-первых (и самое важное), Activity (EventReportAlbumActivity) отвечает за создание фрагмента карты (и связанные с ним фрагменты сохраняют ссылку на него). Во время ротации активность (конечно) разрушается и воссоздается. Таким образом, последующие вызовы getMapFragment будут генерировать новый SupportMapFragment. Между тем, фрагменты, отображающие карту, все еще имеют ссылку на предыдущий фрагмент карты.

Всякий раз, когда в setMapFragment делался вызов на удаление, он ссылался на старый SupportMapFragment и, таким образом, создавал исключение.

Это также приводило к тому, что карта не отображалась должным образом после поворота.

Исправление состоит в том, чтобы убрать вызов для удаления в setMapFragment (ViewPager обрабатывает удаление с предыдущей страницы в onPageSelected) и добавить "новый" SupportMapFragment в setMapFragment по мере необходимости.

То, как оригинальный код работал до обновления до версии 22, остается загадкой, так как это кажется ошибкой независимо от версии поддержки. И решение по-прежнему не совсем корректно, так как для SupportMapFragment установлено значение setRetainInstance, равное true, что подразумевает, что мы хотим, чтобы он жил в течение всего жизненного цикла фрагмента, но действие завершается перезаписью его, когда он воссоздается во время вращения. Так что, если у кого-то есть более элегантное решение, я бы с удовольствием его услышал.

Соответствующий обновленный код ниже:

public class EventAlbumItemFragment extends Fragment
{
   public void setMapFragment ( @Nullable SupportMapFragment mapFragment )
  {
     final FragmentManager fragmentManager = getChildFragmentManager();

     // The working assumption is that our map fragment, if already added, is added to
     // our current fragment.
     if ( mapFragment != null && mapFragment.isAdded() == false )
     {
        // Add the map view to the current view
        fragmentManager.beginTransaction()
           .add( R.id.mapview_container, mapFragment, MAP_FRAGMENT_TAG )
           .commit() ;
        fragmentManager.executePendingTransactions();
     }

     m_mapFragment = mapFragment;
     configureMap();
  }

  @Override
  public void onResume ()
  {
     super.onResume();

     // We can get here in (at least) 3 different ways:
     // 1. The first time this fragment is created.
     //       In this case, we will not have a local reference to a map fragment and will will add
     //       it if we are the currently displaying fragment.
     // 2. After rotation
     //       Because the actual map fragment is maintained by our activity, the map fragment will
     //       have been recreated and our reference to it will no longer be valid.
     // 3. After a pause event (such as the device going into power saving mode).
     //       We should still have a reference to the current map fragment and it should be
     //       valid. We can test that by checking if the map fragment is attached. This is done
     //       in the setMapFragment() method.
     //
     final AlbumActivity activity = (AlbumActivity)getActivity();

     Fragment currentFragment = activity.getCurrentItemFragment() ;

     if ( m_mapFragment == null && this == currentFragment )
     {
        setMapFragment( activity.getMapFragment() );
     }
     else if ( m_mapFragment != null )
     {
        setMapFragment( activity.getMapFragment() );
     }

     configureMap();
   }
}