Brauchen Fragmente wirklich einen leeren Konstruktor?

Ich habe einen Fragmentmit einem Konstruktor, der mehrere Argumente akzeptiert. Meine App hat während der Entwicklung gut funktioniert, aber in der Produktion sehen meine Benutzer manchmal diesen Absturz:

android.support.v4.app.Fragment$InstantiationException: Unable to instantiate fragment 
make sure class name exists, is public, and has an empty constructor that is public

Ich könnte einen leeren Konstruktor erstellen, wie diese Fehlermeldung andeutet, aber das macht für mich keinen Sinn, da ich seitdem eine separate Methode aufrufen müsste, um die Einrichtung des zu beenden Fragment.

Ich bin gespannt, warum dieser Absturz nur gelegentlich vorkommt. Vielleicht benutze ich das ViewPagerfalsch? Ich instanziiere alle Fragments selbst und speichere sie in einer Liste innerhalb der Activity. Ich verwende keine FragmentManagerTransaktionen, da die ViewPagerBeispiele, die ich gesehen habe, dies nicht erforderten und alles während der Entwicklung zu funktionieren schien.

Ja, das tun sie.

Sie sollten den Konstruktor sowieso nicht wirklich überschreiben. Sie sollten eine newInstance()statische Methode definiert haben und alle Parameter über Argumente (Bundle) übergeben.

Beispielsweise:

public static final MyFragment newInstance(int title, String message) {
    MyFragment f = new MyFragment();
    Bundle bdl = new Bundle(2);
    bdl.putInt(EXTRA_TITLE, title);
    bdl.putString(EXTRA_MESSAGE, message);
    f.setArguments(bdl);
    return f;
}

Und natürlich die Argumente so packen:

@Override
public void onCreate(Bundle savedInstanceState) {
    title = getArguments().getInt(EXTRA_TITLE);
    message = getArguments().getString(EXTRA_MESSAGE);

    //...
    //etc
    //...
}

Dann würden Sie von Ihrem Fragmentmanager aus wie folgt instanziieren:

@Override
public void onCreate(Bundle savedInstanceState) {
    if (savedInstanceState == null){
        getSupportFragmentManager()
            .beginTransaction()
            .replace(R.id.content, MyFragment.newInstance(
                R.string.alert_title,
                "Oh no, an error occurred!")
            )
            .commit();
    }
}

Auf diese Weise kann der Objektstatus über die Argumente gespeichert werden, wenn er getrennt und erneut angehängt wird. Ähnlich wie Bündel, die an Absichten befestigt sind.

Grund - Zusätzliche Lektüre

Ich dachte, ich würde erklären, warum für Leute, die sich fragen, warum.

Wenn Sie Folgendes überprüfen: https://android.googlesource.com/platform/frameworks/base/+/master/core/java/android/app/Fragment.java

Sie werden sehen, dass die instantiate(..)Methode in der FragmentKlasse die newInstanceMethode aufruft :

public static Fragment instantiate(Context context, String fname, @Nullable Bundle args) {
    try {
        Class<?> clazz = sClassMap.get(fname);
        if (clazz == null) {
            // Class not found in the cache, see if it's real, and try to add it
            clazz = context.getClassLoader().loadClass(fname);
            if (!Fragment.class.isAssignableFrom(clazz)) {
                throw new InstantiationException("Trying to instantiate a class " + fname
                        + " that is not a Fragment", new ClassCastException());
            }
            sClassMap.put(fname, clazz);
        }
        Fragment f = (Fragment) clazz.getConstructor().newInstance();
        if (args != null) {
            args.setClassLoader(f.getClass().getClassLoader());
            f.setArguments(args);
        }
        return f;
    } catch (ClassNotFoundException e) {
        throw new InstantiationException("Unable to instantiate fragment " + fname
                + ": make sure class name exists, is public, and has an"
                + " empty constructor that is public", e);
    } catch (java.lang.InstantiationException e) {
        throw new InstantiationException("Unable to instantiate fragment " + fname
                + ": make sure class name exists, is public, and has an"
                + " empty constructor that is public", e);
    } catch (IllegalAccessException e) {
        throw new InstantiationException("Unable to instantiate fragment " + fname
                + ": make sure class name exists, is public, and has an"
                + " empty constructor that is public", e);
    } catch (NoSuchMethodException e) {
        throw new InstantiationException("Unable to instantiate fragment " + fname
                + ": could not find Fragment constructor", e);
    } catch (InvocationTargetException e) {
        throw new InstantiationException("Unable to instantiate fragment " + fname
                + ": calling Fragment constructor caused an exception", e);
    }
}

http://docs.oracle.com/javase/6/docs/api/java/lang/Class.html#newInstance () Erklärt, warum bei der Instanziierung überprüft wird, ob der Accessor vorhanden ist publicund ob dieser Klassenladeprogramm den Zugriff darauf zulässt.

Alles in allem ist es eine ziemlich böse Methode, aber sie erlaubt FragmentMangeres, Fragmentsmit Zuständen zu töten und neu zu erschaffen . (Das Android-Subsystem macht ähnliche Dinge mit Activities).

Beispielklasse

Ich werde oft gefragt, ob ich anrufen soll newInstance. Verwechseln Sie dies nicht mit der Klassenmethode. Dieses ganze Klassenbeispiel sollte die Verwendung zeigen.

/**
 * Created by chris on 21/11/2013
 */
public class StationInfoAccessibilityFragment extends BaseFragment implements JourneyProviderListener {

    public static final StationInfoAccessibilityFragment newInstance(String crsCode) {
        StationInfoAccessibilityFragment fragment = new StationInfoAccessibilityFragment();

        final Bundle args = new Bundle(1);
        args.putString(EXTRA_CRS_CODE, crsCode);
        fragment.setArguments(args);

        return fragment;
    }

    // Views
    LinearLayout mLinearLayout;

    /**
     * Layout Inflater
     */
    private LayoutInflater mInflater;
    /**
     * Station Crs Code
     */
    private String mCrsCode;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        mCrsCode = getArguments().getString(EXTRA_CRS_CODE);
    }

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

    @Override
    public void onViewCreated(View view, Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);
        mLinearLayout = (LinearLayout)view.findViewBy(R.id.station_info_accessibility_linear);
        //Do stuff
    }

    @Override
    public void onResume() {
        super.onResume();
        getActivity().getSupportActionBar().setTitle(R.string.station_info_access_mobility_title);
    }

    // Other methods etc...
}

Wie von CommonsWare in dieser Frage https://stackoverflow.com/a/16064418/1319061 festgestellt , kann dieser Fehler auch auftreten, wenn Sie eine anonyme Unterklasse eines Fragments erstellen, da anonyme Klassen keine Konstruktoren haben können.

Erstelle keine anonymen Unterklassen von Fragment :-)

Ja, wie Sie sehen, instanziiert das Support-Paket auch die Fragmente (wenn sie zerstört und wieder geöffnet werden). Ihre FragmentUnterklassen benötigen einen öffentlichen leeren Konstruktor, da dies vom Framework aufgerufen wird.

Hier ist meine einfache Lösung:

1 - Definieren Sie Ihr Fragment

public class MyFragment extends Fragment {

    private String parameter;

    public MyFragment() {
    }

    public void setParameter(String parameter) {
        this.parameter = parameter;
    } 
}

2 - Erstellen Sie Ihr neues Fragment und füllen Sie den Parameter aus

    myfragment = new MyFragment();
    myfragment.setParameter("here the value of my parameter");

3 - Genieße es!

Natürlich können Sie den Typ und die Anzahl der Parameter ändern. Schnell und einfach.