En trådfråga

jag har följande klass som används från en activity:

Kod:

public class SimpleTimer extends Thread{

	private int interval;
	private SimpleTimerListener listener;
	private boolean closed;

	/**
	 * @param interval i sekunder
	 */
	public SimpleTimer(int interval, SimpleTimerListener listener){
		this.interval = interval;
		this.listener = listener;
	}
	
	public void run() {
		closed = false;
		while(!closed){
			listener.doTask();	
			try {
				Thread.sleep(interval * 1000);
			} catch (InterruptedException e) {}
		}
	}
	
	public void close(){
		closed = true;
	}
	
	public int getInterval() {
		return interval;
	}

	public void setInterval(int interval) {
		this.interval = interval;
	}	
}

När jag startar timern så får jag följande exception:

Kod:

07-22 12:03:10.372: ERROR/AndroidRuntime(463): FATAL EXCEPTION: Thread-8
07-22 12:03:10.372: ERROR/AndroidRuntime(463): java.lang.RuntimeException: Can't create handler inside thread that has not called Looper.prepare()
07-22 12:03:10.372: ERROR/AndroidRuntime(463):     at android.os.Handler.<init>(Handler.java:121)
07-22 12:03:10.372: ERROR/AndroidRuntime(463):     at android.location.LocationManager$ListenerTransport$1.<init>(LocationManager.java:139)
07-22 12:03:10.372: ERROR/AndroidRuntime(463):     at android.location.LocationManager$ListenerTransport.<init>(LocationManager.java:137)
07-22 12:03:10.372: ERROR/AndroidRuntime(463):     at android.location.LocationManager._requestLocationUpdates(LocationManager.java:708)
07-22 12:03:10.372: ERROR/AndroidRuntime(463):     at android.location.LocationManager.requestLocationUpdates(LocationManager.java:630)
...

Vad gör jag för fel?

 

Problemt ligger nog inte i den koden du lagt upp. Vad gör du i callback-metoden SimpleTimerListener.doTask()?

 

Jag håller med, problemet ligger inte i koden som visas här.

@112. Två minor saker. closed samt interval skall deklareras som volatile.

 

Ja ni har nog rätt i att felet inträffar efter det jag visade.

doTask() finns i själva activityn och startar en ny tråd där som hämtar en gps-position

det fungerar om jag anropar doTask() utan att anropa den från mitt SimpleTimer-objekt

 

Känns spontant som om det är onödigt många trådar inblandade...
Vilken requestLocationUpdates-metod använder du?

requestLocationUpdates(String provider, long minTime, float minDistance, LocationListener listener)
bör enligt dokumentationen anropas från actvityns huvudtråd.

 

så här ser hela stacktracet ut:

Kod:

07-22 21:22:31.292: ERROR/AndroidRuntime(623): java.lang.RuntimeException: Can't create handler inside thread that has not called Looper.prepare()
07-22 21:22:31.292: ERROR/AndroidRuntime(623):     at android.os.Handler.<init>(Handler.java:121)
07-22 21:22:31.292: ERROR/AndroidRuntime(623):     at android.location.LocationManager$ListenerTransport$1.<init>(LocationManager.java:139)
07-22 21:22:31.292: ERROR/AndroidRuntime(623):     at android.location.LocationManager$ListenerTransport.<init>(LocationManager.java:137)
07-22 21:22:31.292: ERROR/AndroidRuntime(623):     at android.location.LocationManager._requestLocationUpdates(LocationManager.java:708)
07-22 21:22:31.292: ERROR/AndroidRuntime(623):     at android.location.LocationManager.requestLocationUpdates(LocationManager.java:630)
07-22 21:22:31.292: ERROR/AndroidRuntime(623):     at se.ClientSide.MyLocation.getLocation(MyLocation.java:35)
07-22 21:22:31.292: ERROR/AndroidRuntime(623):     at se.ClientSide.ClientInterface.getNewLocation(ClientInterface.java:145)
07-22 21:22:31.292: ERROR/AndroidRuntime(623):     at se.ClientSide.ClientInterface.doUpdate(ClientInterface.java:132)
07-22 21:22:31.292: ERROR/AndroidRuntime(623):     at se.ClientSide.ClientInterface.doTask(ClientInterface.java:127)
07-22 21:22:31.292: ERROR/AndroidRuntime(623):     at se.ClientSide.SimpleTimer.run(SimpleTimer.java:20)

 

Hemma har jag fulkod som nog liknar det du vill göra. Om du har en egen tråd som skall lyssna på location updates så måste det skötas via en Looper.

 

dela gärna med dig

 

Den här lyssnar efter location updates för både GPS och NETWORK. Har även en timer som stänger ner den efter en viss tid.

http://github.com/johannilsson/sthl...partist/sthlmtraveling/MyLocationManager.java

 

Min klass som hämtar positionen ser nästan exakt ut som den

 

Ah, men då funkar det ju

 

Ja det fungerar, men inte när jag kör från min timer-klass

 

Jag postar kod (troligen under morgondagen)

 

Vad är det du vill göra, kanske lättare att börja där? Ett alternativ är att kolla på Handler. Eller fula in ett runOnUiThread anrop från din doTask som tvingar den att köra från ui tråden.

 

Det jag vill göra är att hämta en position med ett visst intervall via gps eller nätverk.

requestLocationUpdates(provider, tid, avstånd, lyssnare);

ska kolla lite på vad man kan göra med tid-parametern ..

 

Om jag minns rätt så är det precis vad minTime parametern för requestLocationUpdates gör. Klassen som jag postade wrappar både gps och nätverks providern. Jag byggde den för mitt specifika case men den borde täcka det du vill göra med. Finns ingen anledning som jag ser det att sätta upp en egen timer för att anropa den i ett visst interval då detta redan sker internt i LocationManager.

 

Ja det fungerade.. även om det är liiite ostabilt när den måste ta getLastKnownLocation()..

Har en fråga till:

Jag har en ScrollView där jag lägger till text efter hand med appendText(blabla), hur gör jag om jag vill ändra färg på texten varje gång jag anropar appendText()? ska jag göra en ny textView för varje ny text jag lägger till och använda .setTextColor()? jag försökte med det men det fungerade inte..


07-23 13:44:35.433: ERROR/AndroidRuntime(934): Caused by: java.lang.IllegalStateException: ScrollView can host only one direct child

 

Ok. Då behöver jag inte posta min kod

Använd en Spannable istället för en String när du lägger till text. En Spannable kan innehålla information om hur texten skall rendreras. Färg, font osv.

 

Något liknandes:

Kod:

SpannableString spannable = new SpannableString(myString);
spannable.setSpan(new ForegroundColorSpan(0xffff0000), 0, myString.length(), Spannable.SPAN_EXCLUSIVE_EXCLUSIVE);

Helt otestat

Edit: En förtydligan. TextView.append tar CharSequence som argument, och SpannableString är en CharSequence

 

Funkade utmärkt, tackar