Einfaches Widget bringt Android zum Stillstand

sudave · 8. Dezember 2011

Um mich in Java und die Android Entwicklung einzufinden, habe ich eine einfache Binär-Uhr programmiert.

Die Uhrzeit wird dabei in Stunden, Minuten, Sekunden umgewandelt.
Diese drei Werte in einen Binär-String umgewandelt und dieser auf 5 Zeichen für die Stunden und 6 Zeichen für Min/Sek formatiert.
Je nachdem ob an den Stellen 1 - 5 (6) eine 0 oder eine 1 ist, wird ein ImageView angezeigt/ausgeblendet.

Soweit funktioniert auch alles.
Nach einigen Minuten wird allerdings alles sehr langsam. "Tastendrücke" reagieren sehr spät (nach sehr langer Zeit gar nicht mehr), der animierte Hintergrund wird statisch und Musik hört auf zu spielen.
Beim Emulator unter Windows 7 ist es genau das gleiche, jedoch schon nach kürzerer Zeit.
Prozessorauslastung ist bei beiden Systemen im normalen Bereich, Android 2% und Windows zwischen 15% und 20%.
Allerdings wird der Speicherverbrauch unter Windows in etwa 4KB größer mit jeder Sekunde.

Ich habe so gut wie alle Variablen schon außerhalb deklariert und bin deshalb ratlos. Normalerweise hat jede App ja auch einen begrenzten Speicher zur Verfügung, deswegen dürfte ja nicht das ganze System darunter leiden..

Der relevante Code:

Code

public String dec2bin(int decimal)
	{
    	digits = "01";
    	binary = "";
    	while(decimal != 0)
    	{
        	binary = digits.charAt(decimal % 2) + binary;
        	decimal = decimal / 2;
    	}
    	return binary;
	}
		
	protected void setTimer()
	{
		taskHandler = new Handler();
		
		Ticker = new Runnable() {			
			public void run()
			{
				runNextTask();
				//curDate = new Date();
				//RV.setTextViewText(R.id.textView1, curDate.toLocaleString());
				//AWM.updateAppWidget(AWI, RV);
				
	        	if (isComplete) return;
            	now = SystemClock.uptimeMillis();
            	next = now + (1000 - now % 1000);
            	taskHandler.postAtTime(Ticker, next);
			}
		};
		
		Ticker.run();
	}
	
	protected void runNextTask()
	{
			curDate = new Date();
			Hours = curDate.getHours();
			Mins = curDate.getMinutes();
			Secs = curDate.getSeconds();
			
			binSecs = dec2bin(Secs);
			int len = binSecs.length();
			if( len &lt; sFillStrWithWantLen.length() )
			  binSecs = (sFillStrWithWantLen + binSecs).substring(len);


			RV.setViewVisibility(R.id.ImageView36, ((binSecs.charAt(0) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView35, ((binSecs.charAt(1) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView34, ((binSecs.charAt(2) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView33, ((binSecs.charAt(3) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView32, ((binSecs.charAt(4) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView31, ((binSecs.charAt(5) == '0') ? -1 : 0));
			
			String binMins = dec2bin(Mins);
			len = binMins.length();
			if( len &lt; sFillStrWithWantLen.length() )
			  binMins = (sFillStrWithWantLen + binMins).substring(len);


			RV.setViewVisibility(R.id.ImageView26, ((binMins.charAt(0) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView25, ((binMins.charAt(1) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView24, ((binMins.charAt(2) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView23, ((binMins.charAt(3) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView22, ((binMins.charAt(4) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView21, ((binMins.charAt(5) == '0') ? -1 : 0));
			
			String binHours = dec2bin(Hours);
			len = binHours.length();
			if( len &lt; sFillStrWithWantLenH.length() )
			  binHours = (sFillStrWithWantLenH + binHours).substring(len);


			RV.setViewVisibility(R.id.ImageView15, ((binHours.charAt(0) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView14, ((binHours.charAt(1) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView13, ((binHours.charAt(2) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView12, ((binHours.charAt(3) == '0') ? -1 : 0));
			RV.setViewVisibility(R.id.ImageView11, ((binHours.charAt(4) == '0') ? -1 : 0));
			
			// RV.setTextViewText(R.id.textView1, String.format("%d", new Random().nextInt()));


			AWM.updateAppWidget(AWI, RV);
	}

Alles anzeigen

RV ist das RemoteView-Objekt, das in onUpdate erstellt wird und
AWM der AppWidgetManager auch aus onUpdate.

Vielleicht fällt jemandem etwas dazu ein? Es ist mir wie gesagt ein Rätsel woher und wie das Widget ein Handy "aufhängen" kann..

titus · 16. Dezember 2011

Hi sudave,

1.) bitte schreibt Kommentare in deinem Code (zbsp. was soll mit dem Code erreicht werden oder was wird hier berechnet)

2.)

Zitat

Ich habe so gut wie alle Variablen schon außerhalb deklariert und bin deshalb ratlos.

Das macht deinen Code nur unleserlicher und du schafst dir zusätzliche Fehlerquellen durch globale Variablen.

3.)

Zitat

Allerdings wird der Speicherverbrauch unter Windows in etwa 4KB größer mit jeder Sekunde.

Meinst du RAM oder Speicher auf der Festplatte?
Ich geh mal davon aus du meinst den RAM. Das würde auch das Verhalten deines Apps erklären.
In der DDMS gibt es Möglichkeiten sich das Laufzeitverhalten der Applikation an zu sehen.

4.) Warum arbeitest du eigentlich mit Zeichenketten? Strings sind in Java nicht veränderbar das bedeutet das sowas

Zitat

binary = digits.charAt(decimal % 2) + binary;

jedesmal ein neue Instanz der Klasse String erzeugt. Da Binärzahlen nur aus 0 und 1 "bestehen" würde sich doch viel eher sowas wie int anbieten.

MFg Titus

sudave · 6. Januar 2012

Zu 2 und 3: habe die Variablen lokal gemacht, wo es nicht unbedingt global sein musste.
Und meinte den RAM, ja.

Habe die dec2bin Funktion auf einen Integer umgeschrieben:

Code

public String dec2bin(int decimal)
	{
    	Integer binary = 0; 
    	while(decimal != 0)
    	{
        	binary = binary * 10;
        	if (decimal % 2 != 0) { 
        		binary = binary + 1;
        	}
        	
        	decimal = decimal / 2;
    	}
    	return binary.toString();
	}

Alles anzeigen

Allerdings brauche ich den String später, weil am Anfang der Zeichenkette auch eine 0 stehen kann, was einem Integer natürlich nichts bedeutet..

Dadurch hat sich nichts geändert.

Habe nebenbei aber noch bemerkt, dass alles von Anfang an schon ausgebremst ist.
Läuft das Widget, ruckelt das App-Menü beim Seitenwechsel.
Der Timer löst aber wenigstens jede Sekunde aus, dadurch sollte es vielleicht jede Sekude kurz stocken, aber doch nicht durchgehend..

titus · 16. Januar 2012

Hi sudave,

haste mal in der DDMS geschaut nach der Möglichkeit sich das Laufzeitverhalten anzeigen zu lassen.

Profiling with TraceView
Quelle: Android Developers

Mfg Titus

Einfaches Widget bringt Android zum Stillstand

Jetzt mitmachen!

Teilen