Discussion :

[poussinvert]

Bonjour,
Je ne maitrise pas encore tout à fait les Thread et processus, mais sur mon appli android, je voudrais faire un décompte de 10..9..8.. jusqu'a 0, et à zéro, j'effectue une opération.
Pour cela, je souhaite créer un Thread qui tourne derrière le Thread UI et qui change un textview toute les secondes (le décompte).
J'ai créé une classe qui étend Thread dans un autre fichier que mon activity, et dans mon activity, j'ai créé un Handler qui reçoit le décompte du Thread pour mettre à jour la vue.

Mon thread :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
public class ThreadAutoStop extends Thread {
 
	// Bundle key
	public String MSG_KEY = "mykey";
 
	@Override
	public void run() {
		// TODO Auto-generated method stub
		super.run();
 
		int cpt = 10;
 
		while(cpt<=0){
 
			Message msg = SlotMachine.mHandler.obtainMessage();
 
			Bundle bundle = new Bundle();
			bundle.putString(MSG_KEY,str);
			msg.setData(bundle);
 
			msg.sendToTarget();
 
 
			// sleep a little while (1 second) and increment counter
			Thread.sleep(1000);
 
			cpt--;
		}
	}
}

Mon handler dans mon activity :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
public Handler mHandler = new Handler(){
 
			@Override
			public void handleMessage(Message msg) {
				// TODO Auto-generated method stub
				super.handleMessage(msg);
 
				TextView tv = (TextView) findViewById(R.id.tvTime);
				tv.setText(msg.getData().getString(ThreadAutoStop.MSG_KEY));
 
			}
 
		};

Le problème est que dans mon ma classe thread, le SlotMachine.mHandler est souligné en rouge en me demandant de changer le Handler en static. Si je change le handler en static, le findViewById(R.id.tvTime) dans le handler est souligné en rouge en me disant que c'est pas possible que le handler soit en static.
Le chien se mort la queue.
Merci pour votre aide =)

[nicroman]

Moi je pense qu'il n'y a pas besoin de thread surtout....

Quand on a un handler, on a le "sendMessageAtTime()"... qui permet d'envoyer un message à une date donnée... il suffit d'incrémenter cette date de 1000ms à chaque nouveau message.

Donc on crée le handler...
on fait un premier sendMessage(now);

Et dans la reception du message:
on récupère le 'time', on 'décrémente' le compteur, on change la textview et on fait (si compteur > 0) un sendMessageAtTime(now+1000,now+1000);

[grunk]

Pour répondre à ton problème.
Il faut effectivement passer par une classe static , et pour avoir la référence de l'activitie en cours , il faut utiliser une WeakReference :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
static class MyHandler extends Handler
{
	WeakReference<MonActivitie> ref;
 
	MainHandler(MonActivitie areref)
	{
		ref = new WeakReference<MonActivitie>(areref);
	}
 
	public void handleMessage(Message msg)
	{
		MonActivitie that = ref.get();
		if(that != null)
		{
			TextView tv = (TextView) that.findViewById(R.id.tvTime);
		}
	}
}

Plus d'infos ici :
http://developer.android.com/referen...Reference.html
http://stackoverflow.com/questions/1...ncominghandler

[poussinvert]

j'ai vu ce que vous me proposez :
@nicroman, c'est sûrement une bonne solution, mais je souhaiterais vraiment utiliser Handler et Thread, même si ce n'est pas optimisé du tout. L'application que je suis en train de concevoir est juste un test pour me permettre de comprendre comment tout cela fonctionne. Donc mon idée c'est d'utiliser un Thread qui envoie un Message au Handler pour mettre à jour la vue. Le message que je souhaite envoyer sera encapsulé dans un Bundle (j'ai vu qu'un Message avait arg1 et arg2, mais je souhaite utiliser setData()).

Pour bien expliquer mon problème, je suis la lignée de mon premier post. J'essaie de faire une machine à sous de type bandit manchot comme l'a deviné nicroman. j'ai réussi à faire tourner les 3 roues, et j'ai 3 boutons qui les arrête une par une.

Maintenant, Je souhaite ajouter un décompte qui par de 10, et arrivé à 0 il arrête la machine sur un symbole au hasard. Une fois que le décompte sera fait (j'espère avec un thread histoire de me faire la main), il faut déclencher l'arrêt de la machine. Donc je voulais faire un monThread.stop(), puis redéfinir le onStop() du thread pour arrêter les roues. * Bref, je ne sais pas comment faire tout ça. J'ai aussi le problème que si je clique deux fois sur start, il se lance 2 threads, donc je voudrais trouver un moyen de ne pas lancer le thread s'il existe déjà.

Pour l'instant, j'ai tout le programme, le thread et le handler dans un même fichier. Et une fois que mon compteur est à zéro, je ne sais que faire pour arrêter la machine. Mon idée aurait aussi été de séparer le Thread dans un autre fichier.

@grunk, c'est sympa ce que tu me proposes, mais j'avoue que là je suis définitevement largué. J'ai tenté de rechercher des tutos sur WeakReference, mais je ne comprends pas en quoi ça peut m'aider... A ce que j'ai compris, c'est pas mal utilisé pour faire du JAVA pur...

Pourriez-vous m'éclaircir les idées s'il vous plait ?
Même si je suis borné sur mes idées de thread, Je vous remercie du temps que vous passez à me proposez des solutions alternatives.

[grunk]

Un Message à une référence à Handler qui ,lorsqu'il n'est pas static mais une simple innerclass à également une référence à Activity .

Donc quand tu as des messages longs (genre delayed messages) il y'a une fuite de mémoire sur toutes les ressources et vues de l'activity , même si elle n'est plus affichée.
Tant que la file d'attente n'est pas vide on perd donc des ressources.

L'intérêt de la weakreference est de toujours être en mesure d'avoir une référence à l'activity (pour par exemple écrire dans un textview) mais que si besoin cette référence puisse être détruite par le garbage collector.

Plus d'infos sur les références ici : http://blog.developpez.com/adiguba/p2107/java/comprendre_les_references_en_java

[nicroman]

Salut,

Grunk a très bien expliqué le cas... pour faire simple:

Il y a 3 types de références en Java:
* Les "Strong" (la référence par défaut en Java).
* Les "Soft" (SoftReference<>)
* Les "Weak" (WeakReference<>)

Quand le garbage collector passe en revue les objets...
Les objets non référencés, ou seulement en "weak" sont immédiatement collectés.
Les objets référencés seulement en "soft" ne sont collectés *que* si il y a besoin de mémoire.
Les objets référencés "strong" ne sont pas collectés du tout.


Ceci dit, la perte de mémoire avec un Handler non statique peut parfaitement être évitée si les message sont correctement supprimés du handler au bon moment. (removeMessages ou removeCallbacks dans le onStop par exemple)


Quant au thread qui "sleep", quelqu'un m'a dit (et mon expérience fait que j'ai tendance à le croire), que le seul thread qui devrait pouvoir "dormir" est un scheduler d'interruptions...
Tous les autres threads n'ont jamais besoin de faire 'sleep'... et si on écrit "sleep" quelque part c'est qu'on a ou bien fait un thread de trop, ou oublié un "wait/signal" quelque part ailleurs ^^

[poussinvert]

Eh merci, c'est super sympa vos explications !
Alors en fait, si je comprends bien, une fuite de mémoire ou perte de mémoire, c'est le fait de laisser trainer trop de messages qui ne servent à rien dans le garbage collector ?

Pour la question des sleep, mon thread me sert à afficher un décompte pour ensuite arrêter les roues une par unes... j'ai donc une boucle avec un sleep qui passe autant de fois qu'il y a de secondes... donc 10 secondes, 10 sleep.

Je ne vois pas comment faire autrement..

Merci pour la lecture, je prends le temps de bien comprendre et je reviens si j'ai des questions.

[Feanorin]

Quant au thread qui "sleep", quelqu'un m'a dit (et mon expérience fait que j'ai tendance à le croire), que le seul thread qui devrait pouvoir "dormir" est un scheduler d'interruptions...
Tous les autres threads n'ont jamais besoin de faire 'sleep'... et si on écrit "sleep" quelque part c'est qu'on a ou bien fait un thread de trop, ou oublié un "wait/signal" quelque part ailleurs ^^

avec une alarme qui te donnera le signal pour le thread .