Discussion :

[grunk]

Bonjour à tous,

J'ai une classe qui lors de son initialisation va chercher une information via une requête HTTP et initialise l'un de ces attribut :

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
class MaClasse
{
	private api_level
 
	public Maclasse()
	{
		getAPI();
	}
 
	public initURL()
	{
		if(api_level == 1)
			url = "...";
		else
			url = "xxx";
	}
 
	private void getAPI()
	{
		new Thread(new Runnable(){
			//Requête HTTP + lecture de réponse
 
			api_level = response; 
			initURL();
		}).start();
	}
}

puis cette classe est passée à un autre objet qui va venir lire l'attribut initialisé (url) et ce dans un thread :

Code :

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

1
2
MaClasse mc = new MaClasse();
AutreClasse.setObjet(mc);

Je n'ai pas encore effectué de test , mais j'imagine que au moment de AutreClasse.setObjet(mc); selon la durée de la requête, il est tout à fait possible que initURL(); n'est pas encore été appelé , exact ?

Dans ce cas comment m'assurer qu'au moment ou je vais faire appel à MaClasse.getUrl(); j'aurais bien une valeur correctement initialisée ?

merci

[le y@m's]

Remarque: Dans l'état actuel, en plus de diverse fautes dans ton code, le thread n'est jamais démarré (méthode start() jamais appelée) dans la méthode getAPI().

Pour ta problémtatique tu peux utiliser l'interface Future et son implementation FutureTask qui permettent d'exécuter une méthode et de récupérer un résultat de façon asynchrone.
Par contre ce n'est pas terrible de lancer une initialisation asynchrone dans le constructeur de ton objet.
Il est préférable de d'abord faire la requête, puis une fois la réponse reçue, d'instancier la classe.

Voici un exemple :

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
31
32
33
34
35
public class MaClasse {
 
    private final int apiLevel;
 
    private final String url;
 
    public MaClasse(int apiLevel) {
        this.apiLevel = apiLevel;
        switch (apiLevel) {
            case 1:
                url = "...";
                break;
            default:
                url = "xxx";
                break;
        }
    }
 
    public static Future<MaClasse> request() {
        final FutureTask<MaClasse> future = new FutureTask<MaClasse>(maClasseRequest);
        final Thread thread = new Thread(future);
        thread.setDaemon(true);
        thread.start();
        return future;
    }
 
    private static final Callable<MaClasse> maClasseRequest = new Callable<MaClasse>() {
        @Override
        public MaClasse call() throws Exception {
            //Requête HTTP + lecture de réponse
            return new MaClasse(responseApiLevel);
        }
    };
 
}

Donc tu récupères le résultat asynchrone comme suit :

Code :

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

Future<MaClasse> future = MaClasse.request();

Je te laisse regarder la javadoc de l'interface Future pour les différentes méthodes permettant de gérer le résultat asynchrone .

[thelvin]

Eh bien, le plus simple est a priori de ne pas utiliser de thread, vu que dans l'exemple que tu donnes il ne sert à rien et te complique la vie.

Dans un scénario plus compliqué où les threads seraient utiles (genre un pool de threads est chargé des opérations d'IO HTTP de toute l'application pour éviter des context switch à gogo,) tu pourrais juste encadrer l'accès à l'URL avec des méthodes synchronized, et faire wait() tant que l'URL n'a pas été récupérée, et notify() quand elle l'a été.