Discussion :

[hannibal813]

Bonjour je dois développer une application permettant d'envoyer une trame à un serveur. J'arrive a créer la connexion a mon serveur mais au moment de l'envoi, mon application s'arrête.
Version Android : 4.1.2
Téléphone : Samsung Galaxy S3

Voila mon code :

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
 
	public void envoyerMessage()
	{
		// Création de l'objet de type socket : 
		Socket socket;
 
		//Creation des objets permettant la lecture et l'écriture :
		BufferedReader in;
		BufferedWriter out;
 
		try 
		{
 
			  socket = new Socket(adresse,port);
 
			  //Création des objets de lecture et d'écriture : 
			  in = new BufferedReader(new InputStreamReader(socket.getInputStream()));
			  out = new BufferedWriter(new OutputStreamWriter(socket.getOutputStream()));
 
			  //Envoie de la trame au serveur: 
			  out.write(trame);
 
			  //On vide le flux d'envoie de donnees : 
			  out.flush();
			  Log.i("TcpClient", "sent: " + trame);
 
			  String reponseServeur = in.readLine() + System.getProperty("line.separator");
			  Log.i("TcpClient", "received: " + reponseServeur);
 
			  socket.close();
 
			  etatConnexion = 1;
 
		} 
 
		catch (UnknownHostException e) 
		{
			// TODO Auto-generated catch block
			e.printStackTrace();
			etatConnexion = 0;
 
		} 
 
		catch (IOException e) 
		{
			// TODO Auto-generated catch block
			e.printStackTrace();
			etatConnexion = 0;
		}
	}

Merci de votre aide

[nicroman]

Il manque des infos.... comme le LogCat (obligatoire si l'application 's'est arrêtée') , et le contexte d'appel de la fonction (par exemple "trame","addresse" et "port" ici ne sont pas définis).


Sinon....


Ne jamais utiliser e.printStackTrace(); mais préférer: Log.e("XXXXX","Erreur lors de xxxxxxx",e); (les xxxxx étant à remplacer par quelque chose de cohérent bien sur).

D'autre part pourquoi catcher les exceptions (et ne pas simplement les propager) ? L'appelant de la fonction ne doit pas savoir qu'une exception a eu lieu ?

Il me semble bizarre d'avoir au même niveau de données "trame" et "addresse" (conceptuellement ce devrait être dans deux objets séparés).


etatConnexion = 0;
etatConnexion = 1;
Utiliser un enum pour définir une variable à états prédéfinis permet d'éviter pas mal de "bugs".