Discussion :

[aldagon72]

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]

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]

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]

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]

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]

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

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(""));