Problème affichage AlertDialog

**aldagon72** · 27/03/2012, 09h08

Salut,

Je dev une appli dans laquelle il est nécessaire de s'identifier. Lors de l'identification, le nom et le pass sont envoyés à un serveur qui va vérifier si ils sont correct. Si ce n'est pas le cas, je dois afficher un message disant qu'il y a une erreur d'identification.
Le problème, c'est que ce que j'ai fais fait planter l'appli.

Ceci est le morceau de code appelant la méthode que j'ai crée pour afficher l'AlertDialog

Code :

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

1
2
3
4
	if(reponse.equals("")) {
		ident.setReponseLue(false);
		ident.msgErreur("Erreur", "Erreur identification");
	}

Et voici le code de la fonction msgErreur

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	public void msgErreur(String titre, String message) {
		AlertDialog.Builder builder = new AlertDialog.Builder(this);
		builder.setMessage(message);
		builder.setTitle(titre);
		builder.setPositiveButton("OK", new DialogInterface.OnClickListener() {
			@Override
			public void onClick(DialogInterface dialog, int id) {
				IdentificationActivity.this.finish();
			}
		});
		AlertDialog alert = builder.create();
		alert.show();
	}

**Hizin** · 27/03/2012, 10h55

Quelle est l'erreur (stacktrace complet avec pointage vers les lignes mises en causes s'il te plaît) ?

Là, pour le moment, les deux points que je vois :

c'est une NullPointerException sur response.equals("")
Une exception liée à l'EDT "can't create ... on a thread which have not call Looper.prepare" (quelque chose comme ça), c'est à dire que tu tenterai d'influer sur le thread graphique via un autre thread. Dans ce cas-là, appeler myActivity.runOnUiThread(new Runnable {//code ici});.

**aldagon72** · 27/03/2012, 11h07

C'est bien ça c'est une erreur NullPointerException que j'ai sur mon equals. Et elle se répète en boucle vu que j'ai un while.
Le hic c'est que j'ai beau cherché, je vois pas comment régler le souci

**Hizin** · 27/03/2012, 11h17

Pour éviter la NPE, tu peux faire "".equals(reponse);, mais ça ne répondra aucunement à ton besoin (afficher un message d'erreur dans le code d'une mauvaise réponse).
Fais donc un test if (reponse == null || "".equals(reponse)).

Pour le possible problème de communication/réponse, impossible de répondre, vu que le code n'est pas là

**aldagon72** · 27/03/2012, 11h30

Alors voila ce que me dis le Logcat :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
03-27 11:24:57.550: E/IdentificationActivity(2825): S: Error
03-27 11:24:57.550: E/IdentificationActivity(2825):	java.lang.RuntimeException: Can't create handler inside thread that has not called Looper.prepare()
03-27 11:24:57.550: E/IdentificationActivity(2825): 	at android.os.Handler.<init>(Handler.java:121)
03-27 11:24:57.550: E/IdentificationActivity(2825): 	at android.app.Dialog.<init>(Dialog.java:100)
03-27 11:24:57.550: E/IdentificationActivity(2825): 	at android.app.AlertDialog.<init>(AlertDialog.java:96)
03-27 11:24:57.550: E/IdentificationActivity(2825): 	at android.app.AlertDialog$Builder.create(AlertDialog.java:891)
03-27 11:24:57.550: E/IdentificationActivity(2825): 	at projet.viametris.IdentificationActivity.msgErreur(IdentificationActivity.java:92)
03-27 11:24:57.550: E/IdentificationActivity(2825): 	at projet.viametris.ClientThread.run(ClientThread.java:57)
03-27 11:24:57.550: E/IdentificationActivity(2825): 	at java.lang.Thread.run(Thread.java:1020)
03-27 11:24:57.560: E/IdentificationActivity(2825): C: Error
03-27 11:24:57.560: E/IdentificationActivity(2825): java.lang.NullPointerException
03-27 11:24:57.560: E/IdentificationActivity(2825): 	at projet.viametris.ClientThread.run(ClientThread.java:71)
03-27 11:24:57.560: E/IdentificationActivity(2825): 	at java.lang.Thread.run(Thread.java:1020)

Et voici le code complet de ma méthode run() :

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
36
37
38
39
40
41
42
43
44
45
46
	public void run() {
		// TODO Auto-generated method stub
		try {
			Log.d("IdentificationActivity", "C: Connecting...");
			Socket socket = new Socket("192.168.58.22", 1234);
			connected = true;
			IdentificationActivity ident = (IdentificationActivity)ThreadAppelant;
			do {
			if(connected) {
				try {
					Log.d("IdentificationAcitivity", "C: Sending Command");
					PrintWriter out = new PrintWriter(new BufferedWriter(new OutputStreamWriter(socket.getOutputStream())), true);
					BufferedReader rd = new BufferedReader(new InputStreamReader(socket.getInputStream()));
					out.println("IDENT,"+nom+","+pass);
					out.flush();
					Log.d("IdentificationActivity", "C: Sent");
					if(ThreadAppelant instanceof IdentificationActivity)
					{
						reponse = rd.readLine();
						ident.setReponseConnection(reponse);
						if(reponse == null || "".equals(reponse)) {
							ident.setReponseLue(false);
							ident.msgErreur("Erreur", "Erreur identification");
						}
						else {
							ident.setReponseLue(true);
						}
						synchronized(ThreadAppelant) {
							ThreadAppelant.notify();
						}
					}
				}
				catch(Exception e) {
					Log.e("IdentificationActivity", "S: Error", e);
				}
			}
			}while(ident.getReponseConnection().equals(""));
			socket.close();
			Log.d("IdentificationAcitivty", "C: Closed");
		}
		catch(Exception e) {
			Log.e("IdentificationActivity", "C: Error", e);
			connected = false;
		}
 
	}

**Hizin** · 27/03/2012, 11h40

Gné, hé, hé, seconde erreur prévue aussi :

Envoyé par Hizin

Une exception liée à l'EDT "can't create ... on a thread which have not call Looper.prepare" (quelque chose comme ça), c'est à dire que tu tenterai d'influer sur le thread graphique via un autre thread. Dans ce cas-là, appeler myActivity.runOnUiThread(new Runnable {//code ici});.

Donc, tout ce que tu as à faire, c'est d'encapsuler le code servant à créer la AlertDialog dans la méthode "run()" du Runnable que tu créeras

Pour la NPE, tu fais un "set" de ta réponse (dans le cas présent, à null), via l'instruction ident.setReponseConnection(reponse);, donc tu as de nouveau une possibilité de NPE à la ligne while(ident.getReponseConnection().equals(""));

**aldagon72** · 27/03/2012, 11h50

Envoyé par Hizin

Donc, tout ce que tu as à faire, c'est d'encapsuler le code servant à créer la AlertDialog dans la méthode "run()" du Runnable que tu créeras

Donc ce que j'ai a faire, c'est placer le code qui créé mon AlertDialog dans la méthode run() c'est bien ça ?

Et pour la seconde NPE, je suis censé régler le pb comment ?

**Hizin** · 27/03/2012, 12h07

Envoyé par Aldagon72

Donc ce que j'ai a faire, c'est placer le code qui créé mon AlertDialog dans la méthode run() c'est bien ça ?

Dans la méthode Run de la classe anonyme Runnable que tu créeras pour l'occasion (c'est une des possibilités).

Envoyé par Aldagon72

Et pour la seconde NPE, je suis censé régler le pb comment ?

De la même manière que le premier

**aldagon72** · 28/03/2012, 16h36

je reposte le code de ma méthode run() :

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
	public void run() {
		// TODO Auto-generated method stub
		try {
			Log.d("IdentificationActivity", "C: Connecting...");
			Socket socket = new Socket("192.168.58.22", 1234);
			connected = true;
			IdentificationActivity ident = (IdentificationActivity)ThreadAppelant;
			do {
			if(connected) {
				try {
					Log.d("IdentificationAcitivity", "C: Sending Command");
					PrintWriter out = new PrintWriter(new BufferedWriter(new OutputStreamWriter(socket.getOutputStream())), true);
					BufferedReader rd = new BufferedReader(new InputStreamReader(socket.getInputStream()));
					out.println("IDENT,"+nom+","+pass);
					out.flush();
					Log.d("IdentificationActivity", "C: Sent");
					if(ThreadAppelant instanceof IdentificationActivity)
					{
						reponse = rd.readLine();
						ident.setReponseConnection(reponse);
						if(reponse == null || "".equals(reponse)) {
							ident.setReponseLue(false);
							AlertDialog.Builder builder = new AlertDialog.Builder(this);
							builder.setMessage("Erreur identification");
							builder.setTitle("Erreur");
							builder.setPositiveButton("OK", new DialogInterface.OnClickListener() {
								@Override
								public void onClick(DialogInterface dialog, int id) {
									ClientThread.this.finish();
								}
							});
							AlertDialog alert = builder.create();
							alert.show();
						}
						else {
							ident.setReponseLue(true);
						}
						synchronized(ThreadAppelant) {
							ThreadAppelant.notify();
						}
					}
				}
				catch(Exception e) {
					Log.e("IdentificationActivity", "S: Error", e);
				}
			}
			}while(ident.getReponseConnection().equals(""));
			socket.close();
			Log.d("IdentificationAcitivty", "C: Closed");
		}
		catch(Exception e) {
			Log.e("IdentificationActivity", "C: Error", e);
			connected = false;
		}
 
	}

J'ai fais tourner l'appli en mode debug, et j'ai constaté qu'il générait une exception lors de l'exécution de la ligne 23.

**Hizin** · 29/03/2012, 08h13

Envoyé par Hizin

Une exception liée à l'EDT "can't create ... on a thread which have not call Looper.prepare" (quelque chose comme ça), c'est à dire que tu tenterai d'influer sur le thread graphique via un autre thread. Dans ce cas-là, appeler myActivity.runOnUiThread(new Runnable {//code ici});.

Donc, tout ce que tu as à faire, c'est d'encapsuler le code servant à créer la AlertDialog dans la méthode "run()" du Runnable que tu créeras

Ton erreur vient du fait que tu veuilles influer sur l'EDT dans un autre thread que le thread principal. Cela te provoque directement cette exception là, qui est relativement commune sous Android.

Pour palier à ce problème, la classe Activity possède une méthode nommée "runOnUIThread(Runnable)".

Il te suffit donc simplement d'invoquer cette méthode et de mettre la construction de ton AlertDialog dans le corps de la méthode run() de l'objet Runnable que tu créeras pour l'occasion, par exemple :

Code java :

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
 
if(reponse == null || "".equals(reponse)) {
	ident.setReponseLue(false);
	createAlertDialog("Erreur d'identification");
[...]
 
private void createAlertDialog(final CharSequence message)
	{
		//activity ici étant un attribut représentant l'activité dans laquelle ton traitement se fait (pour pouvoir lier l'AlertDialog à une interface
		activity.runOnUiThread(new Runnable()
			{
				@Override
				public void run()
					{
						AlertDialog.Builder builder = new AlertDialog.Builder(activity);
						builder.setMessage(message);
						builder.setTitle("Erreur");
						builder.setPositiveButton("OK", new 	DialogInterface.OnClickListener() {
							@Override
							public void onClick(DialogInterface dialog, int id) {
								ClientThread.this.finish();
							}
						});
						AlertDialog alert = builder.create();
						alert.show();
					}
			});
	}

Ps : code non vérifié, je n'ai pas d'IDE avec projet Android sous la main.

**aldagon72** · 29/03/2012, 08h34

J'ai test ton code et j'ai une chose qui survient.

Code :

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

activity.runOnUiThread(new Runnable() {...} );

En mettant activity, il ne me le trouve pas, je pense que c'est normal vu que je l'ai pas déclaré.
Mais moi je mettrais à la place le nom de l'activité dans laquelle se situe le code. J'ai essayé et cela me donne :

Cannot make a static reference to the non-static method runOnUiThread(Runnable) from the type Activity

Sinon j'ai essayé de faire ce que me dis Eclipse.

Code :

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

final Activity activity = null;

J'ai ensuite effectué un test en mode debug de mon code, et il ne prend même pas la peine d'entrer dans la méthode run(). Il génère une exception lorsqu'il fait :

Code :

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

activity.runOnUiThread(new Runnable () {...} );

Voici ce que me dis le LogCat

Code :

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

1
2
3
4
5
03-29 08:54:22.531: E/IdentificationActivity(1086): S: Error
03-29 08:54:22.531: E/IdentificationActivity(1086): java.lang.NullPointerException
03-29 08:54:22.531: E/IdentificationActivity(1086): 	at projet.viametris.ClientThread.createAlertDialog(ClientThread.java:89)
03-29 08:54:22.531: E/IdentificationActivity(1086): 	at projet.viametris.ClientThread.run(ClientThread.java:59)
03-29 08:54:22.531: E/IdentificationActivity(1086): 	at java.lang.Thread.run(Thread.java:1020)

Je pense que cette exception provient du fait que activity soit initialisé à null. Mais c'est la déclaration par défaut que me donne Eclipse, et je ne vois pas quoi mettre à la place.

**Hizin** · 29/03/2012, 09h20

Il faut aussi que tu initialises cette variable avec quelque chose, c'est le propre des NPE pour le coup

Là, tu as toute les cartes en main, à toi d'initialiser cette variable avec ce qu'il faut, et ce sera bon (au risque de NPE près à la ligne ident.getReponseConnection().equals(""), comme dit plus tôt).

**aldagon72** · 29/03/2012, 10h19

merci pour l'aide apporté

Problème affichage AlertDialog

Android

Discussions similaires

Partager

Partager