Application client_serveur Android

Version imprimable

Bonjour,

j'ai développé une application qui permet de récupérer une liste d'exploitations après avoir saisi une CIN, le problème c'est que la liste ne s'affiche pas sur l'émulateur !!
MainActivity :

Code:

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
package com.example.rech_exploi;
 
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.List;
 
import org.apache.http.HttpResponse;
import org.apache.http.NameValuePair;
import org.apache.http.client.ClientProtocolException;
import org.apache.http.client.HttpClient;
import org.apache.http.client.entity.UrlEncodedFormEntity;
import org.apache.http.client.methods.HttpPost;
import org.apache.http.impl.client.DefaultHttpClient;
import org.apache.http.message.BasicNameValuePair;
import org.json.JSONArray;
import org.json.JSONException;
import org.json.JSONObject;
 
import android.app.AlertDialog;
import android.app.ListActivity;
import android.app.ProgressDialog;
import android.net.ParseException;
import android.os.AsyncTask;
import android.os.Bundle;
import android.util.Log;
import android.view.Menu;
import android.view.View;
import android.view.View.OnClickListener;
import android.widget.Button;
import android.widget.EditText;
import android.widget.ListView;
import android.widget.TextView;
import android.widget.Toast;
 
import com.example.rech_exploi.MonAdapteur.MonAdapteurListener;
 
public class MainActivity extends ListActivity implements MonAdapteurListener{
	Button btnRechercher;
	TextView txtMsg;
	EditText edtMsg;
	JSONObject json_data=null;
	HttpResponse response=null;
	//ArrayList<String> donnees = new ArrayList<String>();
	ArrayList<Exploitation> exploitations = new ArrayList<Exploitation>();
	private ProgressDialog progressDialog;
 
 
	@Override
	protected void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
		setContentView(R.layout.activity_main);
		progressDialog = new ProgressDialog(MainActivity.this);
 
		btnRechercher = (Button)findViewById(R.id.btnRechercher);
		edtMsg = (EditText) findViewById(R.id.edtMsg);
 
	}
	public void onClickNom(Exploitation item, final int position) {
		android.app.AlertDialog.Builder builder = new AlertDialog.Builder(this);
		builder.setIcon(R.drawable.icon);
		builder.setTitle("Exploitation");
 
		builder.setMessage("Vous avez cliqué sur : " + item.nom);
		builder.setNegativeButton("Non", null);
		builder.setPositiveButton("Oui", null);
		/*builder.setPositiveButton("Oui",new android.content.DialogInterface.OnClickListener() {
			public void onClick(DialogInterface dialog, int arg1) {
				Toast.makeText(this,"id"+exploitations.get(position).id, Toast.LENGTH_SHORT).show();
			}
		}
	);*/
		builder.show(); 
	}
 
 
 
	public void Rechercher(View v){
		new asynchtask().execute();        
	}		
 
 
 
 
 
	@Override
	public boolean onCreateOptionsMenu(Menu menu) {
		// Inflate the menu; this adds items to the action bar if it is present.
		getMenuInflater().inflate(R.menu.main, menu);
		return true;
	}
 
 
	class asynchtask extends AsyncTask<Void, Void, Void>{
 
		@Override
		protected void onPreExecute() {
			super.onPreExecute();
			exploitations.clear();
			progressDialog.setMessage("Chargement en cours");
			progressDialog.show();
		}
 
		@Override
		protected Void doInBackground(Void... arg0) {
 
 
			StringBuffer sb = new StringBuffer("");
			BufferedReader br = null;
			try{
				HttpClient client = new DefaultHttpClient();
				HttpPost post = new HttpPost("http://10.0.2.2/webService/Recherche.php");
				String CIN = edtMsg.getText().toString();
				if(CIN.length() > 0){
					try{
						List<NameValuePair> donnees = new ArrayList<NameValuePair>(1);
						donnees.add(new BasicNameValuePair("message", CIN));
						post.setEntity(new UrlEncodedFormEntity(donnees));
					    response = client.execute(post);
						//edtMsg.setText("");
						//Toast.makeText(MainActivity.this, "CIN envoyé!", Toast.LENGTH_SHORT).show();
					    Log.e("", "CIN envoye");
					}
					catch(ClientProtocolException e){
						e.printStackTrace();
					}
					catch(IOException e){
						e.printStackTrace();
					}
				}
				else
					Toast.makeText(MainActivity.this,"Ce champ ne peut etre vide!", Toast.LENGTH_SHORT).show();
				InputStream is = response.getEntity().getContent();
				br = new BufferedReader(new InputStreamReader(is));
				String ligneLue = br.readLine();
				while(ligneLue !=null){
					sb.append(ligneLue);
					sb.append("\n");
					ligneLue = br.readLine();
				}
 
 
 
				//Toast.makeText(MainActivity.this, sb+" result", Toast.LENGTH_SHORT).show();
 
 
			}
 
 
			catch(Exception e){
				e.printStackTrace();
 
				//Log.e("exception", e.getMessage());
 
				Toast.makeText(MainActivity.this, "une erreur est survenue"+e.getMessage(), Toast.LENGTH_SHORT).show();
			}
			finally{
				if(br !=null){
					try{
						br.close();
					}catch(IOException e){
						e.printStackTrace();
						Log.e("exception", e.getMessage());
						Toast.makeText(MainActivity.this, "une erreur grave est survenue", Toast.LENGTH_SHORT).show();
 
					}
				}
			}
 
 
			try{
		          JSONArray jArray = new JSONArray(sb.toString());
 
		             for(int i=0;i<jArray.length();i++)
		             {
 
		                   json_data = jArray.getJSONObject(i);
		                   //donnees.add(json_data.getString("CIN"));
		                   //donnees.add(json_data.getString("exploit"));
		                   exploitations.add(new Exploitation(json_data.getInt("id"),json_data.getString("Nom"), json_data.getString("adr")));
 
		               }
 
		            }
		            catch(JSONException e){
		             Log.i("tagjsonexp",""+e.toString());
		            } catch (ParseException e) {
		             Log.i("tagjsonpars",""+e.toString());
		       }
 
 
			return null;
		}
 
		@Override
		protected void onPostExecute(Void result) {
			if (progressDialog.isShowing()) {
				progressDialog.dismiss();
            }
 
			ListView list = (ListView) MainActivity.this.findViewById(android.R.id.list);
			MonAdapteur adpt = new MonAdapteur(getApplicationContext(),exploitations);
			adpt.addListener(MainActivity.this);
 
	        list.setAdapter(adpt);
 
			//setListAdapter(new ArrayAdapter<String>(MainActivity.this, android.R.layout.simple_expandable_list_item_1, donnees));
            edtMsg.setText("");
			super.onPostExecute(result);
			Log.e("asynchtask", "fin");
		}
 
	}
 
}

MonAdapteur

Code:

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
package com.example.rech_exploi;
 
import java.util.ArrayList;
 
import android.content.Context;
import android.content.Intent;
import android.view.LayoutInflater;
import android.view.View;
import android.view.View.OnClickListener;
import android.view.ViewGroup;
import android.widget.BaseAdapter;
import android.widget.ImageView;
import android.widget.TextView;
 
public class MonAdapteur extends BaseAdapter {
 
	private ArrayList<Exploitation> exploitations;
	private LayoutInflater myInflater;
	private Context context;
 
	public MonAdapteur (Context context, ArrayList<Exploitation> _exploitations)
	{
		this.myInflater = LayoutInflater.from(context);
		this.exploitations = _exploitations;
		this.context=context;
	}
 
	@Override
	public int getCount() {
		return this.exploitations.size();
	}
 
	@Override
	public Object getItem(int arg0) {
		return this.exploitations.get(arg0);
	}
 
	@Override
	public long getItemId(int position) {
		return position;
	}
 
	public static class ViewHolder {
		TextView text01;
		TextView text02;
		ImageView image;
	}
 
	@Override
	public View getView(int position, View convertView, ViewGroup parent) {
     ViewHolder holder;
 
		if (convertView == null)
		{
			convertView = myInflater.inflate(R.layout.listitem, null);
			holder = new ViewHolder();
			holder.text01 = (TextView) convertView.findViewById(R.id.txtNom);
			holder.text02 = (TextView) convertView.findViewById(R.id.txtDetail);
			convertView.setTag(holder);
		} else {
			holder = (ViewHolder) convertView.getTag();
		}
 
		holder.text01.setText(exploitations.get(position).nom);
		holder.text02.setText(exploitations.get(position).detail);
 
		holder.image=(ImageView) convertView.findViewById(R.id.icone);
		holder.image.setTag(position);
		holder.image.setOnClickListener(new OnClickListener() {
 
			@Override
			public void onClick(View v) {
				Integer position = (Integer)v.getTag();
				Intent intent = new Intent(context,AndroidGPSTrackingActivity.class);
				intent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
				intent.putExtra("nom", exploitations.get(position).nom);
				intent.putExtra("detail", exploitations.get(position).detail);
				intent.putExtra("id", exploitations.get(position).id);
				context.startActivity(intent);
				//Toast.makeText(context,"id"+exploitations.get(position).id, Toast.LENGTH_SHORT).show();
				//sendListener(exploitations.get(position), position);
			}
        });
 
 
		return convertView;
 
		//this.exploitations.clear();
		//((BaseAdapter)list.getAdapter()).notifyDataSetChanged();
 
 
	}
 
    // le mécanisme pour gérer l'ajout de listener sur notre adapter
 
    //Contient la liste des listeners
 
	private ArrayList<MonAdapteurListener> mListListener = new ArrayList<MonAdapteurListener>();
 
	//Pour ajouter un listener sur notre adapter
    public void addListener( MonAdapteurListener aListener) {
    	mListListener.add(aListener);
    }
    private void sendListener(Exploitation item, int position) {
    	for(int i = mListListener.size()-1; i >= 0; i--) {
    		mListListener.get(i).onClickNom(item, position);
    	}
    }
 
	//Interface pour écouter les évènements sur le nom d'une personne
 
   public interface MonAdapteurListener {
   	public void onClickNom(Exploitation item, int position);
   }
}

Exploitation

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
package com.example.rech_exploi;
 
public class Exploitation {
	int id;
	String nom;
	String detail;
 
	public Exploitation (int _id,String _nom, String _detail)
	{
		id =_id;
		nom = _nom;
		detail = _detail;
	}
}

Voici le web service

Code:

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
 
<?php
 
if(isset($_POST) && !empty($_POST)){
		$rec=$_POST['message'];
		try{
			//include('conf.ini.php');
			$connexion = new PDO('mysql:host=localhost;dbname=test3','root','');	
			$req = $connexion->query("SELECT id,Nom,adr FROM exploitations WHERE CIN='".$rec."'");
			 while($donnees=$req->fetch()){
					$output[]=$donnees;
					$CIN=$donnees['id'];
					$exploitation=$donnees['Nom'];
					$adr=$donnees['adr'];
		}
 
			print(json_encode($output));
 
	}
	catch (Exception $e){
						die('Erreur : ' .$e->getMessage());
					}
		}
		else echo "veuillez entrez votre CIN";
 
?>

et voici l'erreur qu'affiche le logCat:

Code:

1
2
3
4
09-24 15:36:36.755: W/EGL_emulation(2160): eglSurfaceAttrib not implemented
09-24 15:36:37.927: E/(2160): CIN envoye
09-24 15:36:38.165: I/tagjsonexp(2160): org.json.JSONException: Value <br of type java.lang.String cannot be converted to JSONArray
09-24 15:36:38.484: E/asynchtask(2160): fin

Merci d'avance!

Pour l'erreur, il semble que le serveur (le script PHP) renvoie du HTML au lieu du JSON attendu (c'est d'ailleurs le cas si le script a une erreur... alors qu'il devrait produire une erreur HTTP #500, c'est aussi le cas si il n'y a pas de "CIN" ce qui devrait produire une erreur HTTP #400).
D'ailleurs à ce sujet, ou est traité le cas ou la réponse ne renvoie pas un statut "OK" ?
Code:

1 2 3 4 if (response.getStatusLine().getResponseCode() >= 400) { // erreur HTTP }
Pour le reste il y a pas mal de choses à dire...

Qu'est ce que tout le monde a, à vouloir utiliser ce code:
Code:

1 2 3 4 5 6 7 8 9 InputStream is = response.getEntity().getContent(); br = new BufferedReader(new InputStreamReader(is)); String ligneLue = br.readLine(); while(ligneLue !=null){ sb.append(ligneLue); sb.append("\n"); ligneLue = br.readLine(); }
:calim2::calim2::calim2::calim2:
(en plus le stream n'est pas fermé, et le charset non défini)
On dirait du copy/paste d'un mauvais tutoriel....

Bon ce code peut *entièrement* (et avantageusement) être remplacé par:
Code:

1 2 String resultAsString = EntityUtils.toString(response.getEntity()); JSONArray jArray = new JSONArray(resultAsString);
Ensuite:

Code:

String CIN = edtMsg.getText().toString();

Est à priori interdit en dehors du thread "UI"... (pour des raisons d'accès concurrent). Et devrait être donc réalisé par le thread UI en amont de l'AsyncTask.

La gestion des exceptions n'est pas bonne...
Si il y a une erreur de communication on commence par faire un catch de cette erreur... ne pas la loguer (e.printStackTrace au lieu de

Code:

Log.e("MaClasse","Erreur de communication",e);

)... et le programme continue quand même... Sur

Code:

InputStream is = response.getEntity().getContent();

alors que response n'existe pas...

D'ailleurs ou est déclaré response ? je ne l'ai pas vu dans la tâche... C'est pourtant une variable privée à celle-ci (et même au doInBackground !).

L'utilisation des "Log" est erroné...
La fonction indique le "niveau" d'erreur ("d" un message pour le programmeur, "i" un message d'information important, "w" une erreur inattendue mais récupérable, "e" une erreur irrécupérable).
Le premier paramètre est *toujours* un TAG qui indique quelle partie du code est exécutée... ainsi:
Code:

1 2 3 4 5 catch(JSONException e){ Log.i("tagjsonexp",""+e.toString()); } catch (ParseException e) { Log.i("tagjsonpars",""+e.toString()); }
Les deux catch correspondent au même "try" et donc devraient avoir le même tag (perso je garde le même tag pour toute la classe).

Le second paramètre est un message... c'est LA qu'on dit le problème... ""+e.toString() est complètement inutile (on dirait du php), e.toString() suffirait, sauf que c'est pas bon... car....
Le troisième paramètre est l'exception elle même. Donc il faut écrire:
Code:

1 2 3 4 5 catch(JSONException e){ Log.e("asynchtask","Erreur JSON",e); } catch (ParseException e) { Log.e("asynchtask","Erreur de parsing",e); }
Dans la même veine que la "reponse"... exploitations est définie au niveau de l'activité... avec une "ArrayList" (donc non protégé contre les accès concurrents). L'accès à cette liste est par conséquent reservée à la tâche (d'ailleurs il n'y a qu'elle qui s'en sert vraiment).

Code:

super.onPostExecute(result);

est inutile (onPostExecute ne fait rien par défaut).

Code:

Log.e("asynchtask", "fin");

ce n'est pas une erreur... mais du debug : utiliser "d"

Le nom des classes n'est pas bon...
Outre le fait que "asynchtask" ne suit pas la "norme" de nommage en Java (AsynchTask) son nom est beaucoup trop proche de "AsyncTask" et n'indique nullement ce qu'elle fait (Un appel à recherche.php). C'est un coup à se mélanger les pinceaux.

Pour finir... pourquoi faire hériter MonAdapter de BaseAdapter et recoder (en oubliant deux trois trucs) ArrayAdapter<Exploitation> ?

Bonne nouvelle "Exploitation" est bien... sauf qu'il manque les "getters" typiques aux "POJOs" (getId(), getNom(), getDetail()).

Comme j'avais un peu de temps... voici le code "corrigé":

MainActivity:

Code:

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
 
public class MainActivity extends ListActivity {
	private Button btnRechercher;
	private TextView txtMsg;
	private EditText edtMsg;
 
	private ProgressDialog progressDialog;
 
 
	@Override
	protected void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
		setContentView(R.layout.activity_main);
 
		progressDialog = new ProgressDialog(MainActivity.this);
 
		btnRechercher = (Button)findViewById(R.id.btnRechercher);
		edtMsg = (EditText) findViewById(R.id.edtMsg);
	}
 
        @Override
        protected void onListItemClick (ListView l, View v, int position, long id)
        {
              ArrayAdapter<Exploitation> adapter = (ArrayAdapter<Exploitation>)getListAdapter();
              Exploitation item = adapter.getItem(position);
 
		android.app.AlertDialog.Builder builder = new AlertDialog.Builder(this);
		builder.setIcon(R.drawable.icon);
		builder.setTitle("Exploitation");
 
		builder.setMessage("Vous avez cliqué sur : " + item.getNom());
		builder.setNegativeButton("Non", null);
		builder.setPositiveButton("Oui", null);
 
		builder.show(); 
	}
 
 
 
	@Override
	public boolean onCreateOptionsMenu(Menu menu) {
		getMenuInflater().inflate(R.menu.main, menu);
		return true;
	}
 
        // la tâche prend en paramètre le "CIN" et renvoit une liste d'exploitations
	class RechercheTask extends AsyncTask<String, Void, ArrayList<Exploitation>>{
 
		@Override
		protected void onPreExecute() {
			super.onPreExecute();
			progressDialog.setMessage("Chargement en cours");
			progressDialog.show();
		}
 
		@Override
		protected Void doInBackground(String ... args) {
 
                       ArrayList<Exploitation>   exploitations = new ArrayList<Exploitation>();
 
                       try {
 			    HttpClient client = new DefaultHttpClient();
			    HttpPost post = new HttpPost("http://10.0.2.2/webService/Recherche.php");
 
			    List<NameValuePair> donnees = new ArrayList<NameValuePair>();
			    donnees.add(new BasicNameValuePair("message", args[0]));
			    post.setEntity(new UrlEncodedFormEntity(donnees));
 
			    response = client.execute(post);
			    Log.d("RechercheTask","Requête envoyée");
 
			    if (response.getStatusLine().getStatusCode()>=400)
			        throw new HttpException(response.getReasonPhrase());
 
			    String resultStr = EntityUtils.toString(response.getEntity());
			    Log.d("RechercheTask","Réponse reçue: "+resultStr);
 
 			    JSONArray jArray = new JSONArray(resultStr);
			    for(int i=0;i<jArray.length();i++) {
			        JSONObject json_data = jArray.getJSONObject(i);
			        exploitations.add(new Exploitation(json_data.getInt("id"),json_data.getString("Nom"), json_data.getString("adr")));
 			    }
		       } catch (Exception ex) {
		           Log.e("RechercheTask","Erreur recherche",ex);
		           Toast.makeText(MainActivity.this,"La recherche a échouée !", Toast.LENGTH_LONG).show();
		       }
 		       return exploitations;
		}
 
		@Override
		protected void onPostExecute(ArrayList<Exploitation> exploitations) {
		    if (progressDialog.isShowing()) 
		        progressDialog.dismiss();
 
		    MainActivity.this.setListAdapter(new MonAdapteur(MainActivity.this,exploitations));
		    Log.d("RechercheTask","Adapteur mis à jour");
		}
 
	}
 
}

Ensuite... L'adapteur:

Code:

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
 
public class MonAdapteur extends ArrayAdapter<Exploitation> {
 
	private Context context;
 
	public MonAdapteur (Context context, ArrayList<Exploitation> exploitations)
	{
		super(context,R.id.exploitation_layout,R.id.txtNom,exploitations);
	}
 
 
	public static class ViewHolder {
		TextView text01;
		TextView text02;
		ImageView image;
	}
 
 
	@Override
	public View getView(int position, View convertView, ViewGroup parent) {
     ViewHolder holder;
 
		if (convertView == null)
		{
			convertView = super.getView(position,convertView,parent);
			holder = new ViewHolder();
			holder.text01 = (TextView) convertView.findViewById(R.id.txtNom);
			holder.text02 = (TextView) convertView.findViewById(R.id.txtDetail);
			convertView.setTag(holder);
		} else {
			holder = (ViewHolder) convertView.getTag();
		}
 
		holder.text01.setText(exploitations.get(position).nom);
		holder.text02.setText(exploitations.get(position).detail);
 
		holder.image=(ImageView) convertView.findViewById(R.id.icone);
		holder.image.setTag(position);
 
		return convertView;
 	}
}