Discussion :

[Matf4ke]

Bonjour,

Comme l'indique le titre, ma creation de bitmap via une url echoue et retourne nul.

J'ai un des url d'image dans une liste ->

Code :

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

agenda.get(i).getPicture() // always return a good image url

Dans unThread je fais ca :

Code :

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

1
2
3
4
5
6
    for (int i = 0; i < agenda.size(); i ++)
        {
            Log.e("TEST", "  = " +agenda.get(i).getPicture());
            Bitmap newBitmap = getBitmapFromURL(agenda.get(i).getPicture()); // getPicture return the url
            imagelist.add(i,newBitmap);
        }

et getBitmapFromURL return null a cause de :

Code :

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

BitmapFactory.decodeStream(input)

dans :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
private Bitmap getBitmapFromURL(final String src) {
            try {
                URL url = new URL(src);
                HttpURLConnection connection = (HttpURLConnection) url.openConnection();
                connection.setDoInput(true);
                connection.connect();
                InputStream input = connection.getInputStream();
                myBitmap = BitmapFactory.decodeStream(input);
                connection.disconnect();
            } catch (IOException e) {
                e.printStackTrace();
            }
    return myBitmap;
}

Code :

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

InputStream input = connection.getInputStream();

fail aussi surement ....

Si quelqu'un a une idee...

[nicroman]

1. Remplace le e.printStackTrace par Log.e("MYTAG","Failed to load image",e);
2. Rajoute des Log.d("MYTAG","Been here done that"); de temps en temps...
3. Gère correctement les exceptions....

Par exemple:

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
 
private Bitmap getBitmapFromURL(final String src) throws Exception {
    Bitmap ret = null;
    URL url = new URL(src);
    HttpURLConnection connection = null;
    try {
         url = (HttpURLConnection) url.openConnection();
         connection.setDoInput(true);
         connection.connect();
         Log.d("MYTAG","Connected to "+url);
         InputStream input = connection.getInputStream();
         ret = BitmapFactory.decodeStream(input);
         Log.d("MYTAG","Bitmap decoded");
     } finally {
         connection.disconnect();
     }
    return ret;
}

et

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
for (int i = 0; i < agenda.size(); i ++)
{
     String url = agenda.get(i).getPicture();
     Log.i("MYTAG", "Loading image "+url);
     try {
         Bitmap newBitmap = getBitmapFromURL(url);
         imagelist.add(i,newBitmap);
     } catch (Exception e) {
         Log.e("MYTAG","Failed to load image",e);
     }
}

Ensuite, poste nous le logcat (avec un filtre sur MYTAG) ...

[nicolofontana12]

A supposer que l'url retourné reference bien une image :
Je voie pas aussi l'interet de declarer une exception deja catcher et pas renvoyée.

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
 
private Bitmap getBitmapFromURL(final String src) {
    Bitmap ret = null;
    URL url = new URL(src);
    HttpURLConnection connection = null;
    try {
         connection = (HttpURLConnection) url.openConnection();
       //  connection.setDoInput(true); //Je voie pas l'interet et c'est //optionnel.
       //  connection.connect(); // permet de connecter...or il est deja //connecté au ressource par openConnection
//Si probleme de timeout ou de temps de lecture
         connection.setConnectTimeout(30000);
	 connection.setReadTimeout(30000);
         Log.d("MYTAG","Connected to "+url);
         InputStream input = connection.getInputStream();
         ret = BitmapFactory.decodeStream(input);
         Log.d("MYTAG","Bitmap decoded");
     } finally {
         connection.disconnect();
     }
    return ret;
}

et

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
int agendaSize=agenda.size();
for (int i = 0; i < agendaSize; i ++)
{
     String url = agenda.get(i).getPicture();
     Log.i("MYTAG", "Loading image "+url);
     try {
         Bitmap newBitmap = getBitmapFromURL(url);
         imagelist.add(i,newBitmap);
     } catch (Exception e) {
         Log.e("MYTAG","Failed to load image",e);
     }
}

[nicolofontana12]

Ton code genera une erreur de nullpointerException : c'est pas url plutot connection

1...

Par exemple:

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
 
private Bitmap getBitmapFromURL(final String src) throws Exception {
    Bitmap ret = null;
    URL url = new URL(src);
    HttpURLConnection connection = null;
    try {
         url = (HttpURLConnection) url.openConnection();
//will trow NullPointerException        
 connection.setDoInput(true);
         connection.connect();
         Log.d("MYTAG","Connected to "+url);
         InputStream input = connection.getInputStream();
         ret = BitmapFactory.decodeStream(input);
         Log.d("MYTAG","Bitmap decoded");
     } finally {
         connection.disconnect();
     }
    return ret;
}

[/code]

Ensuite, poste nous le logcat (avec un filtre sur MYTAG) ...

[nicroman]

oui oui un mauvais copier/coller, qui ne devrait même pas compiler de toute maniere (affectation d'une HttpUrlConnection à une URL ?)

Pour le truc optionnel, j'ai juste recopié le code initialement posté

L'exception n'est pas catchée justement pour être transmise à l'appelant. Le try/finally est là pour fermer la connexion en cas d'erreur justement (sinon, risque de fuite mémoire).

Il y a certainement moyen de "réduire" les exceptions lancées, je n'ai pas regardé exactement ce qui pouvait l'être (certainement IOException, ...)