Discussion :

[analystedodo]

Bonjour,

J'ai réalisé, une ListView qui récupère du texte+ image à partir de ma base MySQL, cette Activity marche parfaitement. Maintenant, j'aimerais afficher dans une autre activité une ListView qui affiche du texte (différent au texte affiché dans l'autre Activity) avec des images. Le problème c'est que le texte apparaît mais l'image non.

Le logcat me retourne cette erreur:

resolveUri failed on bad bitmap uri: android.graphics.Bitmap@438484b8

Quelqu'un saurait-il me dire d'où cela peut-il venir ?

Merci d'avance pour votre aide.

[nicroman]

Hmmm oui... l'URI donné à la Bitmap n'est pas bonne....

Mais si tu ne nous donne pas l'URI, on ne pourra pas de dire ce qui ne va pas...

[analystedodo]

C'est quoi l'URI ? Vous voulez dire l'URL de l'image?? si oui voilà:

Code :

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

URL aURL = new URL("http://10.0.2.2"+ o.getString("logo"));

et lorsque j'ai mis un point d'arrêt et j'ai affiché le contenu de o.getString("logo"), eclipse m'a affiché la bonne URL de l'image à afficher.

Voilà 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
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
public class Messages extends ListActivity {
	private static final String MY_PREFERENCES = "mespreferences";
	TextView txt;
	@Override
	public void onCreate(Bundle savedInstanceState) {
		 super.onCreate(savedInstanceState);
		 setContentView(R.layout.list_enregist);
		 SharedPreferences sharedPreferences = getSharedPreferences(MY_PREFERENCES, 0);
	        String idcdt = sharedPreferences.getString("id", "");
	        ArrayList<NameValuePair> post_idsociete = new ArrayList<NameValuePair>();
			post_idsociete.add(new BasicNameValuePair("id", idcdt));
	        ArrayList<HashMap<String, Object>> mylist = new ArrayList<HashMap<String, Object>>();
	        JSONObject json = JSONfunctions.getJSONUrlParam("http://10.0.2.2/mesRequetes/mesmessages.php",post_idsociete);
	        try{
	        	 if (json != null)
	        	 {
	        	JSONArray  offres_e = json.getJSONArray("messages");
 
		        for(int i=0;i<offres_e.length();i++)
					try {
						{						
							HashMap<String, Object> map = new HashMap<String, Object>();	
							JSONObject o = offres_e.getJSONObject(i);
 
							map.put("message", "message:" + o.getString("message"));
							map.put("first_name","société:"+ o.getString("first_name"));
							map.put("idsoc",o.getString("idsoc"));
							map.put("idrep",o.getString("idrep"));
							map.put("idoff",o.getString("idoff"));
							map.put("logo", "" + o.getString("logo"));
							URL aURL = new URL("http://10.0.2.2"+ o.getString("logo")); 
 
							Bitmap bitmap = BitmapFactory.decodeStream(aURL.openStream());
							map.put("logo",bitmap);
							mylist.add(map);
							Log.i("log_tag","message:"+o.getString("message")+ 
									", Société:"+o.getString("first_name")+
									", num msg: "+o.getString("idrep")+
									", id société: "+o.getString("idsoc")+
									", id de l'offre: "+o.getString("idoff")+
									", logo: "+o.getString("logo")
									);
							}
					}
		        catch(JSONException e)        {
			        	 Log.e("log_tag", "Error parsing data "+e.toString());
			        } catch (MalformedURLException e) {
						// TODO Auto-generated catch block
						e.printStackTrace();
					} catch (IOException e) {
						// TODO Auto-generated catch block
						e.printStackTrace();
					}
	        	 }
	        	 else
	        	 {
	        		AlertDialog.Builder adb = new AlertDialog.Builder(Messages.this);
	     	    	adb.setIcon(R.drawable.exclamation1);
	         		adb.setTitle("Aucun message");
	         		adb.setMessage("Vous n'avez aucun message");
	         		adb.setPositiveButton("OK", new DialogInterface.OnClickListener() {
						public void onClick(DialogInterface arg0, int arg1) {
							// TODO Auto-generated method stub 
						}
	        		 });
	        		adb.show(); 
	        	 }
	        } 
	        catch(JSONException e)        {
	        	 Log.e("log_tag", "Error parsing data "+e.toString());
	        }
	        ListAdapter adapter = new SimpleAdapter(this, mylist , R.layout.list_msgs, 
	                        new String[] { "first_name", "message", "logo", "idsoc", "idrep" }, 
	                        new int[] { R.id.TV_societe, R.id.TV_msg, R.id.logosoc });
	        setListAdapter(adapter);
	        final ListView lv = getListView();
	        lv.setTextFilterEnabled(true);	
	        lv.setOnItemClickListener(new OnItemClickListener(){
 
	        	public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
					// TODO Auto-generated method stub
 
	            		@SuppressWarnings("unchecked")
						HashMap<String, String> o = (HashMap<String, String>) lv.getItemAtPosition(position);        		
	        	        Intent intent_msg = new Intent(Messages.this,	DetailMsg.class);
	        	        intent_msg.putExtra("idrep", o.get("idrep"));
	        	        intent_msg.putExtra("idsoc", o.get("idsoc"));
	        	        intent_msg.putExtra("idoff", o.get("idoff"));
 
	        	        startActivity(intent_msg);
 
 
				}
	        });
	  }
 
 
}

[nicroman]

Ok....

J'ai plein de remarques sur le code:

Commençons par le problême initial. SimpleAdapter est ce qu'il est: un "SimpleAdapter", il ne saura donc pas faire correctement tout un tas de trucs... Entre autre, il *attend* une map de string/string (pas de string/object), je suis étonné d'ailleurs que le code compile sans warning enfin bon... Du coup, pour lui l'objet Bitmap passé dans la map avec la clé "logo" est une chaîne de caractères "android.graphics.Bitmap@438484b8" (le Bitmap.toString()). Il essaye donc de charger cette ressource, et bien sur n'y arrive pas.

Je ne connais pas la véritable raison de l'introduction de SimpleAdapter dans Android mais c'est une vraie plaie (une Liste de Map !!) qui annule les avantages d'un langage fortement typé comme Java. En plus il ne sait pas faire tout un tas de chose comme on vient de le voir.

Autre remarque: l'ensemble des opérations (réseau) est faite dans le "onCreate", ceci est interdit par Android (et même vérifié à partir de Android 3.0+ par l'envoi d'une exception "NetworkOnMainThread")... Prenons un exemple simple... le serveur a un "ping" de 100ms (ping très petit sur 3G), un openConnection/get se fait par 2 appels serveurs (+ l'envoi des données qu'on va considérer comme négligeable), donc chaque "get" "coûte" 200ms au minimum... Avec 15 éléments (pas grand chose donc), on se retrouve avec 3,2s dans le onCreate => bim une "ANR" (Application Not Responding). Et je ne parle ni du cas d'une ouverture de connexion est lente, ni de la recherche DNS si utilisation de noms de domaines !!!!

Je ne prétends pas avoir le "bon" code... mais voici la façon (à mon avis de procéder):
1. Commencer par créer une AsyncTask pour charger les élements.
2. Transformer chaque élément en un objet POJO.
3. Surcharger ArrayAdapter pour réaliser ce qu'on veut à partir de ces objets.
4. A la fin, passer l'adapter à la list-view.

Exemples de code:
MyListEntry.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
 
// POJO
class MyListEntry
{
     String    message;
     String    first_name;
     String    idrep;
     String    idsoc;
     String    idoff;
     Bitmap   logo;
 
     // tous les getter/setters associés... par exemple:
     public Bitmap getLogo() { return this.logo; }
     public void setLogo(Bitmap b) { this.logo = b; }
 
     // JSON reading
     public void readFromJSON(JSONObject  obj) throws IOException
     {
           this.message = obj.getString("message");
           this.first_name = obj.getString("first_name");
           this.idrep = obj.getString("idrep");
           this.idsoc= obj.getString("idsoc");
           this.idoff = obj.getString("idoff");
 
           String logurl = obj.getString("logo");
           try {
                URL aURL = new URL("http://10.0.2.2"+ logurl ); 
                this.logo = BitmapFactory.decodeStream(aURL.openStream());
           } catch (Exception ex) {
                Log.w("MyApp","Failed to load the logo !",ex);
           }
     }
}

MyActivity.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
 
class MyActivity extends ListActivity
{
 
    class EntryAdapter extends ArrayAdapter<MyListEntry>
    {
          public EntryAdapter(MyListEntry[] e) {
              super(e,R.layout.list_msgs);
          }
 
          public View getView(int position) {
               View ret = super.getView(position);
               ViewGroup g = (ViewGroup ret;
               MyListEntry e = getItem(position);
               ((TextView)g.findViewById(R.id.TV_societe)).setText(e.getFirstName());
               ((TextView)g.findViewById(R.id.TV_msg)).setText(e.getMessage());
               ((ImageView)g.findViewById(R.id.logosoc)).setBitmap(e.getLogo());
               return ret;
          }
    }
 
    class LoadingTask extends AsyncTask<,,>
    {
         public MyListEntry[] doInBackground()
         {
              MyListEntry[] ret = null;
              SharedPreferences sharedPreferences = getSharedPreferences(MY_PREFERENCES, 0);
	      String idcdt = sharedPreferences.getString("id", "");
  	      ArrayList<NameValuePair> post_idsociete = new ArrayList<NameValuePair>();
			post_idsociete.add(new BasicNameValuePair("id", idcdt));
 
              try{
                  JSONObject json = JSONfunctions.getJSONUrlParam("http://10.0.2.2/mesRequetes/mesmessages.php",post_idsociete);
	          if (json != null) {
                       JSONArray  offres_e = json.getJSONArray("messages");
                       ArrayList<MyListEntry> alist = new ArrayList<MyListEntry>();
                       for(int i=0;i<offres_e.length();i++) {
                            MyListEntry e = new MyListEntry();
                            e.readFromJSON(offres_e.get(i));
                            alist.add(e);
                       }
                       ret = alist.toArray(new MyListEntry[alist.size()]);
                   }
             } catch (Exception ex) {
                  Log.e("MyApp","Failed to load messages !",ex);
             }
 
             return ret;
         }
 
         public void postExecute(MyListEntry[] entries) {
              onEntriesLoaded(entries);
         }
    }
 
    public void onCreate(Bundle bundle)
    {
         super.onCreate(savedInstanceState);
         setContentView(R.layout.list_enregist);
         (new LoadingTask()).execute();
    }
 
    public void onEntriesLoaded(MyListEntry[] entries)
    {
         if (entries != null) {
               EntryAdapter adapter = new EntryAdapter(entries);
               getListView().setAdapter(adapter);
         } else {
               // message "No Message"
         }
    }
}

[analystedodo]

Je vous remercie pour vos explication ainsi que pour le code que vous m'avez proposé. J'ai essayé de l'implémenter mais l'activity contient beaucoup d'erreurs

A propos, j'ai déjà créé une activity avec le même code (dont vous m'avez fait des remarques) et il marchait très bien

[nicroman]

Je n'ai pas dit que ca ne marchait pas.... mais essayez sur un device ICS, ou même sur une tablette Android 3.0+ et le crash sera immédiat (NetworkOnMainThreadException).

La recommandation d'utilisation d'un thread séparé ne date pas de 3.0 pourtant...
http://developer.android.com/trainin...html#AsyncTask

Quant au code que j'ai posté je n'y a pas mis tout le code, c'est juste un "framework" et il est clair qu'il ne compilera pas en l'état...