Activity Lifecycle

Pommes9485 · 15. August 2014

Guten Tag,

ich habe ein wahrscheinlich relativ einfaches Problem:
Ich initialisiere meine Activity in der onCreate Methode.
Bisher war ich davon ausgegangen, wenn ich dann mit dem Pfeil zurück gehe und erneut die Activity starte, sie einfach resumed und onCreate wirklich nur einmal ausgeführt wird, allerdings ist dies nicht der Fall, stattdessen wird sie jedesmal wieder gestartet. Starten tue ich sie so:

Java

Intent i = new Intent(getApplicationContext(),
								BatteryScreen.class);
						i = i.putExtra("name", v.name);
						i = i.putExtra("id", v.id);
						i = i.putExtra("multi", m);
						startActivity(i);

Ist das verwerflich?
Oder muss ich irgendwas bestimmtes in die onDestroy Methode reinschreiben, damit sie alles auf 0 (null) setzt?

Das darausfolgende Problem ist eben, dass nach 6-7 zurück und starten, ich eine OutOfMemory Exception bekomme.

Marco Feltmann · 15. August 2014

Du könntest statt eigener Activities lieber Fragments verwenden, das erleichtert Dir später auch die Anpassung auf's Tablet.
Von irgend etwas auszugehen ist übrigens eine ganz schlechte Idee.

Nun, normalerweise sollte der GarbageCollector Deine Instanz wegräumen, wenn sie nicht mehr benötigt wird und es dürfte nie zu einem Speicherüberlauf kommen.
Du schreibst allerdings, dass Du in der onCreate Deiner Activity eine weitere Activity erstellst.
Nun kann ich nicht mit Bestimmtheit sagen, ob der Garbage Collector überhaupt eine Chance hat da irgendwas wegzuräumen.
Eventuell könntest Du (wenn Du unbedingt den eingeschlagenen Weg weiter verfolgen willst) Deinen Intent in eine Instanzvariable auslagern und auf diese zugreifen.

Da mir aber die Informationen fehlen, wann warum Du eine neue Activity aufrufst, kann ich hier nur raten.

Pommes9485 · 15. August 2014

Ich habe eine MainActivity, welche eine Liste enthält. Wenn auf einen dieser Einträge geklickt wird, starte ich eine neuen Activity, welche dann die "Details" anzeigt.
Diese 2. Activity verursacht auch die Probleme.

Ben · 15. August 2014

in der regel sollte das kein problem darstellen
wenn du auf zurück klickst wird die auf dem stack oben liegende activity zerstört und alle attribute etc in der activity werden freigegeben.
wenn du dann über die liste erneut den eintrag aufrufst sollte eine neue activity erzeugt werden und nicht nochmal ein und die selbe.

was ich nicht ganz verstehe, wie kannst du im onCreate deiner main die zweite activity erstellen, wenn sie doch aufgerufen werden soll wenn du auf die liste klickst

Pommes9485 · 15. August 2014

Die onCreate Methode bezog sich auf die 2. Activity
Dann weiß ich ehrlich gesagt nicht, woran es liegen kann. Ich benutze in der 2. Activity 6 AndroidPlot graphen. Diese haben jeweils eine Werte Array von 240 Werten zu Grunde liegen. Bei einem Aufruf oder auf bei 5 macht das keinen Unterschied nur dann wird es allmählich langsamer, um schließlich am Ende mit einer OoM Exception nur noch leere Felder anzuzeigen.

Ben · 15. August 2014

mh und diese exception kommt immer erst nachdem du die 2 activity zerstört hast und erneut geöffnet hast?
die exception kommt nie beim ersten öffnen?

Pommes9485 · 15. August 2014

Genau, immer erst nach dem 5. oder 6. mal, davor gehe ich halt rein (starte die Activity wie oben beschrieben), erstelle in der OnCreate die Graphen und co., gehe mit "Zurück" zurück in die Liste und dann wieder von vorne.

Ben · 15. August 2014

dann versuch mal in der on destroy deine belegten objekte auf null zu setzen und schau ob das problem dann immernoch auftritt.
Wenn danach alles funktioniert wird es wohl wie Marco schon sagte am Garbage Collector liegen.

Marco Feltmann · 15. August 2014

Ah, Du benutzt dort Views aus einem Third-Party Framework.
In dem Fall würde auch ich auf jeden Fall alles, was Du an Instanzvariablen (bzw. 'Membervariablen') in Deiner Activity hast, explizit auf NULL setzen.
Ansonsten kann es durchaus sein, dass da noch irgendwelche Referenzen gehalten werden und die Garbage Collection nicht vernünftig aufräumen kann.

Wenn die onDestroy() zuverlässig aufgerufen wird, solltest Du dort die Aufräumarbeiten durchführen.

Pommes9485 · 16. August 2014

Dann habe ich noch eine blöde Frage:
Wie setze ich denn die Views auf null?
Also jeder Graph ist in der Klasse XYPlot gespeichert, welche von View erbt, aber auch direkt die Daten zum Anzeigen speichert. Hole ich mir dann einfach den XYPlot mit findViewByID und setze ihn dann auf null? Das erscheint mir irgendwie seltsam, weil ich doch dann im Prinzip nur die "Kopie" des Views auf null setze, oder? Oder läuft das über die Referenz?

Danke schonmal, ich bin sehr optimistisch, dass es das ist

Ben · 18. August 2014

Du nimmst einfach die Variable her in der du die View gespeichert hast, und schreibst variablenname = null;

Pommes9485 · 18. August 2014

Also:

Java

PlotXY view = findViewByID(R.id.plot1);
view=null;

Ben · 18. August 2014

korrekt.
dann sollte theoretisch der speicher den die View belegt wieder freigegeben werden.

Pommes9485 · 18. August 2014

Danach besteht das Problem leider immer noch...
hier die volle Exception:

Code

08-18 15:10:29.562: E/dalvikvm-heap(6758): Out of memory on a 3113056-byte allocation.
08-18 15:10:29.562: I/dalvikvm(6758): "main" prio=5 tid=1 RUNNABLE
08-18 15:10:29.562: I/dalvikvm(6758):   | group="main" sCount=0 dsCount=0 obj=0x4189bf48 self=0x4188a028
08-18 15:10:29.562: I/dalvikvm(6758):   | sysTid=6758 nice=0 sched=0/0 cgrp=apps handle=1074839892
08-18 15:10:29.562: I/dalvikvm(6758):   | state=R schedstat=( 0 0 0 ) utm=836 stm=97 core=1
08-18 15:10:29.562: I/dalvikvm(6758):   at android.graphics.Bitmap.nativeCreate(Native Method)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.graphics.Bitmap.createBitmap(Bitmap.java:809)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.graphics.Bitmap.createBitmap(Bitmap.java:769)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.buildDrawingCache(View.java:13649)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13379)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13445)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.draw(View.java:14223)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewGroup.drawChild(ViewGroup.java:3118)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:2955)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13398)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13445)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.draw(View.java:14223)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewGroup.drawChild(ViewGroup.java:3118)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:2955)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.draw(View.java:14509)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.widget.FrameLayout.draw(FrameLayout.java:472)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.widget.ScrollView.draw(ScrollView.java:1635)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13403)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13445)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.draw(View.java:14223)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewGroup.drawChild(ViewGroup.java:3118)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:2955)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13398)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13445)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.draw(View.java:14223)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewGroup.drawChild(ViewGroup.java:3118)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:2955)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.draw(View.java:14509)
08-18 15:10:29.562: I/dalvikvm(6758):   at com.android.internal.widget.ActionBarOverlayLayout.draw(ActionBarOverlayLayout.java:381)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13403)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13445)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.draw(View.java:14223)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewGroup.drawChild(ViewGroup.java:3118)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:2955)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.draw(View.java:14509)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.widget.FrameLayout.draw(FrameLayout.java:472)
08-18 15:10:29.562: I/dalvikvm(6758):   at com.android.internal.policy.impl.PhoneWindow$DecorView.draw(PhoneWindow.java:2452)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13403)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.View.getDisplayList(View.java:13445)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.HardwareRenderer$GlRenderer.buildDisplayList(HardwareRenderer.java:1570)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.HardwareRenderer$GlRenderer.draw(HardwareRenderer.java:1449)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewRootImpl.draw(ViewRootImpl.java:2431)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewRootImpl.performDraw(ViewRootImpl.java:2301)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:1931)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1047)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:5896)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.Choreographer$CallbackRecord.run(Choreographer.java:761)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.Choreographer.doCallbacks(Choreographer.java:574)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.Choreographer.doFrame(Choreographer.java:544)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:747)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.os.Handler.handleCallback(Handler.java:733)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.os.Handler.dispatchMessage(Handler.java:95)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.os.Looper.loop(Looper.java:136)
08-18 15:10:29.562: I/dalvikvm(6758):   at android.app.ActivityThread.main(ActivityThread.java:5105)
08-18 15:10:29.562: I/dalvikvm(6758):   at java.lang.reflect.Method.invokeNative(Native Method)
08-18 15:10:29.562: I/dalvikvm(6758):   at java.lang.reflect.Method.invoke(Method.java:515)
08-18 15:10:29.562: I/dalvikvm(6758):   at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:792)
08-18 15:10:29.562: I/dalvikvm(6758):   at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:608)
08-18 15:10:29.562: I/dalvikvm(6758):   at dalvik.system.NativeStart.main(Native Method)

Alles anzeigen

Java

@Override
public void onDestroy()
{
	
	super.onDestroy();
	for(int ID : ids)
	{
		XYPlot p = (XYPlot) this.findViewById(ID);
		p = null;
	}
	}

Alles anzeigen

...und meine OnDestroy

Ben · 18. August 2014

du weißt in deinem on Destroy erst den variablen die views zu und löschst sie in der nächsten zeile wieder, HÄ?

es ging darum das du in deiner klasse doch sicher eine globale variable hast wo du deine views drin verwaltest, dort musst du die werte auf null setzen.

Activity Lifecycle

Jetzt mitmachen!

Teilen