FragmentManager

Eddi · 27. Oktober 2019

Hallo,

ich habe eine Activity (AppCompatActivity) in der ein FrameLayout durch ein Fragment ausgetauscht wird.

Java

MyFragment fragment = new MyFragment();
FragmentTransaction transaction = getSupportFragmentManager().beginTransaction();
transaction.replace(R.id.id_framelayout, fragment);
transaction.setTransition(FragmentTransaction.TRANSIT_FRAGMENT_OPEN);
//transaction.addToBackStack(null);
transaction.commit();

In diesem Fragment (MyFragment) ist ein ViewPager mit 3 Fragmenten, welche in einem Adapter zum Activity-FragmentManager hinzugefügt werden.
Also stecken im FragmentManager der Activity 4 Views.
Wenn ich nun das Handy drehe stürtzt meine App ab und ich weiss nicht warum.
Der Absturz passiert in onCreate der Activity, den im Bundle savedInstanceState sind alle 4 Views vermerkt.

Wenn ich vor dem Handy drehen versuche die 3 Views zu entfernen, hat dies keinen Effekt denn sie bleiben im FragmentManager.

Was mache ich falsch oder muss ich einen separaten FragmentManager in MyFragment erstellen damit die 3 Views aus dem ViewPager isoliert sind.

Ich verstehe das nicht, denn ohne den ViewPager funktioniert das drehen, wenn nur ein Fragment im FragmentManager ist.

Grüße Eddi

nono124 · 27. Oktober 2019

Hallo
Um hier eine sinnvolle aussage machen zu können solltest du uns den Code der MyFragment Klasse zeigen.

Ich verstehe auch nicht warum du ein Fragment erstellt und in diesen wieder drei Fragmente mit einen Viewpager.

Ohne Code kommen wir hier nicht weiter.

PS.

Zitat

In diesem Fragment (MyFragment) ist ein ViewPager mit 3 Fragmenten, welche in einem Adapter zum Activity-FragmentManager hinzugefügt werden.

Wenn dem wirklich so ist kann das nicht gehen.

Du hast eine Activity mit einem Layout in dem ist eine FrameLayout das als Container für das Fragment dient.

Wenn du in der Klasse MyFragment wieder ein Fragment startest oder einen Viewpager benutzt, dann muss in dem Layout was du in der MyFragment Klasse lädst wieder ein Container, ViewPager sein. Und in genau diesen Container sollte die Drei Fragmente geladen werden. Nicht in dem Container des Activity Layouts.

Mit dem Activity-FragmentManager haben die drei Fragmente nichts mehr zu tun.

Hoffe das es etwas klarer geworden ist.

Eddi · 27. Oktober 2019

Hallo jogimuc,

ich hab mal den Code zusammengefasst damit es leserlich bleibt.

Java

public class Startseite extends AppCompatActivity
{
    protected void onCreate(Bundle savedInstanceState)
    {
	super.onCreate(savedInstanceState);
	setContentView(R.layout.activity_startseite);


	m_framelayout = findViewById(R.id.id_framelayout);


	m_fragment_termine = new Termine();
	FragmentTransaction transaction = getSupportFragmentManager().beginTransaction();
	transaction.replace(R.id.id_framelayout, m_fragment_termine);
	transaction.setTransition(FragmentTransaction.TRANSIT_FRAGMENT_OPEN);
	//transaction.addToBackStack(null);
	transaction.commit();
    }
}


public class Termine extends Fragment
{
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState)
    {
        View root_view = inflater.inflate(R.layout.fragment_termine, container, false);


	m_viewPager = root_view.findViewById(R.id.id_viewPager);


	m_fragment_termine_1 = new TermineSub();
	m_fragment_termine_2 = new TermineSub();
	m_fragment_termine_3 = new TermineSub();
	m_adapter_fragmente = new AdapterFragmente(getActivity().getSupportFragmentManager());
	m_adapter_fragmente.addFragment(m_fragment_termine_1);
	m_adapter_fragmente.addFragment(m_fragment_termine_2);
	m_adapter_fragmente.addFragment(m_fragment_termine_3);
	m_viewPager.setAdapter(m_adapter_fragmente);
    }
}


public class TermineSub extends Fragment
{
	...
}


public class AdapterFragmente extends FragmentStatePagerAdapter
{
    public final List&lt;Fragment&gt; mFragmentList = new ArrayList&lt;&gt;();
    private FragmentManager mFragmentManager;


    public AdapterFragmente(FragmentManager manager)
    {
        super(manager);
        this.mFragmentManager = manager;
        mFragmentList.clear();
    }


    @Override
    public Fragment getItem(int position)
    {
        return mFragmentList.get(position);
    }


    @Override
    public int getCount()
    {
        return mFragmentList.size();
    }


    public void addFragment(Fragment fragment)
    {
        mFragmentList.add(fragment);
        notifyDataSetChanged();
    }


    public void deleteFragment(Fragment fragment)
    {
        mFragmentList.remove(fragment);
//        FragmentTransaction transaction = mFragmentManager.beginTransaction(); //(funktioniert nicht, fragmente werden nicht gelöscht)
//        transaction.remove(fragment);
//        transaction.commit();
        notifyDataSetChanged();
    }
}

Alles anzeigen

nono124 · 28. Oktober 2019

Hallo
Habe mir mal deinen Code angesehen.

Frage wann und von wo wird die Methode „deleteFragment“ , im Adapter, aufgerufen was soll die überhaupt ? Wieso müssen die Fragmente gelöscht werden? Das macht der GC eigentlich selber wenn die Instanzen nicht merhr gültig sind.

Was soll überhaupt das „notifyDataSetChanged()“ in der Add Methode im Adapter?

Auch ein „notify“ in der deleteFragment Methode finde ich sinnlos.
Die Methode notifyDataSetChanged() innerhalb des Adapters aufzurufen ist für mich sinnlos.
Die Methode ist dafür gedacht von außerhalb das neu Zeichnen zu veranlassen. Nicht intern sich selber.

Warum musst du die liste der Fragments im Constructor löschen wird nicht problematisch sein nur wieso?

Ob das mit dem Fragment Manager aus der Activity so richtig ist kann ich nicht sagen. Habe ich bis jetzt noch nicht gemacht ein Fragment im Fragment.
Da aber es beim ersten Aufbau geht wird das wohl nicht der Grund des Absturz sein.

Welcher Fehler kommt den beim Absturz?
Wo in welcher Methode passierst es denn. Mache dir doch ein paar Logs um das nach zu vollziehen.

Bein drehen wird ja die Activity neu gestartet und auch die onCreate wieder durchlaufen.
Es müssten somit auch deine Fragmente wider neu aufgerufen werden. Und auch der Pager in dem ersten Fragment.

Was ich aber bei dem ersten Fragment nicht richtig finde ist das du es nicht lädst sonder ersetzt. „transaction.replace“ zu dem Zeitpunkt ist eigentlich noch kein Fragment da zum austauschen.
Ich würde transaction.add( … machen. Das erste Fragment sollte immer add sein und anschießend tauschen was du ja gar nicht machst denn du hast ja nur ein Fragment.

Vielleicht machst du auch etwas in der onCreate was beim zweiten Aufruf zum Absturz führt.

Eddi · 28. Oktober 2019

Hallo jogimuc,

der Absturz war in der OnCreate der Activity "Startseite" beim Zugriff auf das Bundle savedInstanceState.
Ich hab die notifyDataSetChanged und deleteFragment entfernt und jetzt funktioniert es.

Jetzt hab ich aber ein weiteres Problem.

Beim drehen des Handys wird ja die Activity neu erstellt aber die Fragmente im FragmentManager bleiben bestehen und es kommen 3 neue hinzu.

(die beim Start ja erstellt werden) Nun habe ich wenn ich mehrmals drehe jedesmal mehr fragmente und alle werden neu initialisiert.

Daher wollte ich die alten Fragmente löschen. Oder wie kann ich prüfen welche Fragemente aktiv sind.

habs jetzt so:
Ich hab jetzt die Fragmente die vom FragmentManager automatisch erstellt werden und den standard construktor durchlaufen, nicht initialisiert.
Ich habe nur die 3 neuen Fragmente die durch einen erweiterten constructor erzeugt werden mit daten gefüllt, somit denke ich habe ich das Problem gelöst ?

nono124 · 29. Oktober 2019

Hallo schon zu hören das es geht.

Nur wundert es mich das nach dem drehen das die Fragmente noch da sein sollen.

Vom Fragmentmanager werden keine Fragmente automatisch erstellt das macht du selber in der onCreateView .
Eigentlich müsste die auch beim drehen wieder durchlaufen werden . kannste mit einem log mal prüfen.

damit erstellt du die Fagmente.
m_fragment_termine_1 = new TermineSub();
m_adapter_fragmente.addFragment(m_fragment_termine_1);

Wenn die Instanz nicht mehr gültig ist sollte das der GC freigeben.

Zeige mal wie du das mit dem Fragmentmanager und Constructor machst.

Kannst ja vorher prüfen ob die Objekte existieren vor dem neu erstellen.
if (m_fragment_termine_1 == Null{
m_fragment_termine_1 = new TermineSub();
m_adapter_fragmente.addFragment(m_fragment_termine_1);
}

FragmentManager

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema