ID duplicado, etiqueta nula o ID principal con otro fragmento para com.google.android.gms.maps.MapFragment

Resuelto hermann asked hace 54 años • 23 respuestas

Tengo una aplicación con tres pestañas.

Cada pestaña tiene su propio archivo .xml de diseño. El main.xml tiene su propio fragmento de mapa. Es el que aparece cuando se inicia la aplicación por primera vez.

Todo funciona bien excepto cuando cambio entre pestañas. Si intento volver a la pestaña del fragmento del mapa, aparece este error. Cambiar entre otras pestañas funciona bien.

¿Qué podría estar mal aquí?

Esta es mi clase principal y mi main.xml, así como una clase relevante que uso (también encontrará el registro de errores en la parte inferior).

clase principal

package com.nfc.demo;

import android.app.ActionBar;
import android.app.ActionBar.Tab;
import android.app.Activity;
import android.app.Fragment;
import android.app.FragmentTransaction;
import android.os.Bundle;
import android.widget.Toast;

public class NFCDemoActivity extends Activity {

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

        ActionBar bar = getActionBar();
        bar.setNavigationMode(ActionBar.NAVIGATION_MODE_TABS);
        bar.setDisplayOptions(0, ActionBar.DISPLAY_SHOW_TITLE);

        bar.addTab(bar
                .newTab()
                .setText("Map")
                .setTabListener(
                        new TabListener<MapFragment>(this, "map",
                                MapFragment.class)));
        bar.addTab(bar
                .newTab()
                .setText("Settings")
                .setTabListener(
                        new TabListener<SettingsFragment>(this, "settings",
                                SettingsFragment.class)));
        bar.addTab(bar
                .newTab()
                .setText("About")
                .setTabListener(
                        new TabListener<AboutFragment>(this, "about",
                                AboutFragment.class)));

        if (savedInstanceState != null) {
            bar.setSelectedNavigationItem(savedInstanceState.getInt("tab", 0));
        }
        // setContentView(R.layout.main);

    }

    @Override
    protected void onSaveInstanceState(Bundle outState) {
        super.onSaveInstanceState(outState);
        outState.putInt("tab", getActionBar().getSelectedNavigationIndex());
    }

    public static class TabListener<T extends Fragment> implements
            ActionBar.TabListener {
        private final Activity mActivity;
        private final String mTag;
        private final Class<T> mClass;
        private final Bundle mArgs;
        private Fragment mFragment;

        public TabListener(Activity activity, String tag, Class<T> clz) {
            this(activity, tag, clz, null);
        }

        public TabListener(Activity activity, String tag, Class<T> clz,
                Bundle args) {
            mActivity = activity;
            mTag = tag;
            mClass = clz;
            mArgs = args;

            // Check to see if we already have a fragment for this tab,
            // probably from a previously saved state. If so, deactivate
            // it, because our initial state is that a tab isn't shown.
            mFragment = mActivity.getFragmentManager().findFragmentByTag(mTag);
            if (mFragment != null && !mFragment.isDetached()) {
                FragmentTransaction ft = mActivity.getFragmentManager()
                        .beginTransaction();
                ft.detach(mFragment);
                ft.commit();
            }
        }

        public void onTabSelected(Tab tab, FragmentTransaction ft) {
            if (mFragment == null) {
                mFragment = Fragment.instantiate(mActivity, mClass.getName(),
                        mArgs);
                ft.add(android.R.id.content, mFragment, mTag);
            } else {
                ft.attach(mFragment);
            }
        }

        public void onTabUnselected(Tab tab, FragmentTransaction ft) {
            if (mFragment != null) {
                ft.detach(mFragment);
            }
        }

        public void onTabReselected(Tab tab, FragmentTransaction ft) {
            Toast.makeText(mActivity, "Reselected!", Toast.LENGTH_SHORT)
                         .show();
        }
    }

}

principal.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" >

    <fragment
        xmlns:android="http://schemas.android.com/apk/res/android"
        android:id="@+id/mapFragment"
        android:name="com.google.android.gms.maps.MapFragment"
        android:layout_width="match_parent"
        android:layout_height="match_parent" />
</LinearLayout>

clase relevante ( MapFragment.java )

package com.nfc.demo;

import android.app.Fragment;
import android.os.Bundle;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;

public class MapFragment extends Fragment {

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
            Bundle savedInstanceState) {
        super.onCreateView(inflater, container, savedInstanceState);
        return inflater.inflate(R.layout.main, container, false);
    }

    public void onDestroy() {
        super.onDestroy();
    }
}

error

android.view.InflateException: Binary XML file line #7: 
     Error inflating class fragment
   at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:704)
   at android.view.LayoutInflater.rInflate(LayoutInflater.java:746)
   at android.view.LayoutInflater.inflate(LayoutInflater.java:489)
   at android.view.LayoutInflater.inflate(LayoutInflater.java:396)
   at com.nfc.demo.MapFragment.onCreateView(MapFragment.java:15)
   at android.app.Fragment.performCreateView(Fragment.java:1695)
   at android.app.FragmentManagerImpl.moveToState(FragmentManager.java:885)
   at android.app.FragmentManagerImpl.attachFragment(FragmentManager.java:1255)
   at android.app.BackStackRecord.run(BackStackRecord.java:672)
   at android.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1435)
   at android.app.FragmentManagerImpl$1.run(FragmentManager.java:441)
   at android.os.Handler.handleCallback(Handler.java:725)
   at android.os.Handler.dispatchMessage(Handler.java:92)
   at android.os.Looper.loop(Looper.java:137)
   at android.app.ActivityThread.main(ActivityThread.java:5039)
   at java.lang.reflect.Method.invokeNative(Native Method)
   at java.lang.reflect.Method.invoke(Method.java:511)
   at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:793)
   at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:560)
   at dalvik.system.NativeStart.main(Native Method)

Caused by: java.lang.IllegalArgumentException: 
     Binary XML file line #7: Duplicate id 0x7f040005, tag null, or 
     parent id 0xffffffff with another fragment for 
     com.google.android.gms.maps.MapFragment
   at android.app.Activity.onCreateView(Activity.java:4722)
   at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:680)
   ... 19 more
hermann avatar Jan 01 '70 08:01 hermann
Aceptado

La respuesta que sugiere Matt funciona, pero hace que el mapa se vuelva a crear y dibujar, lo que no siempre es deseable. Después de muchas pruebas y errores, encontré una solución que me funciona:

private static View view;

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
    if (view != null) {
        ViewGroup parent = (ViewGroup) view.getParent();
        if (parent != null)
            parent.removeView(view);
    }
    try {
        view = inflater.inflate(R.layout.map, container, false);
    } catch (InflateException e) {
        /* map is already there, just return view as it is */
    }
    return view;
}

Por si acaso, aquí está "map.xml" (R.layout.map) con R.id.mapFragment (android:id="@+id/mapFragment"):

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:id="@+id/mapLayout"
    android:layout_width="match_parent"
    android:layout_height="match_parent" >

    <fragment xmlns:android="http://schemas.android.com/apk/res/android"
        android:id="@+id/mapFragment"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        class="com.google.android.gms.maps.SupportMapFragment" />
</LinearLayout>

Espero que esto ayude, pero no puedo garantizar que no tenga ningún efecto adverso.

Editar: Hubo algunos efectos adversos, como al salir de la aplicación y volver a iniciarla. Dado que la aplicación no necesariamente se cierra por completo (sino que simplemente se pone en suspensión en segundo plano), el código anterior que envié fallaría al reiniciar la aplicación. Actualicé el código a algo que funciona para mí, tanto para entrar y salir del mapa como para salir y reiniciar la aplicación. No estoy muy contento con el bit try-catch, pero parece funcionar bastante bien. Al mirar el seguimiento de la pila, se me ocurrió que podía verificar si el fragmento del mapa está en FragmentManager, no es necesario el bloque try-catch, código actualizado.

Más ediciones: resulta que, después de todo, necesitas ese try-catch. Después de todo, simplemente buscar el fragmento del mapa resultó no funcionar tan bien. Blergh.

Vidar Wahlberg avatar Feb 04 '2013 20:02 Vidar Wahlberg

El problema es que lo que estás intentando hacer no debería hacerse. No deberías inflar fragmentos dentro de otros fragmentos. De la documentación de Android :

Nota: No puede inflar un diseño en un fragmento cuando ese diseño incluye un <fragmento>. Los fragmentos anidados solo se admiten cuando se agregan dinámicamente a un fragmento.

Si bien es posible que puedas realizar la tarea con los trucos presentados aquí, te recomiendo encarecidamente que no lo hagas. Es imposible estar seguro de que estos trucos manejarán lo que hace cada nuevo sistema operativo Android cuando intentas inflar un diseño para un fragmento que contiene otro fragmento.

La única forma compatible con Android de agregar un fragmento a otro fragmento es mediante una transacción desde el administrador de fragmentos secundarios.

Simplemente cambie su diseño XML a un contenedor vacío (agregue una ID si es necesario):

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

Luego en el onViewCreated(View view, @Nullable Bundle savedInstanceState)método Fragmento:

@Override
public void onViewCreated(View view, @Nullable Bundle savedInstanceState) {
    super.onViewCreated(view, savedInstanceState);
    FragmentManager fm = getChildFragmentManager();
    SupportMapFragment mapFragment = (SupportMapFragment) fm.findFragmentByTag("mapFragment");
    if (mapFragment == null) {
        mapFragment = new SupportMapFragment();
        FragmentTransaction ft = fm.beginTransaction();
        ft.add(R.id.mapFragmentContainer, mapFragment, "mapFragment");
        ft.commit();
        fm.executePendingTransactions();
    }
    mapFragment.getMapAsync(callback);
}
Justin Breitfeller avatar Nov 06 '2013 14:11 Justin Breitfeller

Tuve el mismo problema y pude resolverlo eliminando manualmente el MapFragmentmétodo onDestroy()de la Fragmentclase. Aquí hay un código que funciona y hace referencia al MapFragmentID en el XML:

@Override
public void onDestroyView() {
    super.onDestroyView();
    MapFragment f = (MapFragment) getFragmentManager()
                                         .findFragmentById(R.id.map);
    if (f != null) 
        getFragmentManager().beginTransaction().remove(f).commit();
}

Si no lo elimina MapFragmentmanualmente, quedará suspendido para que no cueste muchos recursos recrear/mostrar la vista del mapa nuevamente. Parece que mantener el subyacente MapViewes excelente para alternar entre pestañas, pero cuando se usa en fragmentos, este comportamiento hace que MapViewse cree un duplicado en cada nueva MapFragmentcon la misma ID. La solución es eliminar manualmente MapFragmenty así recrear el mapa subyacente cada vez que se infla el fragmento.

También noté esto en otra respuesta [ 1 ].

Matt avatar Jan 23 '2013 16:01 Matt

Esta es mi respuesta:

1. Cree un diseño xml como el siguiente:

<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:tools="http://schemas.android.com/tools"
android:id="@+id/map_container"
android:layout_width="match_parent"
android:layout_height="match_parent">
</FrameLayout>

2. En la clase Fragmento, agregue un mapa de Google mediante programación.

import com.google.android.gms.maps.GoogleMap;
import com.google.android.gms.maps.SupportMapFragment;
import android.app.Activity;
import android.os.Bundle;
import android.support.v4.app.Fragment;
import android.support.v4.app.FragmentTransaction;
import android.util.Log;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;

/**
 * A simple {@link android.support.v4.app.Fragment} subclass. Activities that
 * contain this fragment must implement the
 * {@link MapFragment.OnFragmentInteractionListener} interface to handle
 * interaction events. Use the {@link MapFragment#newInstance} factory method to
 * create an instance of this fragment.
 * 
 */
public class MapFragment extends Fragment {
    // TODO: Rename parameter arguments, choose names that match
    private GoogleMap mMap;

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

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
            Bundle savedInstanceState) {
        // Inflate the layout for this fragment
        View view = inflater.inflate(R.layout.fragment_map, container, false);
        SupportMapFragment mMapFragment = SupportMapFragment.newInstance();
        mMap = mMapFragment.getMap();
        FragmentTransaction transaction = getChildFragmentManager().beginTransaction();
        transaction.add(R.id.map_container, mMapFragment).commit();
        return view;
    }

    @Override
    public void onAttach(Activity activity) {
        super.onAttach(activity);
        Log.d("Attach", "on attach");
    }

    @Override
    public void onDetach() {
        super.onDetach();
    }
} 
Zou avatar Apr 09 '2014 10:04 Zou
  1. Como lo mencionó @Justin Breitfeller, la solución @Vidar Wahlberg es un truco que podría no funcionar en futuras versiones de Android.
  2. @Vidar Wahlberg prefiere un truco porque otra solución podría "hacer que el mapa se vuelva a crear y dibujar, lo que no siempre es deseable". Se podría evitar volver a dibujar el mapa manteniendo el fragmento del mapa antiguo, en lugar de crear una nueva instancia cada vez.
  3. La solución @Matt no me funciona (IllegalStateException)
  4. Como lo cita @Justin Breitfeller, "No se puede inflar un diseño en un fragmento cuando ese diseño incluye un archivo . Los fragmentos anidados solo se admiten cuando se agregan a un fragmento dinámicamente".

Mi solución:

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

    // init
    //mapFragment = (SupportMapFragment)getChildFragmentManager().findFragmentById(R.id.map);
    // don't recreate fragment everytime ensure last map location/state are maintain
    if (mapFragment == null) {
        mapFragment = SupportMapFragment.newInstance();
        mapFragment.getMapAsync(this);
    }
    FragmentTransaction transaction = getChildFragmentManager().beginTransaction();
    // R.id.map is a layout
    transaction.replace(R.id.map, mapFragment).commit();

    return view;
}
Desmond Lua avatar Jun 16 '2015 12:06 Desmond Lua