CountDown in un thread a "loop infinito"

[robyrocker89]

Ciao a tutti,
viste le mie scarse conoscenze in java sto cercando di risolvere un enigma da un paio di giorni senza alcun risultato...
Il mio intento è quello di eseguire un countdown all interno di un thread e farlo ripartire ogni volta che il timer arriva a 0...

vi allego il codice:

public class Login extends Activity
{
    /** Called when the activity is first created. */
	private int time_total=5000;
	private int time_tick=1000;
	private TimerCountDown counter;
	private Thread time;
	
	@Override
    public void onCreate(Bundle savedInstanceState) 
    {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.login);
        
        counter = new TimerCountDown(time_total,time_tick);
        
        time=new Thread(new Runnable() 
		{
		    public void run() 
		    {
		    	counter.start();
		    }
		});
        time.start();
      }


public class TimerCountDown extends CountDownTimer
{
	public TimerCountDown(long millisInFuture, long countDownInterval) 
	{
		super(millisInFuture, countDownInterval);
	}
		 
		@Override
		public void onFinish() 
		{
			Toast err_http = Toast.makeText(Login.this, "Finito!!", Toast.LENGTH_LONG);
    	                err_http.show();
		}
		 
		@Override
		public void onTick(long millisUntilFinished) 
		{
			
			Toast err_http = Toast.makeText(Login.this, "Rimanente: " + millisUntilFinished/1000, Toast.LENGTH_LONG);
    	                err_http.show();

		}
		 
}

Grazie in anticipo!!!

[eafkuor]

public class Login extends Activity<br /> {<br /> /** Called when the activity is first created. */<br /> private int time_total=5000;<br /> private int time_tick=1000;<br /> private TimerCountDown counter;<br /> private Thread time;<br /> <br /> @Override<br /> public void onCreate(Bundle savedInstanceState) <br /> {<br /> super.onCreate(savedInstanceState);<br /> setContentView(R.layout.login);<br /> <br /> startTimer();<br /> <br /> }<br /> <br /> public void startTimer(){<br /> <br /> counter = new TimerCountDown(this, time_total,time_tick);<br /> <br /> time=new Thread(new Runnable() <br /> {<br /> public void run() <br /> {<br /> counter.start();<br /> }<br /> });<br /> time.start();<br /> }<br /> <br /> public class TimerCountDown extends CountDownTimer<br /> {<br /> private Activity activity;<br /> public TimerCountDown(Activity a, long millisInFuture, long countDownInterval) <br /> {<br /> super(millisInFuture, countDownInterval);<br /> activity=a;<br /> }<br /> <br /> @Override<br /> public void onFinish() <br /> {<br /> Toast err_http = Toast.makeText(Login.this, "Finito!!", Toast.LENGTH_LONG);<br /> err_http.show();<br /> <br /> activity.startTimer();<br /> }<br /> <br /> @Override<br /> public void onTick(long millisUntilFinished) <br /> {<br /> <br /> Toast err_http = Toast.makeText(Login.this, "Rimanente: " + millisUntilFinished/1000, Toast.LENGTH_LONG);<br /> err_http.show();<br /> <br /> }<br /> <br /> }

L'ho scritto ora nel browser quindi perdonami se è sbagliato, cmq in linea di massima dovrebbe essere giusto.

Una alternativa sarebbe di usare handler.postDelayed()

[robyrocker89]

L'ho scritto ora nel browser quindi perdonami se è sbagliato, cmq in linea di massima dovrebbe essere giusto.

Una alternativa sarebbe di usare handler.postDelayed()

Ti ringrazio!!! funziona alla perfezione!!!
un ultima cosa: questo thread gira "sopra" a google maps ed ogni 5 secondi esegue una query...il problema è che quando esegue la query c'è un rallentamento di google maps (va un po' a scatti per un paio di secondi)...dici che mettendo la visualizzazione della mappa in un nuovo thread risolverei il problema?

Grazie ancora!!

[eafkuor]

Io non so quanto convenga creare un altro Thread.
Perché non provi ad usare un handler?

[robyrocker89]

Io non so quanto convenga creare un altro Thread.
Perché non provi ad usare un handler?

Ciao,
scusa ma ho letto solo ora la risposta!

un handler? potresti cortesemente farmi un esempio di come impostarlo?
che vantaggi avrei rispetto al thread?

ti ringrazio ancora!!

ps: il codice da te postato va che è una meraviglia...peccato che ogni 5 secondi (limite che ho impostato io) esegua una query e la mappa si pianti per un paio di secondi...è risolvibile questo secondo te?

[eafkuor]

Si, usando appunto un Handler
Ora sono appena tornato a casa e sto un po' distrutto, domani ti faccio un esempio.
Cmq è molto facile, devi usare Handler.postDelayed(Runnable r, long millis), poi da dentro al runnable alla fine fai if(devo ripetere l'operazione) handler.postDelayed(this, millis)
Meglio che ti faccio un esempio domani mi sa

[eafkuor]

Dentro la classe che esegue la query innanzitutti dichiari una variabile Handler:

Handler handler = new Handler();

poi ci metti una cosa così:

private Runnable sendQuery = new Runnable(){<br /> <br /> public void run() {<br /> <br /> faiLaQuery();<br /> <br /> if(devi fare un'altra query)<br /> handler.postDelayed(sendQuery, 5000);<br /> <br /> }<br /> <br /> };

per iniziare a fare le query:

handler.post(sendQuery);

[robyrocker89]

Dentro la classe che esegue la query innanzitutti dichiari una variabile Handler:

Handler handler = new Handler();

poi ci metti una cosa così:

private Runnable sendQuery = new Runnable(){<br /> <br /> public void run() {<br /> <br /> faiLaQuery();<br /> <br /> if(devi fare un'altra query)<br /> handler.postDelayed(sendQuery, 5000);<br /> <br /> }<br /> <br /> };

per iniziare a fare le query:

handler.post(sendQuery);

mi è tutto chiaro tranne un particolare: quell if dentro al metodo run() è necessario?
non ho ben capito il "if(devi fare un'altra query)"...

io ho provato a impostarlo così ma dopo 2 o 3 cicli va in crash l'app:

<br /> private Runnable update_position = new Runnable(){ <br /> <br /> public void run() <br /> { <br /> try<br /> {<br /> //al posto del toast ci andrà la query<br /> Toast ok_query = Toast.makeText(Act1.this, "query eseguita", Toast.LENGTH_LONG);<br /> ok_query.show();<br /> handler.postDelayed(update_position, 5000);<br /> }<br /> catch(Exception e)<br /> {<br /> //exception...<br /> } <br /> <br /> } <br /> <br /> };<br />

ovviamente poi dopo metto handler.post(update_position);

[eafkuor]

Quell'if serve semplicemente per smettere di eseguire query fino all'infinito, immagino che prima o poi debba smettere no?
Comunque posta il messaggio di errore che ti da!

p.s. per svuotare la coda dei runnable da eseguire usa handler.removeCallbacks(update_position), forse è quello il problema

[robyrocker89]

Quell'if serve semplicemente per smettere di eseguire query fino all'infinito, immagino che prima o poi debba smettere no?
Comunque posta il messaggio di errore che ti da!

p.s. per svuotare la coda dei runnable da eseguire usa handler.removeCallbacks(update_position), forse è quello il problema

il fatto è proprio quello!
ti spiego il progetto a cui sto lavorando: in pratica l'app mi visualizza la mappa di google maps...alla pressione di un pulsante l'app fa 3 cose:
1. aggancia il satellite GPS per la rilevazione della posizione
2. esegue una query di update (ogni 5 secondi) su un db mysql (in remoto) delle mie coordinate
3. esegue una query di select (ogni 5 secondi) sullo stesso db mysql per prendermi le coordinate di un altro utente in base al suo username e visualizza un overlay nella posizione relativa alle coordinate prese dalla select...

ecco...grazie a te riesco ad eseguire le 2 query in un handler (o forse meglio 2 handler?) ogni 5 secondi...il problema è che l'interfaccia grafica si pianta per troppo tempo mentre esegue le 2 query e rende parecchio frustrante l'esperienza dell utente...mi chiedevo se c'era un modo per evitare che ciò accadesse...forse un modo sarebbe utilizzare i metodi wait() ecc... ma sinceramente non avendo grande esperienza in java (e in particolare con l'uso di thread/handler) non saprei dove piazzarli...

ti allego il codice nella speranza che tu possa darmi una mano (si lo so, sono una persona frustrante ma se ti rompo non esitare a dirmelo!! ):

<br /> public class Act1 extends MapActivity<br /> {<br /> <br /> private MapView mapView;<br /> private MapController mc;<br /> private LocationManager locationManager;<br /> private MyLocationOverlay myLocationOverlay;<br /> <br /> private String tvLatitudine;<br /> private String tvLongitudine;<br /> private TextView tvLatitudine2;<br /> private TextView tvLongitudine2;<br /> <br /> //private TextView tvLatitudine;<br /> //private TextView tvLongitudine;<br /> protected boolean _active = true;<br /> protected int _splashTime = 5000;<br /> <br /> private int time_total=5000;<br /> private int time_tick=1000;<br /> private Thread time;<br /> <br /> private Intent login_user;<br /> private Intent login_user_search;<br /> private Intent login_user_myfriends;<br /> private Intent login_user_myfollower;<br /> private Intent login_user_mynewfollower;<br /> private String pkg;<br /> <br /> private String pkg_search=null;<br /> private String pkg_myfriends=null;<br /> private String pkg_myfollower=null;<br /> private String pkg_mynewfollower=null;<br /> private int new_follower;<br /> private GeoPoint point2;<br /> private Handler handler = new Handler(); <br /> private Handler handler2 = new Handler(); <br /> private String user;<br /> private int lng=0;<br /> private int ltt=0;<br /> <br /> //Metodo "main()" che viene lanciato all inizio dell'app.<br /> //Carica e visualizza la mappa di Google Maps<br /> @Override public void onCreate(Bundle savedInstanceState) <br /> {<br /> super.onCreate(savedInstanceState);<br /> setContentView(R.layout.main);<br /> <br /> mapView = (MapView) findViewById(R.id.mapview);<br /> mapView.setBuiltInZoomControls(true);<br /> mc = mapView.getController(); <br /> }<br /> <br /> final Runnable update_position = new Runnable()<br /> {<br /> public void run() <br /> {<br /> //eseguo l'update della mia posizione<br /> update_mylocation();<br /> handler.postDelayed(this, 8000);<br /> }<br /> };<br /> <br /> final Runnable see_friends = new Runnable()<br /> {<br /> public void run() <br /> {<br /> //select delle coordinate dell'altro utente<br /> i_see_my_friend();<br /> <br /> //Gestiamo gli Overlay<br /> Drawable img = getResources().getDrawable(R.drawable.pushpin);<br /> Posizioneutente overlays_interesse = new Posizioneutente(getBaseContext(),img,ltt,lng);<br /> <br /> //sovrappongo l'Overlay<br /> mapView.getOverlays().add(overlays_interesse);<br /> handler2.postDelayed(this, 8000);<br /> }<br /> };<br /> <br /> @Override<br /> protected boolean isRouteDisplayed() {<br /> return false;<br /> }<br /> <br /> //Creazione del menù<br /> @Override<br /> public boolean onCreateOptionsMenu(Menu menu)<br /> {<br /> super.onCreateOptionsMenu(menu);<br /> menu.add(0,0,1,"I'm here!").setIcon(R.drawable.imhere);<br /> return true;<br /> }<br /> <br /> @Override<br /> public boolean onOptionsItemSelected(MenuItem item) {<br /> switch (item.getItemId()) {<br /> <br /> //Localizzazione della posizione tramite GPS e disegno del pallino azzurro<br /> case 0:<br /> locationManager = (LocationManager) getSystemService(Context.LOCATION_SERVICE);<br /> boolean isGPS = locationManager.isProviderEnabled (LocationManager.GPS_PROVIDER);<br /> <br /> if(isGPS==true)<br /> {<br /> myLocationOverlay = new MyLocationOverlay(this, mapView);<br /> List<Overlay> overlays = mapView.getOverlays();<br /> overlays.add(myLocationOverlay);<br /> myLocationOverlay.enableMyLocation();<br /> locationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER, 0,0, new GeoUpdateHandler());<br /> handler.postDelayed(update_position, 8000);<br /> handler2.postDelayed(see_friends, 8000);<br /> <br /> }<br /> //else<br /> {<br /> GPS_Settings();<br /> }<br /> break;<br /> <br /> }<br /> return false;<br /> }<br /> <br /> //Memorizzazione e visualizzazione in tempo reale delle coordinate GPS<br /> public class GeoUpdateHandler implements LocationListener <br /> {<br /> public void onLocationChanged(Location location) <br /> {<br /> int lat = (int) (location.getLatitude() * 1E6);<br /> int lng = (int) (location.getLongitude() * 1E6);<br /> <br /> tvLatitudine=Double.toString(getRound(location.getLatitude(), 5));<br /> tvLongitudine=Double.toString(getRound(location.getLongitude(), 5));<br /> <br /> <br /> //GeoPoint point = new GeoPoint(lat, lng);<br /> <br /> //mc.animateTo(point);<br /> }<br /> <br /> public void onProviderDisabled(String provider) {}<br /> public void onProviderEnabled(String provider) {}<br /> public void onStatusChanged(String provider, int status, Bundle extras) {}<br /> }<br /> <br /> //Metodo di conversione e calcolo delle coordinate in "double"<br /> public static double getRound(double x, int digits)<br /> {<br /> double powerOfTen = Math.pow(10, digits);<br /> return ((double)Math.round(x * powerOfTen) / powerOfTen);<br /> }<br /> }<br />



ho tagliato delle parti che non servivano a nulla per semplificare un po'...
ho provato a fare 2 handler al posto di uno ma non cambia nulla...
sicuramente c'è qualcosa che va sistemato in termini di "sincronizzazione" e di priorità...
se ti va dacci un occhio...se un giorno ci incontreremo non esiterò a offrirti da bere...anzi, ti compro il bar!!!