Problème Socket Android

Invité · 30/01/2014, 08h49

Bonjour,

Je développe actuellement une application android avec des sockets et quand je la lance elle s'arrête.Cette application permet d'envoyer des commandes d'une Nexus 7 sous Android KitKat vers un boitier domotique (IPX 800) via le protocole M2M. L'application fonctionne très bien sous Android 2.2 mais pas sur 4.4.2.

Voici le logCat qd je lance l'application :

01-30 07:25:06.183: W/dalvikvm(995): threadid=1: thread exiting with uncaught exception (group=0x40a71930)
01-30 07:25:06.273: E/AndroidRuntime(995): FATAL EXCEPTION: main
01-30 07:25:06.273: E/AndroidRuntime(995): java.lang.RuntimeException: Unable to start activity ComponentInfo{com.android.domotique/com.android.domotique.Admin}: java.lang.NullPointerException
01-30 07:25:06.273: E/AndroidRuntime(995): at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2180)
01-30 07:25:06.273: E/AndroidRuntime(995): at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2230)
01-30 07:25:06.273: E/AndroidRuntime(995): at android.app.ActivityThread.access$600(ActivityThread.java:141)
01-30 07:25:06.273: E/AndroidRuntime(995): at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1234)
01-30 07:25:06.273: E/AndroidRuntime(995): at android.os.Handler.dispatchMessage(Handler.java:99)
01-30 07:25:06.273: E/AndroidRuntime(995): at android.os.Looper.loop(Looper.java:137)
01-30 07:25:06.273: E/AndroidRuntime(995): at android.app.ActivityThread.main(ActivityThread.java:5041)
01-30 07:25:06.273: E/AndroidRuntime(995): at java.lang.reflect.Method.invokeNative(Native Method)
01-30 07:25:06.273: E/AndroidRuntime(995): at java.lang.reflect.Method.invoke(Method.java:511)
01-30 07:25:06.273: E/AndroidRuntime(995): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:793)
01-30 07:25:06.273: E/AndroidRuntime(995): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:560)
01-30 07:25:06.273: E/AndroidRuntime(995): at dalvik.system.NativeStart.main(Native Method)
01-30 07:25:06.273: E/AndroidRuntime(995): Caused by: java.lang.NullPointerException
01-30 07:25:06.273: E/AndroidRuntime(995): at com.android.domotique.Admin.onCreate(Admin.java:117)
01-30 07:25:06.273: E/AndroidRuntime(995): at android.app.Activity.performCreate(Activity.java:5104)
01-30 07:25:06.273: E/AndroidRuntime(995): at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1080)
01-30 07:25:06.273: E/AndroidRuntime(995): at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2144)
01-30 07:25:06.273: E/AndroidRuntime(995): ... 11 more

voici le code java de mon application :

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
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
package com.android.domotique;
 
 
import java.io.BufferedReader;
 
import java.io.BufferedWriter;
 
import java.io.IOException;
 
import java.io.InputStreamReader;
 
import java.io.OutputStreamWriter;
 
import java.io.PrintWriter;
 
import java.net.Inet4Address;
 
import java.net.InetAddress;
 
import java.net.Socket;
 
import java.net.UnknownHostException;
 
 
import android.os.Bundle;
 
import android.view.View;
 
import android.widget.Button;
 
import android.widget.ToggleButton;
 
 
public class Admin extends MainActivity {
 
 
    private Button bPrecedent;
 
    private Button bSuivant;
 
    private ToggleButton Relai1;//Eclairage 1
 
    private ToggleButton Relai2;//Eclairage Ext
 
    private ToggleButton Relai3;//Eclairage Degagement
 
    private ToggleButton Relai4;//Portail
 
 
    String AdresseIP = "10.129.101.193";
 
    int Port = 9870;
 
    Socket SocketIPX;
 
    PrintWriter BuffW;
 
    BufferedReader BuffR;
 
 
    @Override
 
    protected void onCreate(Bundle savedInstanceState) {
 
        super.onCreate(savedInstanceState);
 
 
        bPrecedent = new Button(this);
 
        Relai1 = new ToggleButton(this);
 
        Relai2 = new ToggleButton(this);
 
        Relai3 = new ToggleButton(this);
 
        Relai4 = new ToggleButton(this);
 
 
        setContentView(R.layout.admin);
 
 
        bPrecedent = (Button) findViewById(R.id.Buttonpre);
 
        Relai1 = (ToggleButton) findViewById(R.id.toggleButton1);
 
        Relai2 = (ToggleButton) findViewById(R.id.toggleButton2);
 
        Relai3 = (ToggleButton) findViewById(R.id.toggleButton3);
 
        Relai4 = (ToggleButton) findViewById(R.id.toggleButton4);
 
 
        bPrecedent.setOnClickListener(new View.OnClickListener() {
 
 
            @Override
 
            public void onClick(View v) {
 
// TODO Auto-generated method stub
 
                finish();
 
 
            }
 
        });
 
 
        Relai1.setOnClickListener(new View.OnClickListener() {
 
 
            @Override
 
            public void onClick(View v) {
 
// TODO Auto-generated method stub
 
                Thread tTest = new Thread() {
 
                    public void run() {
 
                        if (Relai1.isChecked())
 
                            CommunicationIPX("Set11");
 
                        else
 
                            CommunicationIPX("Set10");
 
                    }
 
                };
 
                tTest.start();
 
 
            }
 
        });
 
 
        Relai2.setOnClickListener(new View.OnClickListener() {
 
 
            @Override
 
            public void onClick(View v) {
 
// TODO Auto-generated method stub
 
                if (Relai2.isChecked())
 
                    CommunicationIPX("Set21");
 
                else
 
                    CommunicationIPX("Set20");
 
            }
 
        });
 
 
        Relai3.setOnClickListener(new View.OnClickListener() {
 
 
            @Override
 
            public void onClick(View v) {
 
// TODO Auto-generated method stub
 
                if (Relai3.isChecked())
 
                    CommunicationIPX("Set31");
 
                else
 
                    CommunicationIPX("Set30");
 
            }
 
        });
 
 
        Relai4.setOnClickListener(new View.OnClickListener() {
 
 
            @Override
 
            public void onClick(View v) {
 
// TODO Auto-generated method stub
 
                if (Relai1.isChecked())
 
                    CommunicationIPX("Set41");
 
                else
 
                    CommunicationIPX("Set40");
 
            }
 
        });
 
 
        bSuivant.setOnClickListener(new View.OnClickListener() {
 
 
            @Override
 
            public void onClick(View v) {
 
// TODO Auto-generated method stub
 
 
            }
 
        });
 
 
        try {
 
            InetAddress serverAddr = Inet4Address.getByName(AdresseIP);
 
            SocketIPX = new Socket(serverAddr, Port);
 
        } catch (UnknownHostException e1) {
 
            e1.printStackTrace();
 
        } catch (IOException e1) {
 
            e1.printStackTrace();
 
        }
 
 
        //ouverture du buffeur pour envoyer dans le socket qu'on vient de créer
 
        try {
 
            BuffW = new PrintWriter(new BufferedWriter(new OutputStreamWriter(SocketIPX.getOutputStream())), true);
 
        } catch (IOException e) {
 
            // TODO Auto-generated catch block
 
            e.printStackTrace();
 
        }
 
 
        //idem pour recevoir
 
        try {
 
            BuffR = new BufferedReader(new InputStreamReader(SocketIPX.getInputStream()));
 
        } catch (IOException e) {
 
            // TODO Auto-generated catch block
 
            e.printStackTrace();
 
        }
 
 
    }
 
    public String CommunicationIPX(String Ordre) {
 
 
        String sClientMsg = "";
 
 
        //envoie l'ordre passé en argument de la fonction en l'état (si c'est un truc qui n'existe pas, l'IPX nous le dira)
 
        BuffW.println(Ordre);
 
        BuffW.flush();
 
 
        //lire la réponse
 
        try {
 
            sClientMsg = BuffR.readLine();
 
        } catch (IOException e) {
 
            // TODO Auto-generated catch block
 
            e.printStackTrace();
 
        }
 
 
        //return la réponse de l'IPX
 
        return sClientMsg;
 
    }//fin Communication IPX
 
 
}

**Hizin** · 30/01/2014, 10h06

Premièrement : pense à bien indenter ton code. C'est un raccourci sous la plupart des IDE et éditeur de texte. Cela le rend plus lisible et facilite le boulot de relecture pour tout le monde.

Deuxièmement : Les conventions Java.

Classes en UpperCamelCase : débutent par une majuscule, suivi de minuscules, la première lettre de chaque mot est une majuscule.
Variables et méthodes en lowerCamelCase : débutent par une minuscule, première lettre de chaque mot en majuscule.
Constantes : écrites en majuscules avec des tirets bas en séparateur (seul endroit où ceux-ci sont autorisés).

Les respecter aide aussi à la relecture.

Au niveau de ton erreur, tu as une NullPointerException (NPE), l'exception la plus commune en Java. Elle indique que tu invoques une méthodes sur un objet null, ou que tu passes un objet null en paramètre d'une méthode qui ne le supporte pas.

Caused by: java.lang.NullPointerException
at com.android.domotique.Admin.onCreate(Admin.java:117)

NPE dans le package com.android.domotique, classe Admin, méthode onCreate. L'erreur est levée dans le fichier Admin.java, ligne 117.

Avec l'extrait de code que tu as fournis, la correspondance des lignes est KO.

Outre ça, tu as beaucoup d'incohérences et de soucis dans ton code.

Code Java :

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

1
2
3
4
5
6
7
8
9
10
 
        bPrecedent = new Button(this);
 
        [...]
 
        setContentView(R.layout.admin);
 
         [...]
 
        bPrecedent = (Button) findViewById(R.id.Buttonpre);

Tu initialises bPrecedent, tu fais un setContentView, et tu le remplaces. Multiplié par 5 puisque tu le fais sur chaque variable. Ce qui est avant setContentView est inutile.

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
        try {
 
            InetAddress serverAddr = Inet4Address.getByName(AdresseIP);
 
            SocketIPX = new Socket(serverAddr, Port);
 
        } catch (UnknownHostException e1) {
 
            e1.printStackTrace();
 
        } catch (IOException e1) {
 
            e1.printStackTrace();
 
        }
 
 
        //ouverture du buffeur pour envoyer dans le socket qu'on vient de créer
 
        try {
 
            BuffW = new PrintWriter(new BufferedWriter(new OutputStreamWriter(SocketIPX.getOutputStream())), true);
 
        } catch (IOException e) {
 
            // TODO Auto-generated catch block
 
            e.printStackTrace();
 
        }
 
 
        //idem pour recevoir
 
        try {
 
            BuffR = new BufferedReader(new InputStreamReader(SocketIPX.getInputStream()));
 
        } catch (IOException e) {
 
            // TODO Auto-generated catch block
 
            e.printStackTrace();
 
        }

Tout ce code-là te fera obtenir une jolie NetworkOnMainThreadExpection, et il y a de bonnes chances que ce soit la source de ta NPE, la gestion des erreurs étant calamiteuse.

Première chose : pas de printStackTrace sur Android. On ne sait pas où stderr sort. Préfère utiliser Log.e(TAG, message, e); où TAG te permet de filtrer facilement dans le LogCat, message te permet de décrire le log et e est l'exception.

Pourquoi est-ce calamiteux ?

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
29
30
        try {
 
            InetAddress serverAddr = Inet4Address.getByName(AdresseIP);
 
            SocketIPX = new Socket(serverAddr, Port);
 
        } catch (UnknownHostException e1) {
 
            e1.printStackTrace();
 
        } catch (IOException e1) {
 
            e1.printStackTrace();
 
        }
 
 
        //ouverture du buffeur pour envoyer dans le socket qu'on vient de créer
 
        try {
 
            BuffW = new PrintWriter(new BufferedWriter(new OutputStreamWriter(SocketIPX.getOutputStream())), true);
 
        } catch (IOException e) {
 
            // TODO Auto-generated catch block
 
            e.printStackTrace();
 
        }

Si tu as une exception dans le premier bloc, SocketIPX est null, mais c'est pas grave, on continue !
Donc quand tu l'utilises ensuite => NPE !

Un truc un peu mieux serait :

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
try {
 
            InetAddress serverAddr = Inet4Address.getByName(AdresseIP);
 
            SocketIPX = new Socket(serverAddr, Port);
 
            BuffW = new PrintWriter(new BufferedWriter(new OutputStreamWriter(SocketIPX.getOutputStream())), true);
 
            BuffR = new BufferedReader(new InputStreamReader(SocketIPX.getInputStream()));
 
        } catch (UnknownHostException e1) {
 
            Log.e("IPX Connection", "problem resolving host", e1);
 
        } catch (IOException e1) {
 
            Log.e("IPX Connection", "issue while reading", e1);
 
        }

A coté de ça... la NetworkOnMainThreadException, qui va se révéler juste après.
C'est une très mauvaise pratique que de faire des opérations longues dans le Thread graphique. Il faut faire en sorte que l'interface ne soit jamais gelée. Android, depuis HoneyComb (3.0) sanctionne chaque appel réseau sur le thread graphique par l'exception bien nommée : NetworkOnMainThreadException.

Chaque opération potentiellement longue (dont font parti les appels réseaux) doivent être asynchrones. La classe AsyncTask permet de les réaliser assez facilement.

Invité · 30/01/2014, 11h48

Merci pour votre réponse rapide. Je ne comprends pas pourquoi j'ai encore une NPE.

LogCat :

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
01-30 10:34:49.082: E/AndroidRuntime(1547): FATAL EXCEPTION: main
01-30 10:34:49.082: E/AndroidRuntime(1547): java.lang.NullPointerException
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at com.android.domotique.Ipx800.Communication(Ipx800.java:57)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at com.android.domotique.Admin$2.onClick(Admin.java:47)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at android.view.View.performClick(View.java:4204)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at android.widget.CompoundButton.performClick(CompoundButton.java:100)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at android.view.View$PerformClick.run(View.java:17355)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at android.os.Handler.handleCallback(Handler.java:725)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at android.os.Handler.dispatchMessage(Handler.java:92)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at android.os.Looper.loop(Looper.java:137)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at android.app.ActivityThread.main(ActivityThread.java:5041)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at java.lang.reflect.Method.invokeNative(Native Method)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at java.lang.reflect.Method.invoke(Method.java:511)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:793)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:560)
01-30 10:34:49.082: E/AndroidRuntime(1547): 	at dalvik.system.NativeStart.main(Native Method)

Code du main, Admin.java :

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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
package com.android.domotique;
 
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.ToggleButton;
 
public class Admin extends MainActivity {
 
	private Button bPrecedent;
	private ToggleButton relaiEclairage1;//Eclairage 1
	private ToggleButton relaiEclairageExt;//Eclairage Ext
	private ToggleButton relaiEclairageDeg;//Eclairage Degagement
	private ToggleButton relaiPortail;//Portail
	private ToggleButton relaiChauffage;//Chauffage
 
	@Override
	protected void onCreate(Bundle savedInstanceState) {
			super.onCreate(savedInstanceState);
 
		setContentView(R.layout.admin);
 
		bPrecedent = (Button)findViewById(R.id.Buttonpre);
        relaiEclairage1 = (ToggleButton)findViewById(R.id.toggleButton1);
        relaiEclairageExt = (ToggleButton)findViewById(R.id.toggleButton2);
        relaiEclairageDeg = (ToggleButton)findViewById(R.id.toggleButton3);
        relaiPortail = (ToggleButton)findViewById(R.id.toggleButton4);
        relaiChauffage = (ToggleButton)findViewById(R.id.toggleButton5);
 
        bPrecedent.setOnClickListener(new View.OnClickListener() {
 
			@Override
			public void onClick(View v) {
				// TODO Auto-generated method stub
				finish();
 
			}
		});
 
        relaiEclairage1.setOnClickListener(new View.OnClickListener() {
 
			@Override
			public void onClick(View v) {
				// TODO Auto-generated method stub
				Ipx800 essaiIpx = new Ipx800();
				essaiIpx.execute();
				essaiIpx.communicationIpx("Set41");
			}
		});
 
        relaiEclairageExt.setOnClickListener(new View.OnClickListener() {
 
			@Override
			public void onClick(View v) {
				// TODO Auto-generated method stub
 
			}
		});
 
        relaiEclairageDeg.setOnClickListener(new View.OnClickListener() {
 
			@Override
			public void onClick(View v) {
				// TODO Auto-generated method stub
 
			}
		});
 
        relaiPortail.setOnClickListener(new View.OnClickListener() {
 
			@Override
			public void onClick(View v) {
				// TODO Auto-generated method stub
 
			}
		});
 
        relaiChauffage.setOnClickListener(new View.OnClickListener() {
 
			@Override
			public void onClick(View v) {
				// TODO Auto-generated method stub
 
			}
		});
 
	} 
 
 
}

J'ai crée une classe pour utiliser AsyncTask, Ipx800.java

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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
package com.android.domotique;
 
import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.IOException;
import java.io.InputStreamReader;
import java.io.OutputStreamWriter;
import java.io.PrintWriter;
import java.net.Inet4Address;
import java.net.InetAddress;
import java.net.Socket;
import java.net.UnknownHostException;
 
import android.os.AsyncTask;
import android.util.Log;
 
public class Ipx800 extends AsyncTask<Void, Void, Void> {
 
	String adresseIP = "10.129.101.193";
	int portIpx = 9870;
	Socket socketIpx;
	PrintWriter buffW ;
	BufferedReader buffR ; 
 
	@Override
	protected Void doInBackground(Void... params) {
		// TODO Auto-generated method stub
 
			try {
 
	            InetAddress serverAddr = Inet4Address.getByName(adresseIP);
 
	            socketIpx = new Socket(serverAddr, portIpx);
 
	            buffW = new PrintWriter(new BufferedWriter(new OutputStreamWriter(socketIpx.getOutputStream())), true);
 
	            buffR = new BufferedReader(new InputStreamReader(socketIpx.getInputStream()));
 
	        } catch (UnknownHostException e1) {
 
	            Log.e("IPX Connection", "problem resolving host", e1);
 
	        } catch (IOException e1) {
 
	            Log.e("IPX Connection", "issue while reading", e1);
 
	        }
		return null;
	}
 
	public String communicationIpx(String Ordre){
 
		     String sClientMsg = "";
 
		     //envoie l'ordre passé en argument de la fonction en l'état (si c'est un truc qui n'existe pas, l'IPX nous le dira)
		     Log.e("Buffer", "Erreru Buf1");
		     buffW.println(Ordre);
		     Log.e("Buffer", "Erreru Buf2");
		     buffW.flush();
		     Log.e("Buffer", "Erreru Buf3");
 
		     //lire la réponse
		      try {
		       sClientMsg = buffR.readLine();
		    } catch (IOException e) {
		       // TODO Auto-generated catch block
		    	 Log.e("IPX Communicationn", "problem resolving host", e);
		    }
 
		      //return la réponse de l'IPX
		     return sClientMsg;
		  }//fin Communication IPX*/
}

L'erreur NPE semble être au niveau de la ligne buffW.printIn(Ordre) mais je ne comprends pas le problème ?

**Hizin** · 30/01/2014, 12h01

Les paramètres suivent la même convention d'écriture que les variables :p

Si c'est bien sur cette ligne, soit Ordre est null, soit buffW l'est.
Tu n'as pas d'autres erreurs dans ton LogCat par hasard ?